Attachment 348896 Details for Bug 189292 – patch for landing.

[patch] patch for landing.

bug-189292.patch (text/plain), 3.04 KB, created by Mark Lam on 2018-09-04 23:17:20 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Lam

Created: 2018-09-04 23:17:20 PDT

Size: 3.04 KB

patch

obsolete

>Index: JSTests/ChangeLog
>===================================================================
>--- JSTests/ChangeLog	(revision 235659)
>+++ JSTests/ChangeLog	(working copy)
>@@ -1,3 +1,13 @@
>+2018-09-04  Mark Lam  <mark.lam@apple.com>
>+
>+        isAsyncGeneratorMethodParseMode() should check for SourceParseMode::AsyncGeneratorWrapperMethodMode.
>+        https://bugs.webkit.org/show_bug.cgi?id=189292
>+        <rdar://problem/38907433>
>+
>+        Reviewed by Saam Barati.
>+
>+        * stress/regress-189292.js: Added.
>+
> 2018-09-04  David Fenton  <david_fenton@apple.com>
> 
>         Skip Test on 32-bit platforms.
>Index: JSTests/stress/regress-189292.js
>===================================================================
>--- JSTests/stress/regress-189292.js	(nonexistent)
>+++ JSTests/stress/regress-189292.js	(working copy)
>@@ -0,0 +1,18 @@
>+//@ runDefault
>+
>+function assert(a, b) {
>+    if (a != b)
>+        throw "FAIL";
>+}
>+
>+function test(script) {
>+    try {
>+        eval(script);
>+    } catch (e) {
>+        return e;
>+    }
>+}
>+
>+assert(test("class C1 { async constructor() { } }"), "SyntaxError: Cannot declare an async method named 'constructor'.");
>+assert(test("class C1 { *constructor() { } }"), "SyntaxError: Cannot declare a generator function named 'constructor'.");
>+assert(test("class C1 { async *constructor() { } }"), "SyntaxError: Cannot declare an async generator method named 'constructor'.");
>Index: Source/JavaScriptCore/ChangeLog
>===================================================================
>--- Source/JavaScriptCore/ChangeLog	(revision 235659)
>+++ Source/JavaScriptCore/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2018-09-04  Mark Lam  <mark.lam@apple.com>
>+
>+        isAsyncGeneratorMethodParseMode() should check for SourceParseMode::AsyncGeneratorWrapperMethodMode.
>+        https://bugs.webkit.org/show_bug.cgi?id=189292
>+        <rdar://problem/38907433>
>+
>+        Reviewed by Saam Barati.
>+
>+        Previously, isAsyncGeneratorMethodParseMode() was checking for AsyncGeneratorWrapperFunctionMode
>+        instead of AsyncGeneratorWrapperMethodMode.  This patch fixes it
>+        to check for AsyncGeneratorWrapperMethodMode (to match what is expected as indicated
>+        in the name isAsyncGeneratorMethodParseMode).
>+
>+        * parser/ParserModes.h:
>+        (JSC::isAsyncGeneratorMethodParseMode):
>+
> 2018-09-04  Michael Saboff  <msaboff@apple.com>
> 
>         Unreviewed indentations change.
>Index: Source/JavaScriptCore/parser/ParserModes.h
>===================================================================
>--- Source/JavaScriptCore/parser/ParserModes.h	(revision 235659)
>+++ Source/JavaScriptCore/parser/ParserModes.h	(working copy)
>@@ -188,7 +188,7 @@ ALWAYS_INLINE bool isAsyncMethodParseMod
>     
> ALWAYS_INLINE bool isAsyncGeneratorMethodParseMode(SourceParseMode parseMode)
> {
>-    return SourceParseModeSet(SourceParseMode::AsyncGeneratorWrapperFunctionMode).contains(parseMode);
>+    return SourceParseModeSet(SourceParseMode::AsyncGeneratorWrapperMethodMode).contains(parseMode);
> }
> 
> ALWAYS_INLINE bool isMethodParseMode(SourceParseMode parseMode)

Actions: View | Formatted Diff | Diff

Attachments on bug 189292: 348881 | 348896