Title: [216837] trunk/Source/_javascript_Core
Revision
216837
Author
[email protected]
Date
2017-05-13 22:37:52 -0700 (Sat, 13 May 2017)

Log Message

Unused lambda capture in JSContextGroupAddMarkingConstraint()
<https://webkit.org/b/172084>

Reviewed by Saam Barati.

Fixes the following warning with newer clang:

    Source/_javascript_Core/API/JSMarkingConstraintPrivate.cpp:78:11: error: lambda capture 'vm' is not used [-Werror,-Wunused-lambda-capture]
            [&vm, constraintCallback, userData]
              ^

* API/JSMarkingConstraintPrivate.cpp:
(JSContextGroupAddMarkingConstraint): Remove unused lambda
capture for '&vm'.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/API/JSMarkingConstraintPrivate.cpp (216836 => 216837)


--- trunk/Source/_javascript_Core/API/JSMarkingConstraintPrivate.cpp	2017-05-14 05:27:18 UTC (rev 216836)
+++ trunk/Source/_javascript_Core/API/JSMarkingConstraintPrivate.cpp	2017-05-14 05:37:52 UTC (rev 216837)
@@ -75,7 +75,7 @@
     auto constraint = std::make_unique<MarkingConstraint>(
         toCString("Amc", constraintIndex, "(", RawPointer(bitwise_cast<void*>(constraintCallback)), ")"),
         toCString("API Marking Constraint #", constraintIndex, " (", RawPointer(bitwise_cast<void*>(constraintCallback)), ", ", RawPointer(userData), ")"),
-        [&vm, constraintCallback, userData]
+        [constraintCallback, userData]
         (SlotVisitor& slotVisitor, const VisitingTimeout&) {
             Marker marker;
             marker.IsMarked = isMarked;
@@ -88,6 +88,3 @@
     
     vm.heap.addMarkingConstraint(WTFMove(constraint));
 }
-
-
-

Modified: trunk/Source/_javascript_Core/ChangeLog (216836 => 216837)


--- trunk/Source/_javascript_Core/ChangeLog	2017-05-14 05:27:18 UTC (rev 216836)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-05-14 05:37:52 UTC (rev 216837)
@@ -1,5 +1,22 @@
 2017-05-13  David Kilzer  <[email protected]>
 
+        Unused lambda capture in JSContextGroupAddMarkingConstraint()
+        <https://webkit.org/b/172084>
+
+        Reviewed by Saam Barati.
+
+        Fixes the following warning with newer clang:
+
+            Source/_javascript_Core/API/JSMarkingConstraintPrivate.cpp:78:11: error: lambda capture 'vm' is not used [-Werror,-Wunused-lambda-capture]
+                    [&vm, constraintCallback, userData]
+                      ^
+
+        * API/JSMarkingConstraintPrivate.cpp:
+        (JSContextGroupAddMarkingConstraint): Remove unused lambda
+        capture for '&vm'.
+
+2017-05-13  David Kilzer  <[email protected]>
+
         [JSC] config.rb fails when checking some clang versions
         <https://webkit.org/b/172082>
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to