Message208878
Ram Rachum (and to whom it may concern):
This patch was missing the size argument for readlines(), and it did not match the overall style of pathlib.py. (using ''' instead of """, and other docstring style). It also clobbered the builtin 'file'.
I've attached another patch that tries very hard to match that 'style', and wrote tests that try to match what the original 'io' functions test for.
This is my first patch for python-dev, I am attaching it in the hope it will help resolve the issue, or generate more discussion about it.
I ran both the individual test_pathlib tests, and the entire test suite. Also, the patchcheck sanity test. |
|
Date |
User |
Action |
Args |
2014-01-23 04:29:43 | cjwelborn | set | recipients:
+ cjwelborn, pitrou, cool-RR, serhiy.storchaka |
2014-01-23 04:29:43 | cjwelborn | set | messageid: <1390451383.6.0.345361951556.issue20218@psf.upfronthosting.co.za> |
2014-01-23 04:29:43 | cjwelborn | link | issue20218 messages |
2014-01-23 04:29:42 | cjwelborn | create | |
|