LGTM
http://codereview.chromium.org/18761/diff/1/2 File src/serialize.cc (right): http://codereview.chromium.org/18761/diff/1/2#newcode499 Line 499: }; Consider moving this to class scope. I don't think we have any other types with function scope. http://codereview.chromium.org/18761/diff/1/2#newcode501 Line 501: static const RefTableEntry ref_table[] = { Same with static const tables. http://codereview.chromium.org/18761/diff/1/2#newcode575 Line 575: Ditto. http://codereview.chromium.org/18761 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
