http://codereview.chromium.org/108002/diff/1/3 File src/mark-compact.cc (right):
http://codereview.chromium.org/108002/diff/1/3#newcode572 Line 572: HeapObject* object = HeapObject::cast(*p); On 2009/05/05 05:22:34, Kasper Lund wrote: > The newlines here look weird. How about moving the HeapObject* object definition > below the comment? I could do that, but the comment goes with the continue. http://codereview.chromium.org/108002/diff/1/3#newcode580 Line 580: object->IterateBody(map->instance_type(), On 2009/05/05 05:22:34, Kasper Lund wrote: > I'd compute the size before the IterateBody call, stuff it in a local variable, > and use it to get nicely formatted code where the IterateBody call fits on one > line. But that's just me. That's not the way I roll. http://codereview.chromium.org/108002/diff/1/2 File src/mark-compact.h (right): http://codereview.chromium.org/108002/diff/1/2#newcode159 Line 159: UpdateLiveObjectCount(obj, 1); On 2009/05/05 05:22:34, Kasper Lund wrote: > Do you need the scale argument to UpdateLiveObjectCount anymore? No, you're right. I'll revert it when we clean this all up. http://codereview.chromium.org/108002 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
