303723 - API compatibility vs. the fix for bug 299209

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Brendan Eich [:brendan]]

See bug 303163.  Someone remind me of a report (in n.p.m.jseng, or by mail) of
this problem, where compiling or evaluating "function () {...}" via tha JS API
used to work, but no longer does because of the fix for bug 299209.

We could hack an internal flag or option that tells the compiler to allow this
case.  We'd want it to apply only directly (when the API is passed a string of
the form "function (...) {...}"), not for other source strings that embed
anonymous function declarations at lower levels of their syntax trees.

/be

Comment 1

[Bob Clary [:bc] (inactive)]

(In reply to comment #0)

Not sure this is what you want since it is not via the C JS API, but
mozilla/js/tests/ecma_3/Statements/regress-194364.js and
mozilla/js/tests/js1_5/Regress/regress-224956.js were modified due to bug 299209.

Comment 2

[Brendan Eich [:brendan]]

Attachment: restore total backward compatibility

Can't be partially backward compatible any more than partly pregnant.  Note the
syntax error for declaration-level anonymous function case that was added for
bug 299209 remains, but only #if !JS_HAS_LEXICAL_CLOSURE.

/be

Comment 3

[Mike Shaver (:shaver emeritus)]

Comment on attachment 191924 [details] [diff] [review]
restore total backward compatibility

r=shaver

Comment 4

[Brendan Eich [:brendan]]

Fixed, thanks.

/be