Attachment 371780 Details for Bug 198724 – PATCH

[patch] PATCH

198724.diff (text/plain), 1.71 KB, created by Basuke Suzuki on 2019-06-10 14:49:38 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Basuke Suzuki

Created: 2019-06-10 14:49:38 PDT

Size: 1.71 KB

patch

obsolete

>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index b708a4becd0..4c7ae9d2f48 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-06-10  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [WinCairo] Remove build warning from RemoteInspector.
>+        https://bugs.webkit.org/show_bug.cgi?id=198724
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        In `RemoteInspectorConnectionClient.h`, an interface was defined with empty implementation.
>+        This method is to be overwritten by sub classes so that parameter name is important
>+        so they are commented out rather than just removing from the definition.
>+
>+        * inspector/remote/RemoteInspector.h:
>+
> 2019-06-10  Adrian Perez de Castro  <aperez@igalia.com>
> 
>         [JSC] Linker fails when unified sources are not in use
>diff --git a/Source/JavaScriptCore/inspector/remote/socket/RemoteInspectorConnectionClient.h b/Source/JavaScriptCore/inspector/remote/socket/RemoteInspectorConnectionClient.h
>index c6a62962222..7bf9db18067 100644
>--- a/Source/JavaScriptCore/inspector/remote/socket/RemoteInspectorConnectionClient.h
>+++ b/Source/JavaScriptCore/inspector/remote/socket/RemoteInspectorConnectionClient.h
>@@ -37,7 +37,7 @@ namespace Inspector {
> class RemoteInspectorConnectionClient : public CanMakeWeakPtr<RemoteInspectorConnectionClient> {
> public:
>     void didReceiveWebInspectorEvent(ConnectionID, Vector<uint8_t>&&);
>-    virtual void didAccept(ConnectionID acceptedID, ConnectionID listenerID, Socket::Domain) { }
>+    virtual void didAccept(ConnectionID /* acceptedID */, ConnectionID /* listenerID */, Socket::Domain) { }
>     virtual void didClose(ConnectionID) = 0;
> 
>     struct Event {

Actions: View | Formatted Diff | Diff

Attachments on bug 198724: 371765 | 371780