Bug 142213 - Fix build warning in WebKit2/Shared module
Summary: Fix build warning in WebKit2/Shared module
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Shivakumar J M
Depends on:
Reported: 2015-03-02 22:47 PST by Shivakumar J M
Modified: 2015-03-03 21:26 PST (History)
3 users (show)

See Also:

Patch (1.24 KB, patch)
2015-03-03 00:32 PST, Shivakumar J M
simon.fraser: review-
simon.fraser: commit-queue-
Details | Formatted Diff | Diff
Patch-Updated-Review1 (1.30 KB, patch)
2015-03-03 20:25 PST, Shivakumar J M
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shivakumar J M 2015-03-02 22:47:01 PST
Got below build warning on latest build, might have casued by issue: https://bugs.webkit.org/show_bug.cgi?id=138684
../../Source/WebKit2/Shared/WebCoreArgumentCoders.cpp:368:13: warning: unused parameter ‘element’ [-Wunused-parameter]
 static void pathPointCountApplierFunction(void* info, const PathElement* element)
Comment 1 Shivakumar J M 2015-03-03 00:32:25 PST
Created attachment 247754 [details]

Fix build warning by using UNUSED_PARAM macro.
Comment 2 Simon Fraser (smfr) 2015-03-03 08:04:37 PST
Comment on attachment 247754 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=247754&action=review

> Source/WebKit2/Shared/WebCoreArgumentCoders.cpp:370
>  static void pathPointCountApplierFunction(void* info, const PathElement* element)
>  {
> +    UNUSED_PARAM(element);

It would be better to just remove the parameter name.
Comment 3 Shivakumar J M 2015-03-03 20:25:08 PST
Created attachment 247828 [details]

updated the patch as per review comments.
Comment 4 WebKit Commit Bot 2015-03-03 21:26:14 PST
Comment on attachment 247828 [details]

Clearing flags on attachment: 247828

Committed r180988: <http://trac.webkit.org/changeset/180988>
Comment 5 WebKit Commit Bot 2015-03-03 21:26:19 PST
All reviewed patches have been landed.  Closing bug.