Title: [206199] trunk/Source/WebCore
Revision
206199
Author
nael.ouedra...@crf.canon.fr
Date
2016-09-20 23:44:04 -0700 (Tue, 20 Sep 2016)

Log Message

VariadicHelper::Result should not include constructor to initialize members
https://bugs.webkit.org/show_bug.cgi?id=162298

Reviewed by Darin Adler.

Remove the constructor since member-wise initialization does the same thing.

No new tests required.

* bindings/js/JSDOMBinding.h: Remove useless constructor.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (206198 => 206199)


--- trunk/Source/WebCore/ChangeLog	2016-09-21 06:34:13 UTC (rev 206198)
+++ trunk/Source/WebCore/ChangeLog	2016-09-21 06:44:04 UTC (rev 206199)
@@ -1,3 +1,16 @@
+2016-09-20  Nael Ouedraogo  <nael.ouedra...@crf.canon.fr>
+
+        VariadicHelper::Result should not include constructor to initialize members
+        https://bugs.webkit.org/show_bug.cgi?id=162298
+
+        Reviewed by Darin Adler.
+
+        Remove the constructor since member-wise initialization does the same thing.
+
+        No new tests required.
+
+        * bindings/js/JSDOMBinding.h: Remove useless constructor.
+
 2016-09-20  Alex Christensen  <achristen...@webkit.org>
 
         Optimize URLParser

Modified: trunk/Source/WebCore/bindings/js/JSDOMBinding.h (206198 => 206199)


--- trunk/Source/WebCore/bindings/js/JSDOMBinding.h	2016-09-21 06:34:13 UTC (rev 206198)
+++ trunk/Source/WebCore/bindings/js/JSDOMBinding.h	2016-09-21 06:44:04 UTC (rev 206199)
@@ -358,11 +358,6 @@
     using Container = Vector<Item>;
 
     struct Result {
-        Result(size_t argumentIndex, Optional<Container>&& arguments)
-            : argumentIndex(argumentIndex)
-            , arguments(WTFMove(arguments))
-        {
-        }
         size_t argumentIndex;
         Optional<Container> arguments;
     };
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to