fix all those warnings

diff --git a/src/mesa/drivers/dri/mach64/mach64_native_vb.c b/src/mesa/drivers/dri/mach64/mach64_native_vb.c
index db7bcd3334852d85b243cacc156c2a978c1de018..7872c206f58ccd5a335697e66c8af1fb8052eff7 100644 (file)
--- a/src/mesa/drivers/dri/mach64/mach64_native_vb.c
+++ b/src/mesa/drivers/dri/mach64/mach64_native_vb.c
@@ -78,18 +78,20 @@ void TAG(translate_vertex)(GLcontext *ctx,
         dst->win[3] = LE32_IN_FLOAT( p++ );
        
       case NOTEX_VERTEX_FORMAT:
-        dst->specular[2] = *((GLubyte *)p)++;
-        dst->specular[1] = *((GLubyte *)p)++;
-        dst->specular[0] = *((GLubyte *)p)++;
-        dst->fog = *((GLubyte *)p)++;
+        dst->specular[2] = ((GLubyte *)p)[0];
+        dst->specular[1] = ((GLubyte *)p)[1];
+        dst->specular[0] = ((GLubyte *)p)[2];
+        dst->fog = ((GLubyte *)p)[3];
+        p++;
 
       case TINY_VERTEX_FORMAT:
         dst->win[2] = UNVIEWPORT_Z( LE32_IN( p++ ) );
 
-        dst->color[2] = *((GLubyte *)p)++;
-        dst->color[1] = *((GLubyte *)p)++;
-        dst->color[0] = *((GLubyte *)p)++;
-        dst->color[3] = *((GLubyte *)p)++;
+        dst->color[2] = ((GLubyte *)p)[0];
+        dst->color[1] = ((GLubyte *)p)[1];
+        dst->color[0] = ((GLubyte *)p)[2];
+        dst->color[3] = ((GLubyte *)p)[3];
+        p++;
         
         {
            GLuint xy = LE32_IN( p );
@@ -149,11 +151,11 @@ void TAG(print_vertex)( GLcontext *ctx, const VERTEX *v )
         {
            GLubyte r, g, b, a;
            
-           b = *((GLubyte *)p)++;
-           g = *((GLubyte *)p)++;
-           r = *((GLubyte *)p)++;
-           a = *((GLubyte *)p)++;
-
+           b = ((GLubyte *)p)[0];
+           g = ((GLubyte *)p)[1];
+           r = ((GLubyte *)p)[2];
+           a = ((GLubyte *)p)[3];
+           p++;
            fprintf(stderr, "spec: r %d g %d b %d a %d\n", r, g, b, a);
         }
 
@@ -165,11 +167,11 @@ void TAG(print_vertex)( GLcontext *ctx, const VERTEX *v )
            
            z = LE32_IN( p++ ) / 65536.0;
 
-           b = *((GLubyte *)p)++;
-           g = *((GLubyte *)p)++;
-           r = *((GLubyte *)p)++;
-           a = *((GLubyte *)p)++;
-        
+           b = ((GLubyte *)p)[0];
+           g = ((GLubyte *)p)[1];
+           r = ((GLubyte *)p)[2];
+           a = ((GLubyte *)p)[3];
+           p++;
            xy = LE32_IN( p );
            x = (GLfloat)(GLshort)( xy >> 16 ) / 4.0;
            y = (GLfloat)(GLshort)( xy & 0xffff ) / 4.0;

diff --git a/src/mesa/drivers/dri/mach64/mach64_native_vbtmp.h b/src/mesa/drivers/dri/mach64/mach64_native_vbtmp.h
index 623e1eae3b46030a5a8e6ca7aa3e058696d71d82..f64b808ee755ea9e6af62511c4d1540fec775071 100644 (file)
--- a/src/mesa/drivers/dri/mach64/mach64_native_vbtmp.h
+++ b/src/mesa/drivers/dri/mach64/mach64_native_vbtmp.h
@@ -274,14 +274,11 @@ static void TAG(emit)( GLcontext *ctx,
 #endif
 
 #if DO_RGBA
-        UNCLAMPED_FLOAT_TO_UBYTE(*((GLubyte *)p), col[0][2]);
-        *((GLubyte *)p)++;
-        UNCLAMPED_FLOAT_TO_UBYTE(*((GLubyte *)p), col[0][1]);
-        *((GLubyte *)p)++;
-        UNCLAMPED_FLOAT_TO_UBYTE(*((GLubyte *)p), col[0][0]);
-        *((GLubyte *)p)++;
-        UNCLAMPED_FLOAT_TO_UBYTE(*((GLubyte *)p), col[0][3]);
-        *((GLubyte *)p)++;
+        UNCLAMPED_FLOAT_TO_UBYTE(((GLubyte *)p)[0], col[0][2]);
+        UNCLAMPED_FLOAT_TO_UBYTE(((GLubyte *)p)[1], col[0][1]);
+        UNCLAMPED_FLOAT_TO_UBYTE(((GLubyte *)p)[2], col[0][0]);
+        UNCLAMPED_FLOAT_TO_UBYTE(((GLubyte *)p)[3], col[0][3]);
+        p++;
         STRIDE_4F(col, col_stride);
 #else
         p++;
@@ -473,45 +470,25 @@ static void TAG(interp)( GLcontext *ctx,
 #endif /* !DO_TEX0 */
    
 #if DO_SPEC
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_SPEC_B */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_SPEC_G */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-   
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_SPEC_R */
-#if DO_FOG
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-#else /* !DO_FOG */
-   ((GLubyte *)dst) += 2; ((GLubyte *)out) += 2; ((GLubyte *)in) += 2;
-#endif /* !DO_FOG */
-#elif DO_FOG
-   ((GLubyte *)dst) += 3; ((GLubyte *)out) += 3; ((GLubyte *)in) += 3;
+   INTERP_UB( t, ((GLubyte *)dst)[0], ((GLubyte *)out)[0], ((GLubyte *)in)[0] );       /* VERTEX_?_SPEC_B */
+   INTERP_UB( t, ((GLubyte *)dst)[1], ((GLubyte *)out)[1], ((GLubyte *)in)[1] );       /* VERTEX_?_SPEC_G */
+   INTERP_UB( t, ((GLubyte *)dst)[2], ((GLubyte *)out)[2], ((GLubyte *)in)[2] );       /* VERTEX_?_SPEC_R */
 #endif
    
 #if DO_FOG
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_SPEC_A */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
+   INTERP_UB( t, ((GLubyte *)dst)[3], ((GLubyte *)out)[3], ((GLubyte *)in)[3] );       /* VERTEX_?_SPEC_A */
 #endif /* DO_FOG */
 
-#if !DO_SPEC && !DO_FOG
    dst++; out++; in++;
-#endif
 
    LE32_OUT( dst, VIEWPORT_Z( dstclip[2] * w ) );              /* VERTEX_?_Z */
    dst++; out++; in++;
   
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_B */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_G */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_R */
-   ((GLubyte *)dst)++; ((GLubyte *)out)++; ((GLubyte *)in)++;
-
-   INTERP_UB( t, *(GLubyte *)dst, *(GLubyte *)out, *(GLubyte *)in );   /* VERTEX_?_A */
-   ((GLubyte *)dst)++; /* ((GLubyte *)out)++; ((GLubyte *)in)++; */
+   INTERP_UB( t, ((GLubyte *)dst)[0], ((GLubyte *)out)[0], ((GLubyte *)in)[0] );       /* VERTEX_?_B */
+   INTERP_UB( t, ((GLubyte *)dst)[1], ((GLubyte *)out)[1], ((GLubyte *)in)[1] );       /* VERTEX_?_G */
+   INTERP_UB( t, ((GLubyte *)dst)[2], ((GLubyte *)out)[2], ((GLubyte *)in)[2] );       /* VERTEX_?_R */
+   INTERP_UB( t, ((GLubyte *)dst)[3], ((GLubyte *)out)[3], ((GLubyte *)in)[3] );       /* VERTEX_?_A */
+   dst++; /*out++; in++;*/
 
    LE32_OUT( dst,
             (VIEWPORT_X( dstclip[0] * w ) << 16) |             /* VERTEX_?_X */