summaryrefslogtreecommitdiff
path: root/handbook/Makefile
diff options
context:
space:
mode:
authorChris Larson <clarson@mvista.com>2009-06-11 13:10:04 -0700
committerRichard Purdie <rpurdie@linux.intel.com>2010-03-22 15:01:59 +0000
commitada2a8494a88b59de25c0a44fce30190f560eff4 (patch)
tree67f60f7ae769b74815757e45c12e4d694270a802 /handbook/Makefile
parent9d9b47bae4b880ec57eda0e647b1d24fbc3ba3cf (diff)
downloadopenembedded-core-ada2a8494a88b59de25c0a44fce30190f560eff4.tar.gz
openembedded-core-ada2a8494a88b59de25c0a44fce30190f560eff4.tar.bz2
openembedded-core-ada2a8494a88b59de25c0a44fce30190f560eff4.zip
Avoid unnecessary calls to keys() when iterating over dictionaries.
dict objects provide an __iter__ method for the iteration which gives you the keys, so calling keys directly is unnecessary, and isn't really a best practice. The only time you really need to call the keys is if there's a danger of the dict changing out from underneith you, either due to external forces or due to modification of the iterable in the loop. Iterations over os.environ are apparently subject to such changes, so they must continue to use keys(). As an aside, also switches a couple spots to using sorted() rather than creating a temporary list with keys() and sorting that. (Bitbake rev: 5b6ccb16c6e71e23dac6920cd2df994d67c2587b) Signed-off-by: Chris Larson <clarson@mvista.com> Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
Diffstat (limited to 'handbook/Makefile')
0 files changed, 0 insertions, 0 deletions