Module: Mesa Branch: master Commit: 08fe9b3e3a5bd18db39daa56ecae3fc1a0708ff5 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=08fe9b3e3a5bd18db39daa56ecae3fc1a0708ff5
Author: Mathieu Bridon <boche...@daitauha.fr> Date: Thu Aug 9 10:27:24 2018 +0200 python: Simplify list sorting Instead of copying the list, then sorting the copy in-place, we can just get a new sorted copy directly. Signed-off-by: Mathieu Bridon <boche...@daitauha.fr> Reviewed-by: Eric Engestrom <eric.engest...@intel.com> Reviewed-by: Dylan Baker <dy...@pnwbakers.com> --- src/mapi/mapi_abi.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mapi/mapi_abi.py b/src/mapi/mapi_abi.py index d4c48ec430..dc48fa5935 100644 --- a/src/mapi/mapi_abi.py +++ b/src/mapi/mapi_abi.py @@ -291,8 +291,7 @@ class ABIPrinter(object): self.entries = entries # sort entries by their names - self.entries_sorted_by_names = self.entries[:] - self.entries_sorted_by_names.sort(key=attrgetter('name')) + self.entries_sorted_by_names = sorted(self.entries, key=attrgetter('name')) self.indent = ' ' * 3 self.noop_warn = 'noop_warn' @@ -441,8 +440,7 @@ class ABIPrinter(object): def c_stub_string_pool(self): """Return the string pool for use by stubs.""" # sort entries by their names - sorted_entries = self.entries[:] - sorted_entries.sort(key=attrgetter('name')) + sorted_entries = sorted(self.entries, key=attrgetter('name')) pool = [] offsets = {} _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-commit