If you want this change to stick around, the comment should go in the code, otherwise, it just looks redundant.
Nate On Tue, Jun 8, 2010 at 5:16 PM, Lisa Hsu <[email protected]> wrote: > changeset 97c34fea328a in /z/repo/m5 > details: http://repo.m5sim.org/m5?cmd=changeset;node=97c34fea328a > description: > flags: Unserializing old checkpoints before the introduction > of the Initialized flag would break, set Initialized for events upon > unserialization. > > diffstat: > > src/sim/eventq.cc | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diffs (13 lines): > > diff -r 2302e04c506e -r 97c34fea328a src/sim/eventq.cc > --- a/src/sim/eventq.cc Mon Jun 07 12:19:59 2010 -0400 > +++ b/src/sim/eventq.cc Tue Jun 08 17:16:36 2010 -0700 > @@ -238,7 +238,9 @@ > // object itself (since they aren't immediately true) > short _flags; > UNSERIALIZE_SCALAR(_flags); > + assert(initialized()); > flags = _flags; > + flags.set(Initialized); > > bool wasScheduled = flags.isSet(Scheduled) && !flags.isSet(Squashed); > flags.clear(Squashed | Scheduled); > _______________________________________________ > m5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/m5-dev > > _______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
