summaryrefslogtreecommitdiffstats
path: root/generic
Commit message (Collapse)AuthorAgeFilesLines
* ckfree() -> Tcl_Free()jan.nijtmans2023-03-061-1/+1
|
* Merge 8.7jan.nijtmans2023-03-051-16/+40
|\
| * Merge-mark (but also remove some lost spacing)jan.nijtmans2023-03-051-1/+1
| |\
| | * Fix [57bfcf43dd]: Remove unreachable code in Tcl_SetWideIntObj()jan.nijtmans2023-03-051-10/+4
| | |
| * | A better fix for Valgrind "still reachable" report in TestcmdtokenCmd().pooryorick2023-03-051-16/+40
| |\ \
| | * | A better fix for Valgrind "still reachable" report in TestcmdtokenCmd().pooryorick2023-03-051-16/+40
| | | |
* | | | Another round of -1 -> TCL_INDEX_NONEjan.nijtmans2023-03-0527-598/+598
| | | |
* | | | More -1 -> TCL_INDEX_NONEjan.nijtmans2023-03-0533-217/+217
| | | |
* | | | Merge 8.7 - Protect zlib errors with check for null interpapnadkarni2023-03-041-10/+19
|\ \ \ \ | |/ / /
| * | | Protect zlib errors with check for null interpapnadkarni2023-03-041-10/+19
| | | |
* | | | Merge 8.7jan.nijtmans2023-03-041-1/+1
|\ \ \ \ | |/ / /
| * | | Merge 8.6jan.nijtmans2023-03-041-1/+1
| |\ \ \ | | |/ / | |/| / | | |/
| | * Fix [1b8df10110]: Tcl_GetTime returns wrong usec value on Windowsjan.nijtmans2023-03-041-1/+1
| | |
| * | Fix Valgrind "still reachable" report in TestcmdtokenCmd().pooryorick2023-03-031-14/+30
| | |
* | | Adapt type-casts to Tcl 9.0 signature of Tcl_Free/Tcl_Realloc/Tcl_AttemptReallocjan.nijtmans2023-03-031-3/+3
| | |
* | | ckfree() shouldn't be used in Tcl 9 core code any morejan.nijtmans2023-03-031-1/+1
| | |
* | | Fix Valgrind "still reachable" report in TestcmdtokenCmd().pooryorick2023-03-031-14/+30
| | |
* | | Merge trunkapnadkarni2023-03-011-6/+6
|\ \ \
| * | | Fix msvc build (with OPTS=symbols)jan.nijtmans2023-03-011-6/+6
| | | |
* | | | Bug [9a978f8323]: crash reading large filesapnadkarni2023-03-011-11/+15
|/ / /
* | | Merge 8.7jan.nijtmans2023-02-282-1/+8
|\ \ \ | |/ /
| * | Put back testcase io-52.20, and re-fix [4a7397e0b3]jan.nijtmans2023-02-282-1/+8
| | |
* | | Merge 8.7jan.nijtmans2023-02-281-1/+1
|\ \ \ | |/ /
| * | Missing ')'jan.nijtmans2023-02-281-1/+1
| | |
* | | Merge 8.7jan.nijtmans2023-02-281-1/+5
|\ \ \ | |/ /
| * | Restore previous behavior for non-blocking mode, as for this mode the ↵jan.nijtmans2023-02-281-1/+5
| | | | | | | | | | | | | | | semantics of [read]/[gets] were not broken. This was the 'some agreement'. The change in line 8286 is necessary for both blocking and non-blocking mode: Whenver the encoding change we need to reset the CHANNEL_ENCODING_ERROR flag.
| * | Reverted [d156af9fb76dd2f4] and removed tests io-52.20 io-75.6 io-75.7, as thispooryorick2023-02-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | commit, intended to fix issue [b8f575aa2398b0e4], breaks the semantics of [read] and [gets]. Such a change would require an accepted TIP. See [b8f575aa2398b0e4] for further discussion. jn: @pouryorick See [b8f575aa2398b0e4] for the reason why this commit is not appropriate: It gets core-8-branch back in the buggy state it was, without even providing a real solution everyone agrees on. You shouldn't revert my patch just because I reverted yours. pooryorick: As I explained, the reason for this reversion is that it hard-codes an unapproved change in the semantics of [read] and [gets] into the test suite. Jan, your statement that it's a "revenge" reversion is false. I spent a month trying to find some alternative to this reversion before actually performing it. A commit that codifes in its tests changes in semantcs to [read]/[gets] simply shouldn't be on core-8-branch.
* | | Reverted [d156af9fb76dd2f4] and removed tests io-52.20 io-75.6 io-75.7, as thispooryorick2023-02-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | commit, intended to fix issue [b8f575aa2398b0e4], breaks the semantics of [read] and [gets]. Such a change would require an accepted TIP. See [b8f575aa2398b0e4] for further discussion. jn: @pouryorick See [b8f575aa2398b0e4] for the reason why this commit is not appropriate: It gets core-8-branch back in the buggy state it was, without even providing a real solution everyone agrees on. You shouldn't revert my patch just because I reverted yours. pooryorick: As I explained, the reason for this reversion is that it hard-codes an unapproved change in the semantics of [read] and [gets] into the test suite. Jan, your statement that it's a "revenge" reversion is false. I spent a month trying to find some alternative to this reversion before actually performing it. A commit that codifes in its tests changes in semantcs to [read]/[gets] simply shouldn't be on core-8-branch.
| | |
| \ \
*-. \ \ Merge 8.7. And fix Tcl_UtfToExternal()/Tcl_ExternalToUtf() (at least - ↵jan.nijtmans2023-02-282-11/+48
|\ \ \ \ | | |/ / | | | | | | | | sufficient for tclZlib.c), such that it now returns an error when src and dst *both* exceed INT_MAX. To be discussed further.
| | * | Merge 8.6jan.nijtmans2023-02-281-9/+38
| | |\ \ | | | |/
| | | * Proposed fix for [f9eafc3886]: Error handling in zlib comment/filename. With ↵jan.nijtmans2023-02-261-9/+38
| | | | | | | | | | | | | | | | testcases
| | | * See [d19fe0a5b] for follow-up to previous commitjan.nijtmans2023-02-231-7/+4
| | | |
| * | | Handle Tcl_UtfToExternal error in tclZlib.cjan.nijtmans2023-02-261-4/+13
| | | |
| * | | Handle statePtr != NULL as welljan.nijtmans2023-02-261-18/+16
| | | |
| * | | Make Tcl_UtfToExternal()/Tcl_ExternalToUtf() report the error, if srcLen and ↵jan.nijtmans2023-02-261-4/+20
| | | | | | | | | | | | | | | | dstLen are both > INT_MAX and therefore not all characters can be handled by this function.
* | | | Merge bug-fb368527ae. Bug [fb368527ae] Large string truncationapnadkarni2023-02-281-21/+69
|\ \ \ \
| * | | | Fix crash. int->size_t needs +1 in comparisons.apnadkarni2023-02-271-1/+1
| | | | |
| * | | | Move the "srcLen -= srcChunkRead;" past the "if ((result != ↵jan.nijtmans2023-02-261-5/+5
| | | | | | | | | | | | | | | | | | | | TCL_CONVERT_NOSPACE)..." (where it originally was), since this isn't needed if the loop ends anyway.
| * | | | Experimental fix for [fb368527ae] - length truncationapnadkarni2023-02-251-19/+67
| | | | |
* | | | | In case of combining TIP #494 (TCL_8_COMPAT) and #628 (building for Tcl 8.7 ↵jan.nijtmans2023-02-283-19/+19
| | | | | | | | | | | | | | | | | | | | with 9.0 headers), ignore TCL_8_COMPAT macro. More Tcl_Size usage.
* | | | | Fix [3d01d51bc4]: Tcl 9: Cannot write large strings to file. Also some ↵jan.nijtmans2023-02-272-53/+69
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | size_t -> Tcl_Size changes.
| * | | | (size_t) -1 -> TCL_INDEX_NONEjan.nijtmans2023-02-261-2/+2
| | | | |
| * | | | Also fix [90ff9b7f73] - writes of exactly 4294967295 bytesapnadkarni2023-02-261-3/+3
| | | | |
| * | | | Minor refactor, add testsapnadkarni2023-02-261-34/+25
| | | | |
| * | | | Fix large writes to file. Need to break into INT_MAX size chunks.apnadkarni2023-02-251-7/+32
| |/ / /
* | | | Fix and tests for [46dda6fc29]apnadkarni2023-02-251-2/+2
|/ / /
* | | Merge 8.7jan.nijtmans2023-02-241-27/+27
|\ \ \ | |/ /
| * | int -> Tcl_Size in tclEncoding.c (making the diff between Tcl 8.7 and 9.0 ↵jan.nijtmans2023-02-241-27/+27
| | | | | | | | | | | | smaller)
| * | minor bug-fix in utf-16/utf-32: 2 testcases failed in Tcl 9 compatibility ↵jan.nijtmans2023-02-221-6/+14
| | | | | | | | | | | | mode (-DTCL_NO_DEPRECATED)
* | | Merge 8.7jan.nijtmans2023-02-221-9/+41
|\ \ \ | |/ /