798642 - AST toSource on getter/setter inserting 'function' keyword

Status: RESOLVED INACTIVE

(Rhino Graveyard :: Core, defect)

Description

[tntim96]

Attachment: RhinoGetterSetterTest.patch

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/20100101 Firefox/15.0.1
Build ID: 20120905151427

Steps to reproduce:

Parsed script with getter/setter logic and printed it out again.


Actual results:

var o = {_x: 123, get xfunction() {
  return this._x;
}, set xfunction(value) {
  this._x = value;
}};



Expected results:

var o = {
  _x: 123,
  get x() {
    return this._x;
  },
  set x(value) {
    this._x = value;
  }
};

Comment 1

[tntim96]

Attachment: Patch to fix syntax but not indentation

The PartialFixButIndentationIncorrect.patch has not been run against all the tests.

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Closing. Bug management is now done here:
https://github.com/mozilla/rhino