Message139983
| Author |
eric.araujo |
| Recipients |
alexis, eric.araujo, tarek, thomas.holmes |
| Date |
2011年07月07日.17:39:29 |
| SpamBayes Score |
1.9344527e-05 |
| Marked as misclassified |
No |
| Message-id |
<1310060370.6.0.854735568348.issue12504@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Thanks for that! I agree that it’s a fake optimization to work with generators instead of lists when the list is small or when we depend on other resources like file handles. There are a few methods we could change in the database module.
The patch looks good, except that I wouldn’t move code to a nested function, but just change the "yield spam" line to "results.append(spam)".
In our message, is the test log showing failure something you got before making changes or after? IOW, does the patch fixes the bug? |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2011年07月07日 17:39:30 | eric.araujo | set | recipients:
+ eric.araujo, tarek, thomas.holmes, alexis |
| 2011年07月07日 17:39:30 | eric.araujo | set | messageid: <1310060370.6.0.854735568348.issue12504@psf.upfronthosting.co.za> |
| 2011年07月07日 17:39:30 | eric.araujo | link | issue12504 messages |
| 2011年07月07日 17:39:29 | eric.araujo | create |
|