summaryrefslogtreecommitdiff
path: root/doc/reference
diff options
context:
space:
mode:
authorOlivier Dony <odo@openerp.com>2017-08-19 01:38:22 +0200
committerOlivier Dony <odo@openerp.com>2017-08-20 23:25:54 +0200
commitf5b7de0e6c48ab00bd4efbb38201b53ad94ee5e2 (patch)
tree5ac5f4b4a5f2909e84fc78a2fa0ac772a994aed7 /doc/reference
parent695716efb0f79f9a0e716952b1dcd433d866371a (diff)
[FIX] P3: correct more dict iterable builtins use
Some bits posterior or leftover from fffaf735f58f1ac57781c095eb552de6d1fcdc47 See also parent commit.
Diffstat (limited to 'doc/reference')
-rw-r--r--doc/reference/guidelines.rst5
1 files changed, 1 insertions, 4 deletions
diff --git a/doc/reference/guidelines.rst b/doc/reference/guidelines.rst
index 058fe7ffb1c..fc15c10f7ab 100644
--- a/doc/reference/guidelines.rst
+++ b/doc/reference/guidelines.rst
@@ -452,12 +452,9 @@ So, you can write ``if some_collection:`` instead of ``if len(some_collection):`
# better
for key in my_dict:
"do something..."
- # creates a temporary list
+ # accessing the key,value pair
for key, value in my_dict.items():
"do something..."
- # only iterates
- for key, value in my_dict.iteritems():
- "do something..."
- Use dict.setdefault