Fix warnings on linux build (strict aliasing, cons init order, vars used only in...
authorbsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 3 Feb 2011 21:30:17 +0000 (21:30 +0000)
committerbsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 3 Feb 2011 21:30:17 +0000 (21:30 +0000)
git-svn-id: http://skia.googlecode.com/svn/trunk@756 2bbb7eff-a529-9590-31e7-b0007b416f81

gpu/src/GrDrawTarget.cpp
gpu/src/GrGLUtil.cpp
src/gpu/SkGpuDevice.cpp

index 96afe2f..0f31e9a 100644 (file)
@@ -243,13 +243,17 @@ void GrDrawTarget::VertexLayoutUnitTest() {
                     GrAssert(!VertexUsesStage(s2, tcMask));
                     GrAssert(-1 == VertexTexCoordsForStage(s2, tcMask));
 
+                #if GR_DEBUG
                     GrVertexLayout posAsTex = tcMask | StagePosAsTexCoordVertexLayoutBit(s2);
+                #endif
                     GrAssert(0 == VertexStageCoordOffset(s2, posAsTex));
                     GrAssert(VertexUsesStage(s2, posAsTex));
                     GrAssert(2*sizeof(GrPoint) == VertexSize(posAsTex));
                     GrAssert(-1 == VertexTexCoordsForStage(s2, posAsTex));
                 }
+            #if GR_DEBUG
                 GrVertexLayout withColor = tcMask | kColor_VertexLayoutBit;
+            #endif
                 GrAssert(2*sizeof(GrPoint) == VertexColorOffset(withColor));
                 GrAssert(2*sizeof(GrPoint) + sizeof(GrColor) == VertexSize(withColor));
             }
index 360e84e..a8f06ba 100644 (file)
@@ -15,6 +15,7 @@
  */\r
 \r
 #include "GrGLConfig.h"\r
+#include "GrTypes.h"\r
 #include <stdio.h>\r
 \r
 bool has_gl_extension(const char* ext) {\r
@@ -74,11 +75,11 @@ void gl_version(int* major, int* minor) {
 typedef void (*glProc)(void);\r
 \r
 #define GET_PROC(EXT_STRUCT, PROC_NAME) \\r
-    *((glProc*) &(EXT_STRUCT-> PROC_NAME)) = (glProc) GR_GL_PROC_ADDRESS((gl ## PROC_NAME)); \\r
+    *(GrTCast<glProc*>(&(EXT_STRUCT-> PROC_NAME))) = (glProc)GR_GL_PROC_ADDRESS((gl ## PROC_NAME)); \\r
     GrAssert(NULL != EXT_STRUCT-> PROC_NAME)\r
 \r
 #define GET_SUFFIX_PROC(EXT_STRUCT, PROC_NAME, SUFFIX) \\r
-    *((glProc*) &(EXT_STRUCT-> PROC_NAME)) = (glProc) GR_GL_PROC_ADDRESS((gl ## PROC_NAME ## SUFFIX)); \\r
+    *(GrTCast<glProc*>(&(EXT_STRUCT-> PROC_NAME))) = (glProc)GR_GL_PROC_ADDRESS((gl ## PROC_NAME ## SUFFIX)); \\r
     GrAssert(NULL != EXT_STRUCT-> PROC_NAME)\r
 \r
 extern void GrGLInitExtensions(GrGLExts* exts) {\r
index 5059569..cc7cb7b 100644 (file)
@@ -465,8 +465,8 @@ public:
         dstPosition->fY = SkScalarToGrScalar(fPoints[i].fY);
     }
 private:
-    int             fCount;
     const SkPoint*  fPoints;
+    int             fCount;
 };
 
 class SkTexCoordSource {
@@ -506,8 +506,8 @@ public:
     }
 
 private:
-    int             fCount;
     const uint16_t* fIndices;
+    int             fCount;
 };
 
 ///////////////////////////////////////////////////////////////////////////////