Re: [Mesa-dev] [PATCH 01/41] glapi: clean imports in python files
Hi, minor nitpicks wrt ordering below Am 01.04.2016 um 02:04 schrieb Dylan Baker: > Completely clean the imports: > - Split so that one module is imported per line > - Remove unused imports > - Group stdlib imports, then 3rd party modules, and finally local > modules > - sort alphabetically within those groups > > Signed-off-by: Dylan Baker > --- > src/mapi/glapi/gen/glX_XML.py | 1 - > src/mapi/glapi/gen/glX_proto_common.py | 4 +++- > src/mapi/glapi/gen/glX_proto_recv.py | 4 +++- > src/mapi/glapi/gen/glX_proto_send.py | 9 +++-- > src/mapi/glapi/gen/glX_proto_size.py | 5 +++-- > src/mapi/glapi/gen/glX_server_table.py | 5 - > src/mapi/glapi/gen/gl_SPARC_asm.py | 3 ++- > src/mapi/glapi/gen/gl_XML.py | 8 +--- > src/mapi/glapi/gen/gl_apitemp.py | 3 ++- > src/mapi/glapi/gen/gl_enums.py | 7 +++ > src/mapi/glapi/gen/gl_genexec.py | 6 +++--- > src/mapi/glapi/gen/gl_gentable.py | 3 ++- > src/mapi/glapi/gen/gl_x86-64_asm.py| 3 ++- > src/mapi/glapi/gen/gl_x86_asm.py | 3 ++- > src/mapi/glapi/gen/remap_helper.py | 2 +- > src/mapi/glapi/gen/typeexpr.py | 4 +++- > 16 files changed, 45 insertions(+), 25 deletions(-) > > diff --git a/src/mapi/glapi/gen/glX_XML.py b/src/mapi/glapi/gen/glX_XML.py > index 12ff291..6987e27 100644 > --- a/src/mapi/glapi/gen/glX_XML.py > +++ b/src/mapi/glapi/gen/glX_XML.py > @@ -27,7 +27,6 @@ > > import gl_XML > import license > -import sys, getopt, string > > > class glx_item_factory(gl_XML.gl_item_factory): > diff --git a/src/mapi/glapi/gen/glX_proto_common.py > b/src/mapi/glapi/gen/glX_proto_common.py > index ae2c2d5..bcfe009 100644 > --- a/src/mapi/glapi/gen/glX_proto_common.py > +++ b/src/mapi/glapi/gen/glX_proto_common.py > @@ -25,9 +25,11 @@ > # Authors: > #Ian Romanick > > -import gl_XML, glX_XML > import string > > +import gl_XML > +import glX_XML You majoritely put glX_* first in the rest of the patch. maybe here too? > + > > class glx_proto_item_factory(glX_XML.glx_item_factory): > """Factory to create GLX protocol oriented objects derived from > gl_item.""" > diff --git a/src/mapi/glapi/gen/glX_proto_recv.py > b/src/mapi/glapi/gen/glX_proto_recv.py > index afee388..ec3f7d7 100644 > --- a/src/mapi/glapi/gen/glX_proto_recv.py > +++ b/src/mapi/glapi/gen/glX_proto_recv.py > @@ -28,7 +28,9 @@ > import argparse > import string > > -import gl_XML, glX_XML, glX_proto_common, license > +import glX_proto_common > +import gl_XML glX_XML is missing here, is that intentional? > +import license > > > class PrintGlxDispatch_h(gl_XML.gl_print_base): > diff --git a/src/mapi/glapi/gen/glX_proto_send.py > b/src/mapi/glapi/gen/glX_proto_send.py > index 2b33030..5904b33 100644 > --- a/src/mapi/glapi/gen/glX_proto_send.py > +++ b/src/mapi/glapi/gen/glX_proto_send.py > @@ -28,9 +28,14 @@ > #Jeremy Kolb > > import argparse > +import copy > +import string > + > +import gl_XML > +import glX_XML > +import glX_proto_common > +import license > > -import gl_XML, glX_XML, glX_proto_common, license > -import copy, string > > def convertStringForXCB(str): > tmp = "" > diff --git a/src/mapi/glapi/gen/glX_proto_size.py > b/src/mapi/glapi/gen/glX_proto_size.py > index 75fc26f..b008e9d 100644 > --- a/src/mapi/glapi/gen/glX_proto_size.py > +++ b/src/mapi/glapi/gen/glX_proto_size.py > @@ -26,9 +26,10 @@ > #Ian Romanick > > import argparse > -import sys, string > +import string > > -import gl_XML, glX_XML > +import glX_XML > +import gl_XML > import license > > > diff --git a/src/mapi/glapi/gen/glX_server_table.py > b/src/mapi/glapi/gen/glX_server_table.py > index 2d21f4e..5e996eb 100644 > --- a/src/mapi/glapi/gen/glX_server_table.py > +++ b/src/mapi/glapi/gen/glX_server_table.py > @@ -27,7 +27,10 @@ > > import argparse > > -import gl_XML, glX_XML, glX_proto_common, license > +import glX_XML > +import glX_proto_common > +import gl_XML Ordering glX_proto_common first --Michael > +import license > > > def log2(value): > diff --git a/src/mapi/glapi/gen/gl_SPARC_asm.py > b/src/mapi/glapi/gen/gl_SPARC_asm.py > index fa6217e..89bccef 100644 > --- a/src/mapi/glapi/gen/gl_SPARC_asm.py > +++ b/src/mapi/glapi/gen/gl_SPARC_asm.py > @@ -27,8 +27,9 @@ > > import argparse > > +import glX_XML > +import gl_XML > import license > -import gl_XML, glX_XML > > class PrintGenericStubs(gl_XML.gl_print_base): > def __init__(self): > diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py > index 2e7123e..8c622aa 100644 > --- a/src/mapi/glapi/gen/gl_XML.py > +++ b/src/mapi/glapi/gen/gl_XML.py > @@ -26,11 +26,13 @@ > #Ian Romanick > > from decimal import Decimal > -import xml.etree.ElementTree as ET > -import re, sys, string > import os.path > -import typeexpr > +import re > +import string > +import xml.etree.ElementTree as ET > + > import static_data > +import typeexpr > > > def parse_GL_API( file_name
Re: [Mesa-dev] [PATCH 01/41] glapi: clean imports in python files
Quoting Michael Schellenberger Costa (2016-04-01 01:30:53) > Hi, > > minor nitpicks wrt ordering below > > Am 01.04.2016 um 02:04 schrieb Dylan Baker: > > Completely clean the imports: > > - Split so that one module is imported per line > > - Remove unused imports > > - Group stdlib imports, then 3rd party modules, and finally local > > modules > > - sort alphabetically within those groups > > > > Signed-off-by: Dylan Baker > > --- > > src/mapi/glapi/gen/glX_XML.py | 1 - > > src/mapi/glapi/gen/glX_proto_common.py | 4 +++- > > src/mapi/glapi/gen/glX_proto_recv.py | 4 +++- > > src/mapi/glapi/gen/glX_proto_send.py | 9 +++-- > > src/mapi/glapi/gen/glX_proto_size.py | 5 +++-- > > src/mapi/glapi/gen/glX_server_table.py | 5 - > > src/mapi/glapi/gen/gl_SPARC_asm.py | 3 ++- > > src/mapi/glapi/gen/gl_XML.py | 8 +--- > > src/mapi/glapi/gen/gl_apitemp.py | 3 ++- > > src/mapi/glapi/gen/gl_enums.py | 7 +++ > > src/mapi/glapi/gen/gl_genexec.py | 6 +++--- > > src/mapi/glapi/gen/gl_gentable.py | 3 ++- > > src/mapi/glapi/gen/gl_x86-64_asm.py| 3 ++- > > src/mapi/glapi/gen/gl_x86_asm.py | 3 ++- > > src/mapi/glapi/gen/remap_helper.py | 2 +- > > src/mapi/glapi/gen/typeexpr.py | 4 +++- > > 16 files changed, 45 insertions(+), 25 deletions(-) > > > > diff --git a/src/mapi/glapi/gen/glX_XML.py b/src/mapi/glapi/gen/glX_XML.py > > index 12ff291..6987e27 100644 > > --- a/src/mapi/glapi/gen/glX_XML.py > > +++ b/src/mapi/glapi/gen/glX_XML.py > > @@ -27,7 +27,6 @@ > > > > import gl_XML > > import license > > -import sys, getopt, string > > > > > > class glx_item_factory(gl_XML.gl_item_factory): > > diff --git a/src/mapi/glapi/gen/glX_proto_common.py > > b/src/mapi/glapi/gen/glX_proto_common.py > > index ae2c2d5..bcfe009 100644 > > --- a/src/mapi/glapi/gen/glX_proto_common.py > > +++ b/src/mapi/glapi/gen/glX_proto_common.py > > @@ -25,9 +25,11 @@ > > # Authors: > > #Ian Romanick > > > > -import gl_XML, glX_XML > > import string > > > > +import gl_XML > > +import glX_XML > You majoritely put glX_* first in the rest of the patch. maybe here too? Good catch, apparently I wasn't as diligent at using the sort function as I thought I was. I've fixed this locally. > > + > > > > class glx_proto_item_factory(glX_XML.glx_item_factory): > > """Factory to create GLX protocol oriented objects derived from > > gl_item.""" > > diff --git a/src/mapi/glapi/gen/glX_proto_recv.py > > b/src/mapi/glapi/gen/glX_proto_recv.py > > index afee388..ec3f7d7 100644 > > --- a/src/mapi/glapi/gen/glX_proto_recv.py > > +++ b/src/mapi/glapi/gen/glX_proto_recv.py > > @@ -28,7 +28,9 @@ > > import argparse > > import string > > > > -import gl_XML, glX_XML, glX_proto_common, license > > +import glX_proto_common > > +import gl_XML > glX_XML is missing here, is that intentional? Yes, it's unused in this file. > > +import license > > > > > > class PrintGlxDispatch_h(gl_XML.gl_print_base): > > diff --git a/src/mapi/glapi/gen/glX_proto_send.py > > b/src/mapi/glapi/gen/glX_proto_send.py > > index 2b33030..5904b33 100644 > > --- a/src/mapi/glapi/gen/glX_proto_send.py > > +++ b/src/mapi/glapi/gen/glX_proto_send.py > > @@ -28,9 +28,14 @@ > > #Jeremy Kolb > > > > import argparse > > +import copy > > +import string > > + > > +import gl_XML > > +import glX_XML > > +import glX_proto_common > > +import license > > > > -import gl_XML, glX_XML, glX_proto_common, license > > -import copy, string > > > > def convertStringForXCB(str): > > tmp = "" > > diff --git a/src/mapi/glapi/gen/glX_proto_size.py > > b/src/mapi/glapi/gen/glX_proto_size.py > > index 75fc26f..b008e9d 100644 > > --- a/src/mapi/glapi/gen/glX_proto_size.py > > +++ b/src/mapi/glapi/gen/glX_proto_size.py > > @@ -26,9 +26,10 @@ > > #Ian Romanick > > > > import argparse > > -import sys, string > > +import string > > > > -import gl_XML, glX_XML > > +import glX_XML > > +import gl_XML > > import license > > > > > > diff --git a/src/mapi/glapi/gen/glX_server_table.py > > b/src/mapi/glapi/gen/glX_server_table.py > > index 2d21f4e..5e996eb 100644 > > --- a/src/mapi/glapi/gen/glX_server_table.py > > +++ b/src/mapi/glapi/gen/glX_server_table.py > > @@ -27,7 +27,10 @@ > > > > import argparse > > > > -import gl_XML, glX_XML, glX_proto_common, license > > +import glX_XML > > +import glX_proto_common > > +import gl_XML > Ordering glX_proto_common first > --Michael That's just an artifact of how vim's sort function works, it puts capitals before lower case letters (so X sorts before x), but I can change that. > > +import license > > > > > > def log2(value): > > diff --git a/src/mapi/glapi/gen/gl_SPARC_asm.py > > b/src/mapi/glapi/gen/gl_SPARC_asm.py > > index fa6217e..89bccef 100644 > > --- a/src/mapi/glapi/gen/gl_SPARC_asm.py > > +++ b/src/mapi/glapi/gen/gl_SPARC_asm.py > > @@ -27,8 +27,9 @@ > > > > import argparse > > > > +import
[Mesa-dev] [PATCH 01/41] glapi: clean imports in python files
Completely clean the imports: - Split so that one module is imported per line - Remove unused imports - Group stdlib imports, then 3rd party modules, and finally local modules - sort alphabetically within those groups Signed-off-by: Dylan Baker --- src/mapi/glapi/gen/glX_XML.py | 1 - src/mapi/glapi/gen/glX_proto_common.py | 4 +++- src/mapi/glapi/gen/glX_proto_recv.py | 4 +++- src/mapi/glapi/gen/glX_proto_send.py | 9 +++-- src/mapi/glapi/gen/glX_proto_size.py | 5 +++-- src/mapi/glapi/gen/glX_server_table.py | 5 - src/mapi/glapi/gen/gl_SPARC_asm.py | 3 ++- src/mapi/glapi/gen/gl_XML.py | 8 +--- src/mapi/glapi/gen/gl_apitemp.py | 3 ++- src/mapi/glapi/gen/gl_enums.py | 7 +++ src/mapi/glapi/gen/gl_genexec.py | 6 +++--- src/mapi/glapi/gen/gl_gentable.py | 3 ++- src/mapi/glapi/gen/gl_x86-64_asm.py| 3 ++- src/mapi/glapi/gen/gl_x86_asm.py | 3 ++- src/mapi/glapi/gen/remap_helper.py | 2 +- src/mapi/glapi/gen/typeexpr.py | 4 +++- 16 files changed, 45 insertions(+), 25 deletions(-) diff --git a/src/mapi/glapi/gen/glX_XML.py b/src/mapi/glapi/gen/glX_XML.py index 12ff291..6987e27 100644 --- a/src/mapi/glapi/gen/glX_XML.py +++ b/src/mapi/glapi/gen/glX_XML.py @@ -27,7 +27,6 @@ import gl_XML import license -import sys, getopt, string class glx_item_factory(gl_XML.gl_item_factory): diff --git a/src/mapi/glapi/gen/glX_proto_common.py b/src/mapi/glapi/gen/glX_proto_common.py index ae2c2d5..bcfe009 100644 --- a/src/mapi/glapi/gen/glX_proto_common.py +++ b/src/mapi/glapi/gen/glX_proto_common.py @@ -25,9 +25,11 @@ # Authors: #Ian Romanick -import gl_XML, glX_XML import string +import gl_XML +import glX_XML + class glx_proto_item_factory(glX_XML.glx_item_factory): """Factory to create GLX protocol oriented objects derived from gl_item.""" diff --git a/src/mapi/glapi/gen/glX_proto_recv.py b/src/mapi/glapi/gen/glX_proto_recv.py index afee388..ec3f7d7 100644 --- a/src/mapi/glapi/gen/glX_proto_recv.py +++ b/src/mapi/glapi/gen/glX_proto_recv.py @@ -28,7 +28,9 @@ import argparse import string -import gl_XML, glX_XML, glX_proto_common, license +import glX_proto_common +import gl_XML +import license class PrintGlxDispatch_h(gl_XML.gl_print_base): diff --git a/src/mapi/glapi/gen/glX_proto_send.py b/src/mapi/glapi/gen/glX_proto_send.py index 2b33030..5904b33 100644 --- a/src/mapi/glapi/gen/glX_proto_send.py +++ b/src/mapi/glapi/gen/glX_proto_send.py @@ -28,9 +28,14 @@ #Jeremy Kolb import argparse +import copy +import string + +import gl_XML +import glX_XML +import glX_proto_common +import license -import gl_XML, glX_XML, glX_proto_common, license -import copy, string def convertStringForXCB(str): tmp = "" diff --git a/src/mapi/glapi/gen/glX_proto_size.py b/src/mapi/glapi/gen/glX_proto_size.py index 75fc26f..b008e9d 100644 --- a/src/mapi/glapi/gen/glX_proto_size.py +++ b/src/mapi/glapi/gen/glX_proto_size.py @@ -26,9 +26,10 @@ #Ian Romanick import argparse -import sys, string +import string -import gl_XML, glX_XML +import glX_XML +import gl_XML import license diff --git a/src/mapi/glapi/gen/glX_server_table.py b/src/mapi/glapi/gen/glX_server_table.py index 2d21f4e..5e996eb 100644 --- a/src/mapi/glapi/gen/glX_server_table.py +++ b/src/mapi/glapi/gen/glX_server_table.py @@ -27,7 +27,10 @@ import argparse -import gl_XML, glX_XML, glX_proto_common, license +import glX_XML +import glX_proto_common +import gl_XML +import license def log2(value): diff --git a/src/mapi/glapi/gen/gl_SPARC_asm.py b/src/mapi/glapi/gen/gl_SPARC_asm.py index fa6217e..89bccef 100644 --- a/src/mapi/glapi/gen/gl_SPARC_asm.py +++ b/src/mapi/glapi/gen/gl_SPARC_asm.py @@ -27,8 +27,9 @@ import argparse +import glX_XML +import gl_XML import license -import gl_XML, glX_XML class PrintGenericStubs(gl_XML.gl_print_base): def __init__(self): diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py index 2e7123e..8c622aa 100644 --- a/src/mapi/glapi/gen/gl_XML.py +++ b/src/mapi/glapi/gen/gl_XML.py @@ -26,11 +26,13 @@ #Ian Romanick from decimal import Decimal -import xml.etree.ElementTree as ET -import re, sys, string import os.path -import typeexpr +import re +import string +import xml.etree.ElementTree as ET + import static_data +import typeexpr def parse_GL_API( file_name, factory = None ): diff --git a/src/mapi/glapi/gen/gl_apitemp.py b/src/mapi/glapi/gen/gl_apitemp.py index 5e985a2..50a599a 100644 --- a/src/mapi/glapi/gen/gl_apitemp.py +++ b/src/mapi/glapi/gen/gl_apitemp.py @@ -27,7 +27,8 @@ import argparse -import gl_XML, glX_XML +import glX_XML +import gl_XML import license class PrintGlOffsets(gl_XML.gl_print_base): diff --git a/src/mapi/glapi/gen/gl_enums.py b/src/mapi/glapi/gen/gl_enums.py index 62cc1b3..49cc0bd 100644 --- a/src/mapi/glapi/gen/gl_enums.py +++ b/src/mapi/glapi/gen/gl_enums.py @@ -27,12 +27,11 @@ #Zack Ru