Title: [231901] trunk/Source/_javascript_Core
Revision
231901
Author
sbar...@apple.com
Date
2018-05-17 09:03:27 -0700 (Thu, 17 May 2018)

Log Message

Unreviewed. Try to fix windows build.

* builtins/BuiltinExecutables.cpp:
(JSC::BuiltinExecutables::defaultConstructorSourceCode):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (231900 => 231901)


--- trunk/Source/_javascript_Core/ChangeLog	2018-05-17 15:52:49 UTC (rev 231900)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-05-17 16:03:27 UTC (rev 231901)
@@ -1,3 +1,10 @@
+2018-05-17  Saam Barati  <sbar...@apple.com>
+
+        Unreviewed. Try to fix windows build.
+
+        * builtins/BuiltinExecutables.cpp:
+        (JSC::BuiltinExecutables::defaultConstructorSourceCode):
+
 2018-05-16  Saam Barati  <sbar...@apple.com>
 
         UnlinkedFunctionExecutable doesn't need a parent source override field since it's only used for default class constructors

Modified: trunk/Source/_javascript_Core/builtins/BuiltinExecutables.cpp (231900 => 231901)


--- trunk/Source/_javascript_Core/builtins/BuiltinExecutables.cpp	2018-05-17 15:52:49 UTC (rev 231900)
+++ trunk/Source/_javascript_Core/builtins/BuiltinExecutables.cpp	2018-05-17 16:03:27 UTC (rev 231901)
@@ -46,7 +46,7 @@
 {
     switch (constructorKind) {
     case ConstructorKind::None:
-        RELEASE_ASSERT_NOT_REACHED();
+        break;
     case ConstructorKind::Base: {
         static NeverDestroyed<const String> baseConstructorCode(MAKE_STATIC_STRING_IMPL("(function () { })"));
         static LazyNeverDestroyed<SourceCode> result;
@@ -66,6 +66,8 @@
         return result;
     }
     }
+    RELEASE_ASSERT_NOT_REACHED();
+    return SourceCode();
 }
 
 UnlinkedFunctionExecutable* BuiltinExecutables::createDefaultConstructor(ConstructorKind constructorKind, const Identifier& name)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to