Attachment 373214 Details for Bug 199352 – Patch

[patch] Patch

bug-199352-20190701103231.patch (text/plain), 2.43 KB, created by Paulo Matos on 2019-07-01 01:32:42 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Paulo Matos

Created: 2019-07-01 01:32:42 PDT

Size: 2.43 KB

patch

obsolete

>Subversion Revision: 246674
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 2b8d0efbeea516ad37c7e822edcccd0bdd1a8732..147ff263933d1a1d86e9e2cfa8af12f4eba7100f 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-07-01  Paulo Matos  <pmatos@linki.tools>
>+
>+        Fix typo in if/else block and remove dead assignment
>+        https://bugs.webkit.org/show_bug.cgi?id=199352
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * yarr/YarrPattern.cpp:
>+        (JSC::Yarr::YarrPattern::dumpPattern): Fix typo in if/else block and remove dead assignment
>+
> 2019-06-20  Justin Michaud  <justin_michaud@apple.com>
> 
>         [WASM-References] Add extra tests for Wasm references + fix element parsing and subtyping bugs
>diff --git a/Source/JavaScriptCore/yarr/YarrPattern.cpp b/Source/JavaScriptCore/yarr/YarrPattern.cpp
>index d218513f5592cac7cfd736dbaff7eac5669c228a..49f77f69c685c65203df3f034c90d1877c92dc9b 100644
>--- a/Source/JavaScriptCore/yarr/YarrPattern.cpp
>+++ b/Source/JavaScriptCore/yarr/YarrPattern.cpp
>@@ -1434,35 +1434,34 @@ void YarrPattern::dumpPattern(PrintStream& out, const String& patternString)
>     dumpPatternString(out, patternString);
> 
>     if (m_flags) {
>-        bool printSeperator = false;
>+        bool printSeparator = false;
>         out.print(" (");
>         if (global()) {
>             out.print("global");
>-            printSeperator = true;
>+            printSeparator = true;
>         }
>         if (ignoreCase()) {
>-            if (printSeperator)
>+            if (printSeparator)
>                 out.print("|");
>             out.print("ignore case");
>-            printSeperator = true;
>+            printSeparator = true;
>         }
>         if (multiline()) {
>-            if (printSeperator)
>+            if (printSeparator)
>                 out.print("|");
>             out.print("multiline");
>-            printSeperator = true;
>+            printSeparator = true;
>         }
>         if (unicode()) {
>-            if (printSeperator)
>+            if (printSeparator)
>                 out.print("|");
>             out.print("unicode");
>-            printSeperator = true;
>+            printSeparator = true;
>         }
>         if (sticky()) {
>-            if (printSeperator)
>+            if (printSeparator)
>                 out.print("|");
>             out.print("sticky");
>-            printSeperator = true;
>         }
>         out.print(")");
>     }

Actions: View | Formatted Diff | Diff

Attachments on bug 199352: 373214