Bug 137107 – Clean up unused parameter warnings

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Yusuke Suzuki]

[GTK] Clean up unused parameter warnings

[Yusuke Suzuki]

Created attachment 238643 [details]
Patch

[Csaba Osztrogonác]

Comment on attachment 238643 [details]
Patch

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

> Source/WebKit2/WebProcess/WebPage/WebInspector.h:62
> -    void didReceiveInvalidMessage(IPC::Connection*, IPC::StringReference messageReceiverName, IPC::StringReference messageName) { close(); }
> +    void didReceiveInvalidMessage(IPC::Connection*, IPC::StringReference messageReceiverName, IPC::StringReference messageName)
> +    {
> +        UNUSED_PARAM(messageReceiverName);
> +        UNUSED_PARAM(messageName);
> +        close();
> +    }

In this case we should simple remove the name of the parameter.
UNUSED_PARAM is used if the parameter is unused only if
some ENABLE(FOO) guard is true or false.

Otherwise it is already fixed in bug137105.

> Source/WebKit2/WebProcess/WebPage/WebInspectorUI.h:57
> -    void didReceiveInvalidMessage(IPC::Connection*, IPC::StringReference messageReceiverName, IPC::StringReference messageName) { closeWindow(); }
> +    void didReceiveInvalidMessage(IPC::Connection*, IPC::StringReference messageReceiverName, IPC::StringReference messageName)
> +    {
> +        UNUSED_PARAM(messageReceiverName);
> +        UNUSED_PARAM(messageName);
> +        closeWindow();
> +    }

Already fixed in bug137105.

[Yusuke Suzuki]

Comment on attachment 238643 [details]
Patch

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

Thank you for your review, I'll upload the revised one.

> Source/WebKit2/Scripts/webkit/messages.py:624
>              result.append('    UNUSED_PARAM(connection);\n')

In this case, we may use `connection` in the generated code.
So using `UNUSED_PARAM`. It is aligned to the following `UNUSED_PARAM(decoder)` etc.

> Source/WebKit2/UIProcess/API/C/WKPage.cpp:1885
>  bool WKPageIsPlayingAudio(WKPageRef page)

I'll drop `page` instead of adding `UNUSED_PARAM(page)`.

>> Source/WebKit2/WebProcess/WebPage/WebInspector.h:62
>> +    }
> 
> In this case we should simple remove the name of the parameter.
> UNUSED_PARAM is used if the parameter is unused only if
> some ENABLE(FOO) guard is true or false.
> 
> Otherwise it is already fixed in bug137105.

Thanks! OK, so I'll drop this change from this patch.

>> Source/WebKit2/WebProcess/WebPage/WebInspectorUI.h:57
>> +    }
> 
> Already fixed in bug137105.

ditto :)

[Yusuke Suzuki]

Created attachment 238653 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 238653 [details]
Patch

Clearing flags on attachment: 238653

Committed r173990: <http://trac.webkit.org/changeset/173990>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.