On 24 September 2011 15:44, Brian Paul <brian.e.p...@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
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to