DO NOT REPLY [Bug 6916] Avoid bundling a modified zlib
samba-bugs at samba.org
samba-bugs at samba.org
Sun Feb 7 18:32:18 MST 2010
https://bugzilla.samba.org/show_bug.cgi?id=6916
------- Comment #6 from matt at mattmccutchen.net 2010-02-07 19:32 CST -------
Some technical details: Z_INSERT_ONLY is used on the sender side to enter
matched blocks in the gzip history without transmitting them in the compressed
stream. The receiver-side counterpart is the code in see_deflate_token in
token.c which inserts into the input stream a synthesized zlib-format block
containing the matched data.
The receiver-side behavior was possible to implement based on knowledge of the
zlib format without actually modifying zlib. Implementing the sender-side
behavior in the same way would require excising the matched data from zlib's
output and flushing in order to make the excision possible. I guess that was
determined to be too hard, though it's not clear to me what is so hard about
it.
Wayne said previously that the receiver-side implementation is a hack and
should be moved into zlib if Z_INSERT_ONLY is upstreamed:
http://lists.samba.org/archive/rsync/2005-December/014180.html
I agree. But note that after doing that, we would still have a dependency on
the zlib format in send_deflated_token, which fiddles with the "0, 0, ff, ff"
bytes generated by flushing.
--
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the rsync
mailing list