LGTM! This is a nice change.
http://codereview.chromium.org/112036/diff/1003/2006 File test/cctest/test-log.cc (right): http://codereview.chromium.org/112036/diff/1003/2006#newcode159 Line 159: struct ParseLogResult { Why not a class? http://codereview.chromium.org/112036 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
