Type | Wish | Status | reviewed | Date | 1-Oct-2010 06:37 |
---|---|---|---|---|---|
Version | alpha 108 | Category | Native | Submitted by | Carl |
Platform | All | Severity | minor | Priority | normal |
Summary | /zlib option for COMPRESS and DECOMPRESS |
---|---|
Description |
I did not want to forget about this one: Add an option for zlib compress format. This is easy to hack yourself (see REBOL.org) but it would be good to make it a standard refinement option. See also #0000361 for related option, LZW. |
Example code |
Assigned to | n/a | Fixed in | - | Last Update | 20-Oct-2010 04:56 |
---|
Comments | |
---|---|
(0002564)
BrianH 1-Oct-2010 08:23 |
See also #328 for this same issue, which has been waiting for some test data to be added to the ticket before we can mark it as tested. Please provide test data (in an attached file perhaps) for this ticket, #328 and #361. |
(0002604)
oldes 19-Oct-2010 16:41 |
The important missing part for zlib support is CRC-32 checksum method. See #1678 |
(0002613)
abolka 20-Oct-2010 03:23 |
This is not fixed in A108 as zlib format (RFC1950) != gzip format (RFC1952). A107 added support for gzip, but not for zlib; see also http://www.zlib.net/zlib_faq.html#faq18 |
(0002627)
BrianH 20-Oct-2010 04:56 |
The magic numbers of zlib and gzip are different, so you don't need a /gzip option for DECOMPRESS, it can just autodetect the difference. We do need a /zlib option for DECOMPRESS because it can't tell the difference between REBOL compressed data and zlib data, so it tries to allocate based on the last four bytes of the zlib data (often a large amount).
For DECOMPRESS, the /zlib option would tell it to not preallocate based on the last 4 bytes of the data (it should use some other method to determine how much to allocate). For COMPRESS, the /zlib option would tell it to not put those 4 bytes on the end of the compressed data. COMPRESS and DECOMPRESS already have CRC-32 checksum support, so #1678 doesn't help with a built-in solution unless it's mezzanine. |
Date | User | Field | Action | Change |
---|---|---|---|---|
20-Oct-2010 05:19 | abolka | Comment : 0002613 | Modified | - |
20-Oct-2010 04:59 | BrianH | Comment : 0002627 | Modified | - |
20-Oct-2010 04:56 | BrianH | Comment : 0002627 | Added | - |
20-Oct-2010 04:49 | BrianH | Status | Modified | built => reviewed |
20-Oct-2010 04:49 | BrianH | Fixedin | Modified | alpha 108 => none |
20-Oct-2010 04:49 | BrianH | Summary | Modified | Zlib option for COMPRESS/DECOMPRESS => /zlib option for COMPRESS and DECOMPRESS |
20-Oct-2010 04:25 | abolka | Comment : 0002613 | Modified | - |
20-Oct-2010 03:39 | abolka | Comment : 0002613 | Modified | - |
20-Oct-2010 03:23 | abolka | Comment : 0002613 | Added | - |
19-Oct-2010 20:38 | carl | Status | Modified | reviewed => built |
19-Oct-2010 20:38 | carl | Fixedin | Modified | => alpha 108 |
19-Oct-2010 16:41 | oldes | Comment : 0002604 | Added | - |
1-Oct-2010 08:24 | BrianH | Status | Modified | submitted => reviewed |
1-Oct-2010 08:23 | BrianH | Comment : 0002564 | Added | - |
1-Oct-2010 08:21 | BrianH | Category | Modified | Unspecified => Native |
1-Oct-2010 06:39 | Carl | Description | Modified | - |
1-Oct-2010 06:38 | Carl | Description | Modified | - |
1-Oct-2010 06:38 | Carl | Description | Modified | - |
1-Oct-2010 06:37 | Carl | Ticket | Added | - |