Mesa (master): r300g: remove default case statements in get_param

Marek Olšák mareko at kemper.freedesktop.org
Tue Nov 22 19:57:10 UTC 2011


Module: Mesa
Branch: master
Commit: 684d74e0009c01773e13f476676c0c575f73c615
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=684d74e0009c01773e13f476676c0c575f73c615

Author: Marek Olšák <maraeo at gmail.com>
Date:   Tue Nov 22 20:44:14 2011 +0100

r300g: remove default case statements in get_param

This will throw a compile warning if there's an unhandled CAP.

---

 src/gallium/drivers/r300/r300_screen.c |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/gallium/drivers/r300/r300_screen.c b/src/gallium/drivers/r300/r300_screen.c
index 24d15c4..426e4d1 100644
--- a/src/gallium/drivers/r300/r300_screen.c
+++ b/src/gallium/drivers/r300/r300_screen.c
@@ -148,12 +148,8 @@ static int r300_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
         /* Render targets. */
         case PIPE_CAP_MAX_RENDER_TARGETS:
             return 4;
-
-        default:
-            debug_printf("r300: Warning: Unknown CAP %d in get_param.\n",
-                         param);
-            return 0;
     }
+    return 0;
 }
 
 static int r300_get_shader_param(struct pipe_screen *pscreen, unsigned shader, enum pipe_shader_cap param)
@@ -162,8 +158,7 @@ static int r300_get_shader_param(struct pipe_screen *pscreen, unsigned shader, e
    boolean is_r400 = r300screen->caps.is_r400;
    boolean is_r500 = r300screen->caps.is_r500;
 
-   switch (shader)
-    {
+   switch (shader) {
     case PIPE_SHADER_FRAGMENT:
         switch (param)
         {
@@ -251,7 +246,6 @@ static int r300_get_shader_param(struct pipe_screen *pscreen, unsigned shader, e
         case PIPE_SHADER_CAP_SUBROUTINES:
         case PIPE_SHADER_CAP_INTEGERS:
             return 0;
-        default:;
         }
         break;
     }




More information about the mesa-commit mailing list