Code review comment for lp:~camptocamp/magentoerpconnect/oerp61-oldstable-import-partners-pagination-2013-02-12
Revision history for this message
| Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote | # |
review:
Needs Fixing
(code review, no test)
l. 28: use xrange instead of range, and you can probably avoid the creation of this list by iterating directly on the xrange object in the for loop line 31.