Mesa (master): u_debug_describe: use switch instead of if chain

Luca Barbieri lb at kemper.freedesktop.org
Sat Aug 21 10:47:54 UTC 2010


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

Author: Luca Barbieri <luca at luca-barbieri.com>
Date:   Sat Aug 21 12:47:18 2010 +0200

u_debug_describe: use switch instead of if chain

---

 src/gallium/auxiliary/util/u_debug_describe.c |   24 +++++++++++++++++-------
 1 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/src/gallium/auxiliary/util/u_debug_describe.c b/src/gallium/auxiliary/util/u_debug_describe.c
index f21ebd0..1c90ff3 100644
--- a/src/gallium/auxiliary/util/u_debug_describe.c
+++ b/src/gallium/auxiliary/util/u_debug_describe.c
@@ -38,20 +38,30 @@ debug_describe_reference(char* buf, const struct pipe_reference*ptr)
 void
 debug_describe_resource(char* buf, const struct pipe_resource *ptr)
 {
-   if(ptr->target == PIPE_BUFFER)
+   switch(ptr->target)
+   {
+   case PIPE_BUFFER:
       util_sprintf(buf, "pipe_buffer<%u>", (unsigned)util_format_get_stride(ptr->format, ptr->width0));
-   else if(ptr->target == PIPE_TEXTURE_1D)
+      break;
+   case PIPE_TEXTURE_1D:
       util_sprintf(buf, "pipe_texture1d<%u,%s,%u>", ptr->width0, util_format_short_name(ptr->format), ptr->last_level);
-   else if(ptr->target == PIPE_TEXTURE_2D)
+      break;
+   case PIPE_TEXTURE_2D:
       util_sprintf(buf, "pipe_texture2d<%u,%u,%s,%u>", ptr->width0, ptr->height0, util_format_short_name(ptr->format), ptr->last_level);
-   else if(ptr->target == PIPE_TEXTURE_RECT)
+      break;
+   case PIPE_TEXTURE_RECT:
       util_sprintf(buf, "pipe_texture_rect<%u,%u,%s>", ptr->width0, ptr->height0, util_format_short_name(ptr->format));
-   else if(ptr->target == PIPE_TEXTURE_CUBE)
+      break;
+   case PIPE_TEXTURE_CUBE:
       util_sprintf(buf, "pipe_texture_cube<%u,%u,%s,%u>", ptr->width0, ptr->height0, util_format_short_name(ptr->format), ptr->last_level);
-   else if(ptr->target == PIPE_TEXTURE_3D)
+      break;
+   case PIPE_TEXTURE_3D:
       util_sprintf(buf, "pipe_texture3d<%u,%u,%u,%s,%u>", ptr->width0, ptr->height0, ptr->depth0, util_format_short_name(ptr->format), ptr->last_level);
-   else
+      break;
+   default:
       util_sprintf(buf, "pipe_martian_resource<%u>", ptr->target);
+      break;
+   }
 }
 
 void




More information about the mesa-commit mailing list