diff options
author | Dan McGee <dan@archlinux.org> | 2012-11-16 16:20:11 -0600 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2012-11-16 16:37:52 -0600 |
commit | 6dd4d54bb0adbbb0f8c2b1beaa92b7a58971cf88 (patch) | |
tree | ce20ff9bf1118c8d383eed352bcdcbf064cb5269 /packages/views/signoff.py | |
parent | b3979dd2315ce1aef779abdb646562d7bba7a44f (diff) |
Use Python 2.7 dictionary comprehension syntax
Rather than the old idiom of dict((k, v) for <> in <>).
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'packages/views/signoff.py')
-rw-r--r-- | packages/views/signoff.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/packages/views/signoff.py b/packages/views/signoff.py index 56eb060c..824a9922 100644 --- a/packages/views/signoff.py +++ b/packages/views/signoff.py @@ -155,8 +155,8 @@ class SignoffJSONEncoder(DjangoJSONEncoder): def default(self, obj): if isinstance(obj, PackageSignoffGroup): - data = dict((attr, getattr(obj, attr)) - for attr in self.signoff_group_attrs) + data = {attr: getattr(obj, attr) + for attr in self.signoff_group_attrs} data['pkgnames'] = [p.pkgname for p in obj.packages] data['package_count'] = len(obj.packages) data['approved'] = obj.approved() @@ -164,9 +164,7 @@ class SignoffJSONEncoder(DjangoJSONEncoder): for attr in self.signoff_spec_attrs) return data elif isinstance(obj, Signoff): - data = dict((attr, getattr(obj, attr)) - for attr in self.signoff_attrs) - return data + return {attr: getattr(obj, attr) for attr in self.signoff_attrs} elif isinstance(obj, Arch) or isinstance(obj, Repo): return unicode(obj) elif isinstance(obj, User): |