> 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. 

Angus



--
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