diff options
| author | Olivier Dony <odo@openerp.com> | 2017-08-19 01:38:22 +0200 |
|---|---|---|
| committer | Olivier Dony <odo@openerp.com> | 2017-08-20 23:25:54 +0200 |
| commit | f5b7de0e6c48ab00bd4efbb38201b53ad94ee5e2 (patch) | |
| tree | 5ac5f4b4a5f2909e84fc78a2fa0ac772a994aed7 /doc/reference/guidelines.rst | |
| parent | 695716efb0f79f9a0e716952b1dcd433d866371a (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/guidelines.rst')
| -rw-r--r-- | doc/reference/guidelines.rst | 5 |
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 |