Issue 128: jsregexp.cc #include ordering confuses Define's
http://code.google.com/p/v8/issues/detail?id=128
New issue report by jcpstuff:
Detailed description of the issue.
By including:
#include "third_party/jscre/pcre.h"
Before:
#include "platform.h"
#include "runtime.h"
#include "top.h"
It undef's DEBUG. From pcre.h:
// JSCRE is very chatty in debug mode, so in order to keep it slient
// while still importing v8.h correctly (it contains #ifdef DEBUGs)
// we allow DEBUG to be set and undef it manually.
#undef DEBUG
Some classes (e.g. AssertNoContextChange) in top.h are dependent on the
DEBUG define for their memory layout, and you wind up having
two 'versions' of AssertNoContextChange that can be linked in some cases.
The fix is to include that file last.
Issue attributes:
Status: New
Owner: ----
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
