For sequences, (strings, lists, tuples), use the fact that empty sequences are false
Bug #1156473 reported by
shu, xinxin
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| neutron |
Fix Released
|
Wishlist
|
shu, xinxin | ||
Bug Description
For sequences, (strings, lists, tuples), use the fact that empty sequences are false(PEP 8 Programming Recommendations)
Yes: if not seq:
if seq:
No: if len(seq)
if not len(seq)
| Changed in quantum: | |
| importance: | Undecided → Wishlist |
| tags: | added: low-hanging-fruit |
| Changed in quantum: | |
| assignee: | nobody → shu, xinxin (xinxin-shu) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to quantum (master) | #1 |
| Changed in quantum: | |
| status: | New → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to quantum (master) | #2 |
| Changed in quantum: | |
| status: | In Progress → Fix Committed |
| Changed in quantum: | |
| milestone: | none → havana-1 |
| Changed in quantum: | |
| status: | Fix Committed → Fix Released |
| Changed in neutron: | |
| milestone: | havana-1 → 2013.2 |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/