Labeled "if(1);" disappears but label remains

VERIFIED FIXED in mozilla1.8.1

Status

()

P2
normal
VERIFIED FIXED
12 years ago
12 years ago

People

(Reporter: jruderman, Assigned: brendan)

Tracking

(Blocks: 1 bug, {testcase, verified1.8.1})

Trunk
mozilla1.8.1
testcase, verified1.8.1
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
> function () { L: if(1); }
function () { L: }

> function () { L: }
SyntaxError on line 1: syntax error

Expected: "if(1);" should probably decompile to something like "{ }" in this context, the same way "if(0);" does.
(Assignee)

Comment 1

12 years ago
Created attachment 235528 [details] [diff] [review]
fix
Assignee: general → brendan
Status: NEW → ASSIGNED
Attachment #235528 - Flags: superreview?(shaver)
Attachment #235528 - Flags: review?(mrbkap)
(Assignee)

Updated

12 years ago
OS: Mac OS X 10.4 → All
Priority: -- → P2
Hardware: Macintosh → All
Target Milestone: --- → mozilla1.8.1
Attachment #235528 - Flags: review?(mrbkap) → review+
Comment on attachment 235528 [details] [diff] [review]
fix

sr=shaver
Attachment #235528 - Flags: superreview?(shaver) → superreview+
(Assignee)

Comment 3

12 years ago
Comment on attachment 235528 [details] [diff] [review]
fix

Super-safe fix to help reduce decompiler fuzz testing exclusions, and maybe even help a real-world decompiler user or two.

/be
Attachment #235528 - Flags: approval1.8.1?
(Assignee)

Comment 4

12 years ago
Fixed on trunk.

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 5

12 years ago
Comment on attachment 235528 [details] [diff] [review]
fix

a=schrep/beltnzer for drivers.
Attachment #235528 - Flags: approval1.8.1? → approval1.8.1+
(Assignee)

Comment 6

12 years ago
Fixed on the 1.8 branch.

/be
Keywords: fixed1.8.1

Comment 7

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

Comment 8

12 years ago
verified fixed 1.8 20060828 windows/mac*/linux
Keywords: fixed1.8.1 → verified1.8.1

Comment 9

12 years ago
verified fixed 1.9 20060830 windows/mac*/linux
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.