Bug 215977 – [JSC] `let [` sequence cannot appear in ExpressionStatement context

• Component: New Bugs
• Status: RESOLVED FIXED

[Yusuke Suzuki]

[JSC] `let [` sequence cannot appear in ExpressionStatement context

[Yusuke Suzuki]

Created attachment 407538 [details]
Patch

[Ross Kirsling]

Comment on attachment 407538 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=407538&action=review

r=me. I'm shocked at how simple this patch is.

> Source/JavaScriptCore/parser/Parser.cpp:1964
> +        failIfTrue(match(OPENBRACKET), "\"let [\" cannot appear at the beginning of expression statement context");

This is fine as an explanation for a spec-aware individual, but I don't think it's clear enough as a user-facing error message.
How about something like "cannot appear in a single-line control flow body"?

[Yusuke Suzuki]

Comment on attachment 407538 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=407538&action=review

>> Source/JavaScriptCore/parser/Parser.cpp:1964
>> +        failIfTrue(match(OPENBRACKET), "\"let [\" cannot appear at the beginning of expression statement context");
> 
> This is fine as an explanation for a spec-aware individual, but I don't think it's clear enough as a user-facing error message.
> How about something like "cannot appear in a single-line control flow body"?

Yeah, I've changed this to `Cannot use lexical declaration in single-statement context.`

[Yusuke Suzuki]

Committed r266327: <https://trac.webkit.org/changeset/266327>

[Radar WebKit Bug Importer]

<rdar://problem/68004458>