On 2013/08/05 16:26:41, Sven Panne wrote:
NOT LGTM. Wouldn't it be much simpler to keep the one-and-only definition
in
include/v8.h and remove the #undef plus the copies of the CPP madness
instead?
I don't think that #undef'ing something we #define'd before will help
anybody.
A
much more common, simple rule is to reserve a prefix for #defines in the
external headers (e.g. GL_ for OpenGL etc.).
Therefore I propose to drop this CL and make a differente CL that makes
all
our
macros have a V8_ prefix + drop copies of redundant macros. No #undefs
involved.
Done. Please review at https://codereview.chromium.org/22363003/
https://codereview.chromium.org/22093003/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
