----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/876/#review1567 -----------------------------------------------------------
I don't really know how this code works so I'm not qualified to say much about it, but what I saw all seems fine. I like the idea of this getting easier to understand. - Gabe On 2011-09-24 09:49:10, Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/876/ > ----------------------------------------------------------- > > (Updated 2011-09-24 09:49:10) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > scons/swig: refactor some of the scons/SWIG code > > - Move the random bits of SWIG code generation out of src/SConscript > file and into methods on the objects being wrapped. > - Cleaned up some variable naming and added some comments to make > the process a little clearer. > - Did a little generated file/module renaming: > - vptype_Foo now Foo_vector > - init_Foo is now Foo_init > This makes it easier to see all the Foo-related files in a > sorted directory listing. > - Made cxx_predecls and swig_predecls normal SimObject classmethods. > - Got rid of swig_objdecls hook, even though this breaks the System > objects get/setMemoryMode method exports. Will be fixing this in > a future changeset. > > > Diffs > ----- > > src/SConscript d182a475a668 > src/python/m5/SimObject.py d182a475a668 > src/python/m5/params.py d182a475a668 > src/sim/System.py d182a475a668 > > Diff: http://reviews.m5sim.org/r/876/diff > > > Testing > ------- > > > Thanks, > > Steve > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
