----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/428/#review755 -----------------------------------------------------------
Ship it! Thanks for all the comments.... an @file would make the file doxygen complete. src/unittest/unittest.hh <http://reviews.m5sim.org/r/428/#comment1017> One more comment. Could we have something like: /** * @file This file defines functions and macros that aid in writing unit tests. .... */ - Ali On 2011-01-17 00:03:59, Gabe Black wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/428/ > ----------------------------------------------------------- > > (Updated 2011-01-17 00:03:59) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > Unit tests: Define a header file for common unit testing functions/macros. > > > Diffs > ----- > > src/unittest/SConscript a82dcad2bc18 > src/unittest/unittest.hh PRE-CREATION > src/unittest/unittest.cc PRE-CREATION > > Diff: http://reviews.m5sim.org/r/428/diff > > > Testing > ------- > > > Thanks, > > Gabe > >
_______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
