Hello,
On Tue, May 3, 2011 at 18:11, Angus Robertson - Magenta Systems Ltd <
an...@magsys.co.uk> wrote:

> > I have overloaded the function with different parameters as there
> > was already some different overloaded versions of it
>
> There was one overloaded version, one line long.
>
> Your new version unnecessarily duplicates large amounts of code.
>
> > but as you guessed forgot to mark the changes. Here it is:
>
> Some changes are marked, not all.
>
> Since this new function seems to be intended to compress part of a file,
> something not mentioned anywhere, it seems strange there is no decompress
> part of a file function?
>
> I feel this should be added to ICS, once it is needed in a high level ICS
> component so it can be tested properly, such as sending a partial file by
> HTTP or FTP.
>
>
>

The HTTP server is buggy in that it compresses the whole file even when a
content-range is present. So there is a need for this function yet it had
not been recognized.

Regards,

SZ
--
To unsubscribe or change your settings for TWSocket mailing list
please goto http://lists.elists.org/cgi-bin/mailman/listinfo/twsocket
Visit our website at http://www.overbyte.be

Reply via email to