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'.
You need to log in
before you can comment on or make changes to this bug.
Description
•