Repository navigation
Function declaration throws in REPL #10950
Description
Activity
- changed the title
[-]Function declatarion throws in REPL[/-][+]Function declaration throws in REPL[/+]on Jan 22, 2017 So, I get these results on
master:> function f(){} f() var f = function f(){} f() ^ SyntaxError: Unexpected identifier > var f = function f(){} f() var f = function f(){} f() ^ SyntaxError: Unexpected identifier > function f(){}, a=9 undefined > a 9 >
To me, I think all variations of
function f(){} f()are incorrect, butfunction f(){}, a=9sounds how commas are "supposed" to work.Note: I'm no spec expert on what is and isn't valid.
It's caused by bb9eabe, which is a fix/workaround for #548 and scheduled to be removed when we upgrade to V8 5.5, see #9618.
I don't think we'll be able to fix this in in v7.x and older without reintroducing the issue that bb9eabe was supposed to address.
EDIT: For posterity: bb9eabe is from #7624; the commit log is missing the PR-URL tag.
Reacted by Jeremiah Senkpiel, Gibson Fahnestock and F. HinkelmannThis is fixed on master now.
Closing according to #10950 (comment), feel free to reopen if the issue persists
The following code is executed normally from a file, but throws from REPL.
I guess the function declaration is converted to function expression which unlike the declaration requires semicolon. However, everything works fine with
5; function f(){} f().Also, this code works in REPL:
function f(){}, a=9, but it shouldn't.