----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3383/#review8121 -----------------------------------------------------------
Ship it! Ship It! - Steve Reinhardt On March 16, 2016, 4:42 a.m., Andreas Sandberg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3383/ > ----------------------------------------------------------- > > (Updated March 16, 2016, 4:42 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11374:e2a7d111ec6e > --------------------------- > style: Refactor the style checker as a Python package > > Refactor the style checker into a Python module that can be reused by > command line tools that integrate with git. In particular: > > * Create a style package in util > * Move style validators from style.py to the style/validators.py. > * Move style verifiers from style.py to the style/verifiers.py. > * Move utility functions (sort_includes, region handling, > file_types) into the style package > * Move generic code from style.py to style/style.py. > > Signed-off-by: Andreas Sandberg <[email protected]> > Reviewed-by: Curtis Dunham <[email protected]> > > > Diffs > ----- > > src/python/m5/util/__init__.py af870035ab6b > src/python/m5/util/region.py af870035ab6b > util/file_types.py af870035ab6b > util/hgstyle.py PRE-CREATION > util/sort_includes.py af870035ab6b > util/style/__init__.py PRE-CREATION > util/style/file_types.py PRE-CREATION > util/style/region.py PRE-CREATION > util/style/sort_includes.py PRE-CREATION > util/style/style.py PRE-CREATION > util/style/validators.py PRE-CREATION > util/style/verifiers.py PRE-CREATION > > Diff: http://reviews.gem5.org/r/3383/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Sandberg > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
