On 16 Oct 2015 16:12, Brian Norris wrote: > Just use dict.items(), since there are few fields and we don't really > care about the extra copying.
Reviewed-by: Mike Frysinger <[email protected]> > --- a/patchwork/bin/pwclient > +++ b/patchwork/bin/pwclient > @@ -237,7 +237,7 @@ def action_info(rpc, patch_id): > s = "Information for patch id %d" % (patch_id) > print(s) > print('-' * len(s)) > - for key, value in sorted(patch.iteritems()): > + for key, value in sorted(patch.items()): the fact we run it through sorted means we end up with the full list eventually anyways, so i doubt we'd notice the overhead w/py2. -mike
signature.asc
Description: Digital signature
_______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
