Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Backout failed attempt to fix [32ae34e63a]. | dgp | 2015-09-25 | 1 | -43/+4 |
|\ | |||||
* \ | [e0a7b3e5f8] Add missing UpdateInterest call in DoRead(). | dgp | 2015-09-23 | 1 | -3/+10 |
|\ \ | |/ | |||||
| * | merge 8.5bug_e0a7b3e5f8 | dgp | 2015-09-23 | 1 | -4/+43 |
| |\ | |||||
| * | | Add the critical missing UpdateInterest() call at the exit of DoRead(). | dgp | 2015-08-10 | 1 | -3/+10 |
| | | | | | | | | | | | | | | | | | | (Compare with same approach in DoReadChars()). This involves removing some other calls that are now replaced by the new one. Also marked several UpdateInterest() calls throughout tclIO.c with comments raising the suspicion that they serve no function. | ||||
* | | | [32ae34e63a] Prevent segfaults and data corruption when CopyData() is called ↵ | dgp | 2015-09-23 | 1 | -4/+43 |
|\ \ \ | | |/ | |/| | | | | | | | recursively. Mark new test io-53.20 as "knownBug". It demos some unknown flaw in MoveBytes(). | ||||
| * | | Protect CopyState buffer from conflicting uses when CopyData() is called | dgp | 2015-09-23 | 1 | -0/+7 |
| | | | | | | | | | recursively. Also, have ReflectWatch() always give driver a chance to act. | ||||
| * | | CopyData() now tolerates recursion enough so that io-53.18 does not segfault. | dgp | 2015-09-23 | 1 | -21/+9 |
| | | | |||||
| * | | WIP | dgp | 2015-09-23 | 1 | -3/+47 |
| |/ | |||||
* | | [b1534b438b] Prevent read outside buffer boundaries. | dgp | 2015-07-15 | 1 | -1/+12 |
|\ \ | |/ | |||||
| * | [b1534b438b] Prevent read outside buffer boundaries. | dgp | 2015-07-15 | 1 | -1/+12 |
| | | |||||
* | | Revert some refcount changes on I/O structs. | dgp | 2015-07-02 | 1 | -2/+2 |
| | | | | | | | | | | | | | | These are refcounts on structs, not Tcl_Obj's. Their scheme doesn't suffer the same difficulties and histories as Tcl_Obj's, and they need not copy every detail, appropriate or not, from Tcl_Obj refcount management. The "significant value" -- as dkf puts it -- for the struct refcounting scheme is 0 not 1. | ||||
* | | Another round of refCount consistancy improvements. | jan.nijtmans | 2015-07-02 | 1 | -29/+28 |
| | | |||||
* | | [894da183c8] Test and fix for regression in flushing blocked channels. | dgp | 2015-04-29 | 1 | -1/+4 |
|\ \ | |/ | |||||
| * | [894da183c8] Test and fix for regression in flushing blocked channels. | dgp | 2015-04-29 | 1 | -1/+4 |
| |\ | |||||
| | * | [894da183c8] Fix and test for bug at the point it was introduced.bug_894da183c8 | dgp | 2015-04-29 | 1 | -0/+3 |
| | | | |||||
* | | | [879a0747be] Repair handling of incomplete multi-byte chars at the ends of ↵ | dgp | 2015-04-24 | 1 | -6/+53 |
|\ \ \ | |/ / | | | | | | | input buffers. Also properly reset input encoding flags when cycling through a fleeting EOF condition. | ||||
| * | | [879a0747be] Repair handling of incomplete multi-byte chars at the ends | dgp | 2015-04-24 | 1 | -7/+54 |
| | | | | | | | | | | | | of input buffers. Also properly reset input encoding flags when cycling through a fleeting EOF condition. | ||||
| * | | Backport of channel mem leak fix that got missed before. | dgp | 2015-02-05 | 1 | -3/+15 |
| | | | |||||
* | | | per ticket [e08c2c407b053bbfaab] (and stray code related to ↵ | bch | 2015-02-19 | 1 | -3/+0 |
| | | | | | | | | | | | | [84f208762f172e]) adj code to passing test suite | ||||
* | | | Even clearer failure messages. | dkf | 2015-02-11 | 1 | -16/+17 |
| | | | |||||
* | | | inputProc and outputProc tests run conditionally on TCL_READABLE, ↵ | bch | 2015-02-08 | 1 | -2/+2 |
| | | | | | | | | | | | | TCL_WRITABLE mask | ||||
* | | | switch raw assert() to Tcl_Panic() per discussion w/ dkf | bch | 2015-02-07 | 1 | -7/+17 |
| | | | |||||
* | | | merge [trunk] | bch | 2015-02-07 | 1 | -2/+2 |
|\ \ \ | |||||
| * | | | Fix [08872796bc] | dgp | 2015-01-29 | 1 | -2/+2 |
| | | | | |||||
* | | | | truncateProc *can* be NULL | bch | 2015-01-21 | 1 | -1/+0 |
| | | | | |||||
* | | | | assert() on missing definitions for Tcl_ChannelCreate() required struct ↵ | bch | 2015-01-20 | 1 | -0/+9 |
|/ / / | | | | | | | | | | Tcl_ChannelType{} fields. | ||||
* | | | Now that we're using TCL_ENCODING_NO_TERMINATE - be careful about acting on | dgp | 2015-01-02 | 1 | -2/+1 |
| | | | | | | | | | | | | the contents of dst -- they could be leftovers. Only check bytes reported to have been written and take care to get the assertions right. | ||||
* | | | Use the new TCL_ENCODING_CHAR_LIMIT flag to have the encoding system manage | dgp | 2014-12-23 | 1 | -4/+11 |
| | | | | | | | | | the max chars to read constraint. | ||||
* | | | merge trunk | dgp | 2014-12-23 | 1 | -0/+3 |
|\ \ \ | |||||
| * | | | [7c187a3773] Fix error in managing inStatePtr->inQueueTail value in the | dgp | 2014-12-18 | 1 | -0/+3 |
| | | | | | | | | | | | | byte-moving optimized path of [chan copy]. Thanks to Benno. | ||||
* | | | | With no padding needed for a terminating NUL, there no need for a distinction | dgp | 2014-11-15 | 1 | -5/+4 |
| | | | | | | | | | | | | between the dstNeeded and dstLimit values. | ||||
* | | | | Tcl_ExternalToUtf appends a terminating NUL to its encoded results. | dgp | 2014-11-15 | 1 | -21/+23 |
|/ / / | | | | | | | | | | | | | | | | | | | Perhaps this is a welcome convenience for some callers, but not for Tcl's I/O system, which has no need for that. Added a new flag value TCL_ENCODING_NO_TERMINATE that callers can use to suppress this behavior. This means buffers don't require so much padding, and a tiny bit of processing is saved. Update I/O callers to use the feature. | ||||
* | | | Likely fix for channel mem leaks.core_8_6_3core_8_6_3_rc | dgp | 2014-11-11 | 1 | -3/+15 |
| | | | |||||
* | | | Same patch re-enabling read after EOF, but here applied to 8.6. | dgp | 2014-11-03 | 1 | -42/+184 |
|\ \ \ | |/ / | | | | Likely additional changes needed in the other channel transforms new in 8.6. | ||||
| * | | [5adc350683] Reworked the management of the EOF states to re-enable the | dgp | 2014-11-03 | 1 | -43/+139 |
| | | | | | | | | | | | | ability to read beyond EOF. Plenty of assert()s to keep thing from going off track again. | ||||
| * | | Work in progress restoring ability to [read] after [eof] and get non-empty | dgp | 2014-10-28 | 1 | -21/+67 |
| | | | | | | | | | | | | | | | strings back in those cases where the channel has them to offer. Also working through all the implications of this possibility on Tcl's more exotic channel features, like stacking. | ||||
* | | | Disable assertion until tls bug it detects is fixed. | dgp | 2014-11-01 | 1 | -1/+7 |
| | | | |||||
* | | | [10dc6daa37] New fix for [gets] on non-blocking channel. This time properly ↵ | dgp | 2014-10-18 | 1 | -6/+8 |
|\ \ \ | |/ / | | | | | | | accounts for the effects of ENCODING_LINESIZE. | ||||
| * | | [10dc6daa37] New fix for [gets] on non-blocking channel. This time | dgp | 2014-10-18 | 1 | -6/+8 |
| |\ \ | | | | | | | | | properly accounts for the effects of ENCODING_LINESIZE. | ||||
| | * | | [10dc6daa37] [gets] on a non-blocking channel must take care so that | dgp | 2014-10-17 | 1 | -0/+6 |
| |/ / | | | | | | | | | | | | | | | | | | | 1) At least one call to the channel driver input proc gets made. Failure to do this locks up the channel - catastrophic FAIL. 2) After any driver call reports BLOCKED, don't call again. This is less serious, but FAILs to respect the non-blocking setting. Code corrections and tests included, to restore 8.5.15 compat. | ||||
* | | | [bf7135428c] Restore the Tcl_Write() return value logic. | dgp | 2014-10-10 | 1 | -1/+4 |
|\ \ \ | |/ / | |||||
| * | | [bf7135428c] Restore the Tcl_Write() return value logic. | dgp | 2014-10-10 | 1 | -1/+4 |
| | | | |||||
* | | | [ed29c4da21] Completed fix for [chan copy] handling [chan blocked]. | dgp | 2014-10-10 | 1 | -1/+5 |
| | | | |||||
* | | | [ed29c4da21] Merge patch from 8.5, still not a complete fix. | dgp | 2014-10-10 | 1 | -5/+23 |
|\ \ \ | |/ / | |||||
| * | | [ed29c4da21] Don't let BLOCKED state get converted into a channel error. | dgp | 2014-10-10 | 1 | -5/+23 |
| | | | |||||
* | | | [bc5b790099] Improper calculation of new dstLimit value. New test io-12.7. | dgp | 2014-10-02 | 1 | -3/+2 |
|\ \ \ | |/ / | |||||
| * | | [bc5b790099] Improper calculation of new dstLimit value. New test io-12.7. | dgp | 2014-10-02 | 1 | -3/+2 |
| | | | |||||
* | | | Make corrections to long-false comments. | dgp | 2014-09-05 | 1 | -9/+9 |
| | | | |||||
* | | | Remove ChannelHandlerEvent struct, which has never been used in all of | dgp | 2014-09-04 | 1 | -11/+0 |
| | | | | | | | | | | | | | | | recorded Tcl history. Still need to purge comments of mentions of ChannelHandlerEventProc() which is similarly pre-historic. | ||||
* | | | merge 8.5; fix notifier mask bug and Tcl_Read performance regression | dgp | 2014-08-22 | 1 | -20/+8 |
|\ \ \ | |/ / |