This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author nirai
Recipients lucifer, nirai, pitrou
Date 2010-01-14.19:44:13
SpamBayes Score 4.244637e-10
Marked as misclassified No
Message-id <1263498260.12.0.192359693373.issue7610@psf.upfronthosting.co.za>
In-reply-to
Content
I uploaded an update for Python 2.7.

> * you should probably write `n = sys.maxsize` instead of `n = 1 << 31 - 1`

sys.maxsize is 64 bit number on my system but the maximum value accepted by zlib's decompress() seems to be INT_MAX defined in pyport.h which equals the number I used.

> * ZipExtFile.read() should support `n=None` as a synonym to `n=-1` 
> (read everything)

Added

> * `bytes` as a variable name isn't very good since it's the built-in 
> name for bytestrings in py3k

Changed (old habits die hard).

> * in ZipExtFile.read(), it seems you have removed the adjustment for 
> encrypted files (see `adjust read size for encrypted files since the 
> first 12 bytes [etc.]`)

Yes, was moved to the constructor.

> * is there a situation where the decompressor might return less bytes 
> than expected? (after all compression doesn't /always/ compress, in 
> unfavourable chunks of data it might actually expand things a bit)

The documentation of io.BufferedIOBase.read() reads "multiple raw reads may be issued to satisfy the byte count". I understood this language to mean satisfying read size is optional. Isn't it?
History
Date User Action Args
2010-01-14 19:44:20niraisetrecipients: + nirai, pitrou, lucifer
2010-01-14 19:44:20niraisetmessageid: <1263498260.12.0.192359693373.issue7610@psf.upfronthosting.co.za>
2010-01-14 19:44:18nirailinkissue7610 messages
2010-01-14 19:44:18niraicreate