On 24 September 2011 15:44, Brian Paul <span dir="ltr">&lt;<a href="mailto:brian.e.paul@gmail.com">brian.e.paul@gmail.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Paul,<br>
<br>
In enable.c:<br>
<br>
1. At line 307, s/GL_CLIP_PLANE0/GL_CLIP_DISTANCE0/ to be consistent.<br>
<br>
2. Need to check that p &lt; ctx-&gt;Const.MaxClipPlanes.<br>
<br>
3. In _mesa_IsEnabled(), need to s/GL_CLIP_PLANE/GL_CLIP_DISTANCE/,<br>
add the two extra switch cases for planes 6,7 and check the plane<br>
index against ctx-&gt;Const.MaxClipPlanes.<br>
<br>
in get.c:<br>
<br>
grep for GL_CLIP_PLANE and make similar changes.<br>
<br>
Thanks.<br>
<font color="#888888"><br>
-Brian<br>
</font></blockquote></div><br>Thanks, Brian.  These are all things I should have thought of.  I&#39;ll send out a patch to take care of this next week.<br><br>Paul<br>