Message259206
Here's a new version of the zf.open() patch following Martin's review (thanks Martin!).
I agree that it feels a bit awkward having two completely different actions for zf.open(), but it is a familiar interface, and since the mode parameter is already there, it requires a minimum of new public API. But I'm happy to add a new method like open_write() or write_handle() if people prefer that.
The comments on the other patch are minimal, I'll put a new version of that together as well. |
|
Date |
User |
Action |
Args |
2016-01-29 12:08:52 | takluyver | set | recipients:
+ takluyver, martin.panter, serhiy.storchaka, mbussonn |
2016-01-29 12:08:51 | takluyver | set | messageid: <1454069331.14.0.690103010061.issue26039@psf.upfronthosting.co.za> |
2016-01-29 12:08:51 | takluyver | link | issue26039 messages |
2016-01-29 12:08:50 | takluyver | create | |
|