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)

DEC
Linux
defect

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;
Assignee: mccabe → rogerl
Roger, can you take a peek at this?

Looks like the supplied patch isn't xp; what would be?
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
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.