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

[Paul Berry]

On 24 September 2011 15:44, Brian Paul <brian.e.paul at gmail.com> 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 switch cases for planes 6,7 and check the plane
> index against ctx->Const.MaxClipPlanes.
>
> in get.c:
>
> grep for GL_CLIP_PLANE and make similar changes.
>
> Thanks.
>
> -Brian
>

Thanks, Brian.  These are all things I should have thought of.  I'll send
out a patch to take care of this next week.

Paul
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20110924/38887793/attachment.html>