-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/878/#review2041
-----------------------------------------------------------


what happened to this?

- Ali Saidi


On Sept. 24, 2011, 9:51 a.m., Steve Reinhardt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/878/
> -----------------------------------------------------------
> 
> (Updated Sept. 24, 2011, 9:51 a.m.)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Description
> -------
> 
> swig: move all swigged objects into m5.internal.swig package
> 
> SWIG-generated packages for params, enums, wrapped C++ SimObjects,
> etc. were being generated directly into m5.internal, with some
> funky code to import some of the declarations into m5.internal.params.
> Moved all generated packages under m5.internal.swig and imported
> things directly from there where necessary.  Cleaned up some
> exposed loose ends.  Turns out m5.internal.params doesn't really seem
> to be needed, so it's gone now.
> 
> Also renamed param_Foo packages that wrap both the C++ version of
> Foo and FooParams to just Foo; now that they're under a separate
> subpackage there's no ambiguity with the primary Python objects,
> and no need to have the param_ prefix to support the funky
> importing into internal.params.
> 
> 
> Diffs
> -----
> 
>   src/SConscript d182a475a668 
>   src/python/SConscript d182a475a668 
>   src/python/m5/SimObject.py d182a475a668 
>   src/python/m5/internal/params.py d182a475a668 
>   src/python/m5/internal/swig/__init__.py PRE-CREATION 
>   src/python/m5/objects/__init__.py d182a475a668 
>   src/python/m5/params.py d182a475a668 
>   src/python/swig/inet.i d182a475a668 
>   src/python/swig/time.i d182a475a668 
> 
> Diff: http://reviews.gem5.org/r/878/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Steve Reinhardt
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to