[Vm-dev] [commit][3004] Fix some compilation warnings in the Squeak3D plugin ( thanks Nicolas!).

[commits at squeakvm.org]

Revision: 3004
Author:   eliot
Date:     2014-06-16 13:38:32 -0700 (Mon, 16 Jun 2014)
Log Message:
-----------
Fix some compilation warnings in the Squeak3D plugin (thanks Nicolas!).

Modified Paths:
--------------
    trunk/platforms/Cross/plugins/Squeak3D/b3dInit.c

Modified: trunk/platforms/Cross/plugins/Squeak3D/b3dInit.c
===================================================================
--- trunk/platforms/Cross/plugins/Squeak3D/b3dInit.c	2014-06-15 22:17:31 UTC (rev 3003)
+++ trunk/platforms/Cross/plugins/Squeak3D/b3dInit.c	2014-06-16 20:38:32 UTC (rev 3004)
@@ -6,7 +6,7 @@
 *   AUTHOR:  Andreas Raab (ar)
 *   ADDRESS: Walt Disney Imagineering, Glendale, CA
 *   EMAIL:   andreasr at wdi.disney.com
-*   RCSID:   $Id: b3dInit.c,v 1.1 2001/10/24 23:12:23 rowledge Exp $
+*   RCSID:   $Id$
 *
 *   NOTES:
 *
@@ -36,7 +36,7 @@
 int b3dInitializeEdgeAllocator(void* base, int length)
 {
 	B3DEdgeAllocList *list = (B3DEdgeAllocList*) base;
-	if(length < sizeof(B3DEdgeAllocList))
+	if(length<0 || (unsigned)length < sizeof(B3DEdgeAllocList))
 		return B3D_GENERIC_ERROR;
 	list->magic = B3D_EDGE_ALLOC_MAGIC;
 	list->This = base;
@@ -50,7 +50,7 @@
 int b3dInitializeFaceAllocator(void* base, int length)
 {
 	B3DFaceAllocList *list = (B3DFaceAllocList*) base;
-	if(length < sizeof(B3DFaceAllocList))
+	if(length<0 || (unsigned)length < sizeof(B3DFaceAllocList))
 		return B3D_GENERIC_ERROR;
 	list->magic = B3D_FACE_ALLOC_MAGIC;
 	list->This = base;
@@ -64,7 +64,7 @@
 int b3dInitializeAttrAllocator(void* base, int length)
 {
 	B3DAttrAllocList *list = (B3DAttrAllocList*) base;
-	if(length < sizeof(B3DAttrAllocList))
+	if(length<0 || (unsigned)length < sizeof(B3DAttrAllocList))
 		return B3D_GENERIC_ERROR;
 	list->magic = B3D_ATTR_ALLOC_MAGIC;
 	list->This = base;
@@ -78,7 +78,7 @@
 int b3dInitializeEdgeList(void* base, int length)
 {
 	B3DPrimitiveEdgeList *list = (B3DPrimitiveEdgeList*) base;
-	if(length < sizeof(B3DPrimitiveEdgeList))
+	if(length<0 || (unsigned)length < sizeof(B3DPrimitiveEdgeList))
 		return B3D_GENERIC_ERROR;
 	list->magic = B3D_EDGE_LIST_MAGIC;
 	list->This = base;
@@ -90,7 +90,7 @@
 int b3dInitializeAET(void* base, int length)
 {
 	B3DActiveEdgeTable *aet = (B3DActiveEdgeTable *) base;
-	if(length < sizeof(B3DActiveEdgeTable))
+	if(length<0 || (unsigned)length < sizeof(B3DActiveEdgeTable))
 		return B3D_GENERIC_ERROR;
 	aet->magic = B3D_AET_MAGIC;
 	aet->This = base;
@@ -105,7 +105,7 @@
 int b3dInitializeFillList(void* base, int length)
 {
 	B3DFillList *list = (B3DFillList*) base;
-	if(length < sizeof(B3DFillList))
+	if(length<0 || (unsigned)length < sizeof(B3DFillList))
 		return B3D_GENERIC_ERROR;
 	list->magic = B3D_FILL_LIST_MAGIC;
 	list->This = base;
@@ -550,7 +550,7 @@
 	obj->nFaces = nQuads * 2;
 	obj->faces = (B3DInputFace*) (obj->vertices + obj->nVertices);
 	{
-		int i, nFaces = obj->nFaces;
+		int i;
 		B3DInputQuad *src = quadPtr;
 		B3DInputFace *dst = obj->faces;