Code review comment for lp:~savoirfairelinux-openerp/openerp-hr/department_sequence_concatination_NoneType
Revision history for this message
| Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote | # |
review:
Approve
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
67d34a1
(Get the code!)
> How about this?
> args = args or []
With this one, you will build a new list when args is an empty list, which is unnecessary.