Reviewed-by: Brian Paul <[email protected]>

On 02/28/2015 12:02 PM, Matt Turner wrote:
Always indenting break statements makes spotting missing ones easier.

Cc: 10.5 <[email protected]>
---
  src/mesa/main/pack.c | 11 ++++++-----
  1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/mesa/main/pack.c b/src/mesa/main/pack.c
index 3b77c49..f723608 100644
--- a/src/mesa/main/pack.c
+++ b/src/mesa/main/pack.c
@@ -1487,20 +1487,20 @@ _mesa_pack_luminance_from_rgba_integer(GLuint n,
           case GL_UNSIGNED_BYTE: {
              GLbyte *dst = (GLbyte *) dstAddr;
              dst[i] = lum32;
+            break;
           }
-         break;
           case GL_SHORT:
           case GL_UNSIGNED_SHORT: {
              GLshort *dst = (GLshort *) dstAddr;
              dst[i] = lum32;
+            break;
           }
-         break;
           case GL_INT:
           case GL_UNSIGNED_INT: {
              GLint *dst = (GLint *) dstAddr;
              dst[i] = lum32;
+            break;
           }
-         break;
           }
        }
        return;
@@ -1525,21 +1525,22 @@ _mesa_pack_luminance_from_rgba_integer(GLuint n,
              GLbyte *dst = (GLbyte *) dstAddr;
              dst[2*i] = lum32;
              dst[2*i+1] = alpha;
+            break;
           }
           case GL_SHORT:
           case GL_UNSIGNED_SHORT: {
              GLshort *dst = (GLshort *) dstAddr;
              dst[i] = lum32;
              dst[2*i+1] = alpha;
+            break;
           }
-         break;
           case GL_INT:
           case GL_UNSIGNED_INT: {
              GLint *dst = (GLint *) dstAddr;
              dst[i] = lum32;
              dst[2*i+1] = alpha;
+            break;
           }
-         break;
           }
        }
        return;


_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to