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) |
Changed in keystone: | |
assignee: | nobody → wanghong (w-wanghong) |
Changed in keystone: | |
status: | Triaged → In Progress |
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. openstack. org/24955
Review: https:/