Bug 215977

Summary: [JSC] `let [` sequence cannot appear in ExpressionStatement context
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: New BugsAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, ross.kirsling, saam, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch ross.kirsling: review+

Description Yusuke Suzuki 2020-08-29 02:27:12 PDT
[JSC] `let [` sequence cannot appear in ExpressionStatement context
Comment 1 Yusuke Suzuki 2020-08-29 02:30:15 PDT
Created attachment 407538 [details]
Patch
Comment 2 Ross Kirsling 2020-08-29 02:53:34 PDT
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"?
Comment 3 Yusuke Suzuki 2020-08-29 13:43:30 PDT
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.`
Comment 4 Yusuke Suzuki 2020-08-29 13:44:31 PDT
Committed r266327: <https://trac.webkit.org/changeset/266327>
Comment 5 Radar WebKit Bug Importer 2020-08-29 13:45:26 PDT
<rdar://problem/68004458>