Bug 142213

Summary: Fix build warning in WebKit2/Shared module
Product: WebKit Reporter: Shivakumar J M <shiva.jm>
Component: Layout and RenderingAssignee: Shivakumar J M <shiva.jm>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
simon.fraser: review-, simon.fraser: commit-queue-
Patch-Updated-Review1 none

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]
Patch

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]
Patch

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]
Patch-Updated-Review1

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

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.