Message137174
I added comments in the code review.
this patch is looking good once the comments are addressed. thanks for your contribution!
As for talk of support for recursion... thats what os.walk() is for. it doesn't belong as part of any particular individual function itself. |
|
Date |
User |
Action |
Args |
2011-05-29 09:13:31 | gregory.p.smith | set | recipients:
+ gregory.p.smith, ncoghlan, eric.smith, giampaolo.rodola, eric.araujo, sandro.tosi, neologix, petri.lehtinen |
2011-05-29 09:13:31 | gregory.p.smith | set | messageid: <1306660411.24.0.0581181139102.issue12191@psf.upfronthosting.co.za> |
2011-05-29 09:13:30 | gregory.p.smith | link | issue12191 messages |
2011-05-29 09:13:30 | gregory.p.smith | create | |
|