Message237328
Updated version of the patch after Victor's code review of scandir-7 and a couple of his comments offline. Full commit log is available on my GitHub project in posixmodule_scandir_main.c if anyone's interested. Again, I haven't looked closely at the docs or tests yet, but hopefully the C code is getting pretty close now!
KNOWN ISSUE: There's a reference leak in the POSIX version (found with "python -m test -R 3:2 test_os"). I'm a Windows guy and this is my first serious Python C extension code, so I'm not sure the best way to track this down. Would someone like to either point out the bug :-) or help me with how to track this down? |
|
Date |
User |
Action |
Args |
2015-03-06 04:53:49 | benhoyt | set | recipients:
+ benhoyt, tebeka, pitrou, vstinner, giampaolo.rodola, tim.golden, abacabadabacaba, akira, socketpair, josh.r, alexei.romanov |
2015-03-06 04:53:49 | benhoyt | set | messageid: <1425617629.66.0.0616508081822.issue22524@psf.upfronthosting.co.za> |
2015-03-06 04:53:49 | benhoyt | link | issue22524 messages |
2015-03-06 04:53:49 | benhoyt | create | |
|