Title: [136185] trunk/Source/WebCore
Revision
136185
Author
[email protected]
Date
2012-11-29 16:48:59 -0800 (Thu, 29 Nov 2012)

Log Message

Unused parameters on plugins/npapi.cpp
https://bugs.webkit.org/show_bug.cgi?id=103676

Patch by KyungTae Kim <[email protected]> on 2012-11-29
Reviewed by Kentaro Hara.

Because 'instance' and 'menu' are not used when !PLATFORM(QT) || !defined(XP_MACOSX),
use UNUSED_PARAM macro to fix build warning -Wunused-parameter

* plugins/npapi.cpp:
(NPN_PopUpContextMenu):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (136184 => 136185)


--- trunk/Source/WebCore/ChangeLog	2012-11-30 00:47:44 UTC (rev 136184)
+++ trunk/Source/WebCore/ChangeLog	2012-11-30 00:48:59 UTC (rev 136185)
@@ -1,3 +1,16 @@
+2012-11-29  KyungTae Kim  <[email protected]>
+
+        Unused parameters on plugins/npapi.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=103676
+
+        Reviewed by Kentaro Hara.
+
+        Because 'instance' and 'menu' are not used when !PLATFORM(QT) || !defined(XP_MACOSX),
+        use UNUSED_PARAM macro to fix build warning -Wunused-parameter
+
+        * plugins/npapi.cpp:
+        (NPN_PopUpContextMenu):
+
 2012-11-29  Pablo Flouret  <[email protected]>
 
         REGRESSION(r134693): Compilation error on @supports grammar code

Modified: trunk/Source/WebCore/plugins/npapi.cpp (136184 => 136185)


--- trunk/Source/WebCore/plugins/npapi.cpp	2012-11-30 00:47:44 UTC (rev 136184)
+++ trunk/Source/WebCore/plugins/npapi.cpp	2012-11-30 00:48:59 UTC (rev 136185)
@@ -207,6 +207,8 @@
     plugin->popUpContextMenu(menu);
     return NPERR_NO_ERROR;
 #else
+    UNUSED_PARAM(instance);
+    UNUSED_PARAM(menu);
     return NPERR_NO_ERROR;
 #endif // PLATFORM(QT) && defined(XP_MACOSX)
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to