352274 - js1_5/Regress/regress-313803.js, result: FAILED

Status: RESOLVED DUPLICATE of bug 352010

(Core :: JavaScript Engine, defect)

Description

[Bob Clary [:bc] (inactive)]

2006091022 1.8 1.9

BUGNUMBER: 313803
STATUS: uneval() on func with embedded objects with getter or setter
FAILED!: uneval() on func with embedded objects with getter or setter
FAILED!: 
Expected value 
'(function ff() {obj = {get foo () {return "foo";}};return 1;})', 
Actual value 
'(function ff() {obj = {get foo() {return "foo";};return 1;})'
FAILED!:

Comment 1

[Brendan Eich [:brendan]]

The test is demanding a space after the getter name -- that's not required, nor should it be (it's not something the decompiler does elsewhere).

/be

Comment 2

[Bob Clary [:bc] (inactive)]

count your braces
(function ff() {obj = {get foo() {return "foo";};return 1;})
SyntaxError on line 1: missing } after property list

Comment 3

[Brendan Eich [:brendan]]

I did count braces.  I ran this in the shell, before marking INVALID and again just now:

js> (function ff() {obj = {get foo () {return "foo";}};return 1;})
function ff() {
    obj = {get foo() {return "foo";}};
    return 1;
}
js> (function ff() {obj = {get foo () {return "foo";}};return 1;}).toString()
function ff() {
    obj = {get foo() {return "foo";}};
    return 1;
}
js> eval((function ff() {obj = {get foo () {return "foo";}};return 1;}).toString())
js> ff
function ff() {
    obj = {get foo() {return "foo";}};
    return 1;
}
js> (function ff() {obj = {get foo () {return "foo";}};return 1;}).toSource()
(function ff() {obj = {get foo() {return "foo";}};return 1;})
js> eval((function ff() {obj = {get foo () {return "foo";}};return 1;}).toSource())
function ff() {
    obj = {get foo() {return "foo";}};
    return 1;
}

eval likes it -- what am I missing?

/be

Comment 4

[Bob Clary [:bc] (inactive)]

*** This bug has been marked as a duplicate of 352010 ***