diff options
author | Philipp Hahn <hahn@univention.de> | 2018-11-20 15:15:46 +0100 |
---|---|---|
committer | Jano Tomko <jtomko@redhat.com> | 2020-09-01 13:26:01 +0000 |
commit | 9f6638d3b2e5492bd7e0e141d51c9ac0b5c0f0d7 (patch) | |
tree | 46fa229e4c199dddcefbbc6692844d62da3f3eaf /generator.py | |
parent | 3b9ef6ecad50bff2f2ce076bd443195b450b15b7 (diff) | |
download | libvirt-python-9f6638d3b2e5492bd7e0e141d51c9ac0b5c0f0d7.tar.gz |
generator: Simplify sorting
Use sorted(..., key=...) instead of using explicit list.sort()
Signed-off-by: Philipp Hahn <hahn@univention.de>
Diffstat (limited to 'generator.py')
-rwxr-xr-x | generator.py | 21 |
1 files changed, 6 insertions, 15 deletions
diff --git a/generator.py b/generator.py index 1eabb30..f2d5f43 100755 --- a/generator.py +++ b/generator.py @@ -1394,9 +1394,8 @@ def buildWrappers(module): if "None" in function_classes: flist = function_classes["None"] - flist.sort(key=functionSortKey) oldfile = "" - for info in flist: + for info in sorted(flist, key=functionSortKey): (index, func, name, ret, args, file, mod) = info if file != oldfile: classes.write("#\n# Functions from module %s\n#\n\n" % file) @@ -1546,9 +1545,8 @@ def buildWrappers(module): classname) flist = function_classes[classname] - flist.sort(key=functionSortKey) oldfile = "" - for info in flist: + for info in sorted(flist, key=functionSortKey): (index, func, name, ret, args, file, mod) = info # # Do not provide as method the destructors for the class @@ -1731,12 +1729,9 @@ def buildWrappers(module): for type, enum in enumvals: enumData.update(enum) - if enumvals is not None: - enumvals.sort() - for type, enum in enumvals: + for type, enum in sorted(enumvals): classes.write("# %s\n" % type) - items = list(resolveEnum(enum, enumData).items()) - items.sort(key=enumsSortKey) + items = sorted(resolveEnum(enum, enumData).items(), key=enumsSortKey) if items[-1][0].endswith('_LAST'): del items[-1] for name, value in items: @@ -1848,9 +1843,7 @@ def qemuBuildWrappers(module): # for type, enum in sorted(qemu_enums.items()): fd.write("# %s\n" % type) - items = list(enum.items()) - items.sort(key=lambda i: (int(i[1]), i[0])) - for name, value in items: + for name, value in sorted(enum.items(), key=lambda i: (int(i[1]), i[0])): fd.write("%s = %s\n" % (name, value)) fd.write("\n") @@ -1953,9 +1946,7 @@ def lxcBuildWrappers(module): # for type, enum in sorted(lxc_enums.items()): fd.write("# %s\n" % type) - items = list(enum.items()) - items.sort(key=lambda i: (int(i[1]), i[0])) - for name, value in items: + for name, value in sorted(enum.items(), key=lambda i: (int(i[1]), i[0])): fd.write("%s = %s\n" % (name, value)) fd.write("\n") |