Changeset 151406 in webkit

Timestamp:

Jun 10, 2013 3:10:09 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[WK2][CoordinatedGraphics] Add default case to switches to fix strict build
​https://bugs.webkit.org/show_bug.cgi?id=117416

Patch by Sergio Correia <Sergio Correia> on 2013-06-10
Reviewed by Noam Rosenthal.

This patch fixes a strict build by asserting on the default case of two
switch statements where gcc was previously complaining of unhandled cases.

• Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:

(CoreIPC::::encode):
(CoreIPC::::decode):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp (modified) (2 diffs)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-06-10  Sergio Correia  <sergio.correia@openbossa.org>
+
+        [WK2][CoordinatedGraphics] Add default case to switches to fix strict build
+        https://bugs.webkit.org/show_bug.cgi?id=117416
+
+        Reviewed by Noam Rosenthal.
+
+        This patch fixes a strict build by asserting on the default case of two
+        switch statements where gcc was previously complaining of unhandled cases.
+
+        * Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:
+        (CoreIPC::::encode):
+        (CoreIPC::::decode):
+
 2013-06-10  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp

@@ -183 +183 @@
                     CustomFilterTransformParameter* transformParameter = static_cast<CustomFilterTransformParameter*>(parameter.get());
                     ArgumentCoder<TransformOperations>::encode(encoder, transformParameter->operations());
+                    break;
+                }
+                default: {
+                    ASSERT_NOT_REACHED();
                     break;
                 }
@@ -320 +324 @@
                     break;
                 }
+                default: {
+                    ASSERT_NOT_REACHED();
+                    return false;
+                }
                 }
             }