Title: [174509] trunk/Source/_javascript_Core
Revision
174509
Author
[email protected]
Date
2014-10-09 10:01:07 -0700 (Thu, 09 Oct 2014)

Log Message

Fix compiler warning on noreturn function
https://bugs.webkit.org/show_bug.cgi?id=137558

Patch by Adrien Destugues <[email protected]> on 2014-10-09
Reviewed by Darin Adler.

The function is marked "noreturn", but the stub implementation does
return. No new tests: function is never called. Only fixes a warning.

* heap/HeapStatistics.cpp:
(JSC::HeapStatistics::exitWithFailure):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (174508 => 174509)


--- trunk/Source/_javascript_Core/ChangeLog	2014-10-09 17:01:00 UTC (rev 174508)
+++ trunk/Source/_javascript_Core/ChangeLog	2014-10-09 17:01:07 UTC (rev 174509)
@@ -1,3 +1,16 @@
+2014-10-09  Adrien Destugues  <[email protected]>
+
+        Fix compiler warning on noreturn function
+        https://bugs.webkit.org/show_bug.cgi?id=137558
+
+        Reviewed by Darin Adler.
+
+        The function is marked "noreturn", but the stub implementation does
+        return. No new tests: function is never called. Only fixes a warning.
+
+        * heap/HeapStatistics.cpp:
+        (JSC::HeapStatistics::exitWithFailure):
+
 2014-10-09  Akos Kiss  <[email protected]>
 
         Ensure that inline assembly Thunk functions don't conflict with the section designations of the compiler

Modified: trunk/Source/_javascript_Core/heap/HeapStatistics.cpp (174508 => 174509)


--- trunk/Source/_javascript_Core/heap/HeapStatistics.cpp	2014-10-09 17:01:00 UTC (rev 174508)
+++ trunk/Source/_javascript_Core/heap/HeapStatistics.cpp	2014-10-09 17:01:07 UTC (rev 174509)
@@ -28,8 +28,8 @@
 
 #include "Heap.h"
 #include "HeapIterationScope.h"
+#include "JSCInlines.h"
 #include "JSObject.h"
-#include "JSCInlines.h"
 #include "Options.h"
 #include <stdlib.h>
 #if OS(UNIX)
@@ -132,6 +132,7 @@
 
 void HeapStatistics::exitWithFailure()
 {
+    exit(-1);
 }
 
 void HeapStatistics::reportSuccess()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to