fixes use the fact that empty sequences are false
Bug #1156472 reported by
shu, xinxin
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Identity (keystone) |
Fix Released
|
Wishlist
|
wanghong | ||
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 keystone: | |
| importance: | Undecided → Wishlist |
| status: | New → Triaged |
| Changed in keystone: | |
| assignee: | nobody → shu, xinxin (xinxin-shu) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to keystone (master) | #1 |
| Changed in keystone: | |
| status: | Triaged → In Progress |
Revision history for this message
| Dolph Mathews (dolph) wrote | #2 |
| Changed in keystone: | |
| assignee: | shu, xinxin (xinxin-shu) → nobody |
| status: | In Progress → Triaged |
| Changed in keystone: | |
| assignee: | nobody → wanghong (w-wanghong) |
| Changed in keystone: | |
| status: | Triaged → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to keystone (master) | #3 |
| Changed in keystone: | |
| status: | In Progress → Fix Committed |
| Changed in keystone: | |
| milestone: | none → icehouse-2 |
| status: | Fix Committed → Fix Released |
| Changed in keystone: | |
| milestone: | icehouse-2 → 2014.1 |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/