Changeset 191316 in webkit

Timestamp:

Oct 19, 2015, 2:21:44 PM (10 years ago)

Author:

Csaba Osztrogonác

Message:

Fix the binding generator after r191176
​https://bugs.webkit.org/show_bug.cgi?id=150320

Reviewed by Darin Adler.

• bindings/scripts/CodeGeneratorJS.pm:

(GenerateConstructorHelperMethods):

• bindings/scripts/test/JS/JSTestInterface.cpp:

(WebCore::JSTestInterfaceConstructor::getConstructData):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/scripts/CodeGeneratorJS.pm (modified) (1 diff)
• bindings/scripts/test/JS/JSTestInterface.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-10-19  Csaba Osztrogonác  <ossy@webkit.org>
+
+        Fix the binding generator after r191176
+        https://bugs.webkit.org/show_bug.cgi?id=150320
+
+        Reviewed by Darin Adler.
+
+        * bindings/scripts/CodeGeneratorJS.pm:
+        (GenerateConstructorHelperMethods):
+        * bindings/scripts/test/JS/JSTestInterface.cpp:
+        (WebCore::JSTestInterfaceConstructor::getConstructData):
+
 2015-10-19  Commit Queue  <commit-queue@webkit.org>
 

trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm

@@ -4928 +4928 @@
     my $conditionalString = $codeGenerator->GenerateConstructorConditionalString($interface);
     if ($conditionalString) {
-        push(@$outputArray, "template<> ConstructType ${constructorClassName}::getConstructData(JSCell*, ConstructData& constructData)\n");
+        push(@$outputArray, "template<> ConstructType ${constructorClassName}::getConstructData(JSCell* cell, ConstructData& constructData)\n");
         push(@$outputArray, "{\n");
         push(@$outputArray, "#if $conditionalString\n");
+        push(@$outputArray, "    UNUSED_PARAM(cell);\n");
         push(@$outputArray, "    constructData.native.function = construct;\n");
         push(@$outputArray, "    return ConstructTypeHost;\n");

trunk/Source/WebCore/bindings/scripts/test/JS/JSTestInterface.cpp

@@ -277 +277 @@
 }
 
-template<> ConstructType JSTestInterfaceConstructor::getConstructData(JSCell*, ConstructData& constructData)
+template<> ConstructType JSTestInterfaceConstructor::getConstructData(JSCell* cell, ConstructData& constructData)
 {
 #if ENABLE(TEST_INTERFACE)
+    UNUSED_PARAM(cell);
     constructData.native.function = construct;
     return ConstructTypeHost;