Re: [Mesa-dev] minor issues with GL_CLIP_DISTANCEx in enable.c, get.c

2011-09-24 Thread Paul Berry
On 24 September 2011 15:44, Brian Paul wrote: > Hi Paul, > > In enable.c: > > 1. At line 307, s/GL_CLIP_PLANE0/GL_CLIP_DISTANCE0/ to be consistent. > > 2. Need to check that p < ctx->Const.MaxClipPlanes. > > 3. In _mesa_IsEnabled(), need to s/GL_CLIP_PLANE/GL_CLIP_DISTANCE/, > add the two extra s

[Mesa-dev] minor issues with GL_CLIP_DISTANCEx in enable.c, get.c

2011-09-24 Thread Brian Paul
Hi Paul, In enable.c: 1. At line 307, s/GL_CLIP_PLANE0/GL_CLIP_DISTANCE0/ to be consistent. 2. Need to check that p < ctx->Const.MaxClipPlanes. 3. In _mesa_IsEnabled(), need to s/GL_CLIP_PLANE/GL_CLIP_DISTANCE/, add the two extra switch cases for planes 6,7 and check the plane index against ctx