Closed
Bug 509136
Opened 16 years ago
Closed 13 years ago
Bogus JS_ASSERT when compiling with VC7.1 and --enable-debug
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: neil, Unassigned)
Details
ExecuteTree contains
>debug_only_stmt(*(uint64*)&global[globalFrameSize] = 0xdeadbeefdeadbeefLL;)
but LeaveTree contains
>JS_ASSERT(*(uint64*)&global[STOBJ_NSLOTS(JS_GetGlobalForObject(cx, cx->fp->scopeChain))] == 0xdeadbeefdeadbeefLL);
debug_only_stmt is always ignored because VC7.1 doesn't support varadic macros and so debug_only_stmt is ignored but JS_ASSERT only depends on --enable-debug.
|
Reporter | |
Comment 1•16 years ago
|
||
Two possible suggestions:
1. Wrap the JS_ASSERT in an #ifndef MOZ_NO_VARADIC_MACROS
2. Wrap the JS_ASSERT in a debug_only_stmt()
|
||
Comment 2•13 years ago
|
||
The tracer was removed.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•