Use the max function and a generator expression instead of a loop and if. This is certainly no less efficient, and saves several lines of code.
Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> --- src/mapi/glapi/gen/gl_XML.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py index b7178d6..f96e44a 100644 --- a/src/mapi/glapi/gen/gl_XML.py +++ b/src/mapi/glapi/gen/gl_XML.py @@ -938,10 +938,7 @@ class gl_api(object): return iter(functions) def functionIterateByOffset(self): - max_offset = -1 - for func in self.functions_by_name.itervalues(): - if func.offset > max_offset: - max_offset = func.offset + max_offset = max(f.offset for f in self.functions_by_name.itervalues()) temp = [None for i in range(max_offset + 1)] for func in self.functions_by_name.itervalues(): -- 2.8.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev