630213 - ASTSerializer::leftAssociate right-associates

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jason Orendorff [:jorendorff]]

I was playing with my minimizer this weekend and came across this.

Reflect.parse("a-b-c") produces a tree for "a-(b-c)".

  js> uneval(Reflect.parse("a-b-c",{loc:0})) ===
      uneval(Reflect.parse("a-(b-c)",{loc:0}))
  true

But they're two different programs.

  js> var a=3, b=2, c=1;
  js> a-b-c
  0
  js> a-(b-c)
  2

Comment 1

[Jason Orendorff [:jorendorff]]

Attachment: v1

Comment 2

[Dave Herman [:dherman]]

Oy, this one's embarrassing. Patch is good, thanks. r=me

Dave

Comment 3

[Jason Orendorff [:jorendorff]]

http://hg.mozilla.org/tracemonkey/rev/5c9527251fe9

Comment 4

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/mozilla-central/rev/5c9527251fe9