rep <m...@mwcollect.org> added the comment:

Xuanji: Patch looks good now with the test cases.

Regarding the other fix suggested by Kevin I think that we should not introduce 
custom behaviour here like appending the eofdata to the comment even though the 
comment size in the record does not say so. In the end we want to make sure 
that correct zip files end up being processed correctly with the specified 
comment size and incorrect zip files with eofdata should not be rejected / 
throw exceptions.

So I agree with Xuanji that his patch (or mine) is the way to go.

You probably agree that we can mark issue 10298 to be superseded by this one 
here.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue10694>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to