Closed
Bug 11504
Opened 25 years ago
Closed 25 years ago
Cast needed for legal code for Linux/Alpha.
Categories
(Core :: JavaScript Engine, defect, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: niles, Assigned: rogerl)
Details
To avoid a segmentation fault the follow cast is required for Linux/Alpha
to work. Perhaps, "int" would be a better choice than "long", I wasn't
sure.
Index: jsinterp.c
===================================================================
RCS file: /cvsroot/mozilla/js/src/jsinterp.c,v
retrieving revision 3.16
diff -r3.16 jsinterp.c
711c711
< vp[-fp->script->depth] = (jsval)fp->pc;
---
> vp[-((long)fp->script->depth)] = (jsval)fp->pc;
Updated•25 years ago
|
Assignee: mccabe → rogerl
Comment 1•25 years ago
|
||
Roger, can you take a peek at this? Looks like the supplied patch isn't xp; what would be?
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 2•25 years ago
|
||
Looks like Brendan already got this one in revision 3.18 by casting to intN, with the cryptic comment 'Warning abatement'.
Comment hidden (collapsed) |
You need to log in
before you can comment on or make changes to this bug.
Description
•