Fixed various warnings which were appearing under MacOSX.
This commit is contained in:
@@ -45,9 +45,6 @@ void Geometry::drawImmediateMode(State& state)
|
||||
if (!_vertexArray.valid()) return;
|
||||
|
||||
// set up the vertex arrays.
|
||||
// glEnableClientState( GL_VERTEX_ARRAY );
|
||||
// glVertexPointer(3,GL_FLOAT,0,_vertexArray->dataPointer());
|
||||
|
||||
state.setVertexPointer(3,GL_FLOAT,0,_vertexArray->dataPointer());
|
||||
|
||||
// set up texture coordinates.
|
||||
@@ -55,16 +52,9 @@ void Geometry::drawImmediateMode(State& state)
|
||||
{
|
||||
Array* array = _texCoordList[i].get();
|
||||
if (array)
|
||||
{
|
||||
// glEnableClientState( GL_TEXTURE_COORD_ARRAY );
|
||||
// glTexCoordPointer(array->dataSize(),array->dataType(),0,array->dataPointer());
|
||||
state.setTexCoordPointer(i,array->dataSize(),array->dataType(),0,array->dataPointer());
|
||||
}
|
||||
else
|
||||
{
|
||||
// glDisableClientState( GL_TEXTURE_COORD_ARRAY );
|
||||
state.disableTexCoordPointer(i);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -75,22 +65,18 @@ void Geometry::drawImmediateMode(State& state)
|
||||
switch (_normalBinding)
|
||||
{
|
||||
case(BIND_OFF):
|
||||
// glDisableClientState( GL_NORMAL_ARRAY );
|
||||
state.disableNormalPointer();
|
||||
break;
|
||||
case(BIND_OVERALL):
|
||||
// glDisableClientState( GL_NORMAL_ARRAY );
|
||||
state.disableNormalPointer();
|
||||
if (normalPointer) glNormal3fv(reinterpret_cast<const GLfloat*>(normalPointer));
|
||||
break;
|
||||
case(BIND_PER_PRIMITIVE):
|
||||
// glDisableClientState( GL_NORMAL_ARRAY );
|
||||
state.disableNormalPointer();
|
||||
break;
|
||||
case(BIND_PER_VERTEX):
|
||||
// glEnableClientState( GL_NORMAL_ARRAY );
|
||||
// if (normalPointer) glNormalPointer(GL_FLOAT,0,normalPointer);
|
||||
if (normalPointer) state.setNormalPointer(GL_FLOAT,0,normalPointer);
|
||||
else state.disableNormalPointer();
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -124,17 +110,17 @@ void Geometry::drawImmediateMode(State& state)
|
||||
colorStride = 16;
|
||||
break;
|
||||
}
|
||||
default:
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
switch (_colorBinding)
|
||||
{
|
||||
case(BIND_OFF):
|
||||
// glDisableClientState( GL_COLOR_ARRAY );
|
||||
state.disableColorPointer();
|
||||
break;
|
||||
case(BIND_OVERALL):
|
||||
// glDisableClientState( GL_COLOR_ARRAY );
|
||||
state.disableColorPointer();
|
||||
if (colorPointer)
|
||||
{
|
||||
@@ -149,17 +135,17 @@ void Geometry::drawImmediateMode(State& state)
|
||||
case(Array::Vec4ArrayType):
|
||||
glColor4fv(reinterpret_cast<const GLfloat*>(colorPointer));
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
}
|
||||
break;
|
||||
case(BIND_PER_PRIMITIVE):
|
||||
// glDisableClientState( GL_COLOR_ARRAY );
|
||||
state.disableColorPointer();
|
||||
break;
|
||||
case(BIND_PER_VERTEX):
|
||||
// glEnableClientState( GL_COLOR_ARRAY );
|
||||
// if (colorPointer) glColorPointer(_colorArray->dataSize(),_colorArray->dataType(),0,colorPointer);
|
||||
if (colorPointer) state.setColorPointer(_colorArray->dataSize(),_colorArray->dataType(),0,colorPointer);
|
||||
else state.disableColorPointer();
|
||||
}
|
||||
|
||||
|
||||
@@ -186,6 +172,8 @@ void Geometry::drawImmediateMode(State& state)
|
||||
case(Array::Vec4ArrayType):
|
||||
glColor4fv(reinterpret_cast<const GLfloat*>(colorPointer));
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
colorPointer += colorStride;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user