Reviewers: Erik Corry,
Message:
There was a comment that the toplevel functions were to avoid putting the
definition of class Parser in parser.h.
It seems to me that the V8 codebase is big enough that free functions are
to be
avoided and a class in a header file is not a big deal.
I've swizzled the includes so parser.h is no longer included by compiler.h
and
now is only included in .cc files where it's needed.
Description:
Make some free functions into static members of class Parser.
Please review this at http://codereview.chromium.org/3518005/show
Affected files:
M src/api.cc
M src/compiler.h
M src/compiler.cc
M src/jsregexp.cc
M src/liveedit.cc
M src/parser.h
M src/parser.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev