352198 - Incorrect decompilation for "yield (yield);"

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect, P2)

Description

[Jesse Ruderman]

js> function() { yield (yield); }

function () {
    yield yield;
}

js> function() { yield yield; }

typein:9: SyntaxError: missing ; before statement:

Comment 1

[Brendan Eich [:brendan]]

Attachment: Followup fix to the patch for bug 352026

I'll fold this into the branch patch for bug 352026.

/be

Comment 2

[Brendan Eich [:brendan]]

Attachment: fix missing paren

Comment 3

[Mike Schroepfer]

Comment on attachment 237793 [details] [diff] [review]
fix missing paren

a=schrep

Comment 4

[Brendan Eich [:brendan]]

Fixed on the 1.8 branch.

/be

Comment 5

[Brendan Eich [:brendan]]

Already fixed on the trunk:

revision 3.172
date: 2006/09/11 21:26:30;  author: brendan%mozilla.org;  state: Exp;  lines: +1 -1
Fix yield (yield) decompilation (352198, r=mrbkap).

/be

Comment 6

[Bob Clary [:bc] (inactive)]

Checking in regress-352198.js;
/cvsroot/mozilla/js/tests/js1_7/geniter/regress-352198.js,v  <--  regress-352198.js
initial revision: 1.1
done

Comment 7

[Bob Clary [:bc] (inactive)]

verified fixed 1.8 20060914 windows/linux 1.9 20060914 windows/mac*/linux