Message265541
There are yet few issues.
1. Currently RuntimeError is widely used in zipfile. But in most cases ValueError would be more appropriate since it programming error (for example ValueError is raised when try to read or write to closed file). See issue24693, but this is backward incompatible change. For new exceptions we are free to use any exception type without breaking backward compatibility. It looks to me, that ValueError is more appropriate is these cases than RuntimeError. What do you think about this Thomas? Here is a patch that changes exceptions types.
2. ZipInfo.is_dir() is not documented and lacks a docstring.
3. It would be better to make force_zip64 a keyword-only parameter. This would allow to add new positional parameters without breaking compatibility. |
|
Date |
User |
Action |
Args |
2016-05-14 20:59:48 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, python-dev, takluyver, martin.panter, mbussonn, Dhiraj_Mishra |
2016-05-14 20:59:48 | serhiy.storchaka | set | messageid: <1463259588.47.0.834036486375.issue26039@psf.upfronthosting.co.za> |
2016-05-14 20:59:48 | serhiy.storchaka | link | issue26039 messages |
2016-05-14 20:59:48 | serhiy.storchaka | create | |
|