Many hyperlinks are disabled.
Use anonymous login
to enable hyperlinks.
Overview
| Comment: | The issue with the skip code was not the limit to 64K, but how the code (a) assumed the wrong type for the result of 'read' (number of bytes, but is the read data), and (b) mis-handled short reads. Using 'string length' to convert the result type into what was needed, and using the proper condition was what fixed the issue, and not getting rid of the limit. I have now re-instated the limit, with in-code explanation of why it makes sense, and verified that this still passes the testsuite. I further changed the variable names to be much more self-documenting as well. |
|---|---|
| Downloads: | See separate download page |
| Timelines: | family | ancestors | descendants | both | pyk-tar |
| Files: | files | file ages | folders |
| SHA1: |
8de453c348b85718a0884e8cdd9b78da |
| User & Date: | andreask 2013-11-23 00:00:39.292 |
Context
|
2013-11-23
| ||
| 00:14 | Bumped package version, updated logs, ready to merge. Fixed ticket [6b7aa0aecc]. check-in: 4a2f75b5da user: andreask tags: pyk-tar | |
| 00:00 | The issue with the skip code was not the limit to 64K, but how the code (a) assumed the wrong type for the result of 'read' (number of bytes, but is the read data), and (b) mis-handled short reads. Using 'string length' to convert the result type into what was needed, and using the proper condition was what fixed the issue, and not getting rid of the limit. I have now re-instated the limit, with in-code explanation of why it makes sense, and verified that this still passes the testsuite. I further changed the variable names to be much more self-documenting as well. check-in: 8de453c348 user: andreask tags: pyk-tar | |
|
2013-11-22
| ||
| 23:39 | Fixed the testsuite up somewhat. (1) Moved helper code into separate file. (2) Replaced helper code blocks in variables with proper procedures. (3) Simplified setup1, cleanup1 No need to search for and create lots of temp dirs in setup, just remove the directory in cleanup. Now use single temp dir, fixed name. (4) Fixed setup of tests in general (4a) Added emacs mode marker to get proper Tcl mode. (4b) Added missing Tcl 8.5 requirement (4c) Use canonical 'use' command for access to local package (memchan). (5) Constrained the one test requiring Tcl 8.6 (zlib) It is not yet clear to me why the 'stream' helper (to create a large file) is so complex. check-in: 624a1ee7b8 user: andreask tags: pyk-tar | |
Changes
Changes to modules/tar/tar.tcl.