Reviewers: Sven,
Message:
Story: I was cleaning up and noticed that not all the functions defined in
ast-inl.h had the 'inline' specifier. When I added it, I found out that
ast-inl.h was not actually included in dozens of places that actually
needed it.
How did it work at all (why did the linker not see duplicate definitions for
these functions)? Because ast-inl.h was included in exactly one place in
V8,
parser.cc.
Anyway, the circularity that caused us to try to introduce ast-inl.h in the
first place has since been resolved, so I've just nuked the file.
Description:
Remove ast-inl.h. This file is not necessary.
[email protected]
BUG=
TEST=
Please review this at http://codereview.chromium.org/8509005/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
D src/ast-inl.h
M src/ast.h
M src/ast.cc
M src/parser.cc
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
