This looks good. Sorry for the missing break :-(
Iago
On Sat, 2015-02-28 at 11:02 -0800, Matt Turner wrote:
> Always indenting break statements makes spotting missing ones easier.
>
> Cc: 10.5 <mesa-sta...@lists.freedesktop.org>
> ---
> 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
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
