Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Script("") no longer works

VERIFIED FIXED

Status

()

Core
JavaScript Engine
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: shutdown, Assigned: brendan)

Tracking

({regression, verified1.8.0.8, verified1.8.1})

Trunk
regression, verified1.8.0.8, verified1.8.1
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
$ dbg.obj/js -e "Script('')"
-e:1: TypeError: self-modifying script detected
(Assignee)

Comment 1

11 years ago
Created attachment 241673 [details] [diff] [review]
fix to handle null case

Sorry about that -- this will be fixed when the bug it blocks is fixed on the 1.8 branch.

/be
Assignee: general → brendan
Status: NEW → ASSIGNED
Attachment #241673 - Flags: review+
(Assignee)

Comment 2

11 years ago
Fixed on trunk:

Checking in jsscript.c;
/cvsroot/mozilla/js/src/jsscript.c,v  <--  jsscript.c
new revision: 3.120; previous revision: 3.119
done

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1

Comment 3

11 years ago
Checking in regress-355982.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-355982.js,v  <--  regress-355982.js
initial revision: 1.1
done
Flags: in-testsuite+

Comment 4

11 years ago
verified fixed 20061009 1.8 windows/linux/mac*, 1.9 windows/linux
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1 → verified1.8.1
marking "fixed1.8.0.8" because this was a regression from bug 355655. Checked in with 355655 on the 1.8.0 branch, but should be verified.
Keywords: fixed1.8.0.8

Comment 6

11 years ago
verified fixed in ff1.5.0.8 release in the browser and 1.8.0.8 20061108 windows/mac*/linux.
Keywords: fixed1.8.0.8 → verified1.8.0.8
You need to log in before you can comment on or make changes to this bug.