331558 - Decompiler: Missing = in default xml namespace statement

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect)

Description

[Erik Fabert]

For example, |default xml namespace = 'abc'| is printed as |default xml namespace 'abc'|. Since I'm not too familiar with E4X this drove me mad for a while when the lines I simply copied and pasted produced syntax errors. ;-)

Comment 1

[Erik Fabert]

Attachment: Add =

I know I promised no more patches but this one was so trivial it doesn't count...

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 216093 [details] [diff] [review]
Add =

brendan must have been asleep when he was writing this. I'll check it in.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Fix checked into trunk.

Comment 4

[Brendan Eich [:brendan]]

Comment on attachment 216093 [details] [diff] [review]
Add =

Totally safe one-constant-char-in-format-string change, good for 1.5.0.3.

/be

Comment 5

[Jay Patel [:jay]]

Comment on attachment 216093 [details] [diff] [review]
Add =

Please check in promptly on the 1.8.0 branch.  Thanks!

Comment 6

[Bob Clary [:bc] (inactive)]

Checking in regress-331558.js;
/cvsroot/mozilla/js/tests/e4x/Regress/regress-331558.js,v  <--  regress-331558
s
initial revision: 1.1
done

Comment 7

[:Gavin Sharp [email: [e-mail]]]

Checked in on both branches.
mozilla/js/src/jsopcode.c 3.89.2.8.2.5
mozilla/js/src/jsopcode.c 3.89.2.12

Comment 8

[Bob Clary [:bc] (inactive)]

verified fixed 1.8.0.4, 1.8, 1.9 win/linux/mac 20060507/20060508 builds