This patch is Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
On 03/31/2016 05:04 PM, Dylan Baker wrote: > Rather than using a loop, and augmenting a variable use a generator and > sum to achieve the same result. > > Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> > --- > src/mapi/glapi/gen/gl_x86_asm.py | 10 ++-------- > 1 file changed, 2 insertions(+), 8 deletions(-) > > diff --git a/src/mapi/glapi/gen/gl_x86_asm.py > b/src/mapi/glapi/gen/gl_x86_asm.py > index f2825cf..320d825 100644 > --- a/src/mapi/glapi/gen/gl_x86_asm.py > +++ b/src/mapi/glapi/gen/gl_x86_asm.py > @@ -46,14 +46,8 @@ class PrintGenericStubs(gl_XML.gl_print_base): > "BRIAN PAUL, IBM") > > def get_stack_size(self, f): > - size = 0 > - for p in f.parameterIterator(): > - if p.is_padding: > - continue > - > - size += p.get_stack_size() > - > - return size > + return sum(p.get_stack_size() for p in f.parameterIterator() > + if not p.is_padding) > > def printRealHeader(self): > print '#include "x86/assyntax.h"' > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
