LGTM! Nice simplifying change.
http://codereview.chromium.org/4135004/diff/1/7 File src/parser.cc (right): http://codereview.chromium.org/4135004/diff/1/7#newcode4292 src/parser.cc:4292: Handle<String> JsonParser::GetSymbol() { Isn't this name misleading as the returned string is not a symbol? http://codereview.chromium.org/4135004/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
