On 22 February 2012 14:22, Ian Romanick <span dir="ltr">&lt;<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</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">
<div class="im">On 02/22/2012 02:17 PM, Paul Berry wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On 22 February 2012 13:52, Ian Romanick &lt;<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a><br></div><div class="im">
&lt;mailto:<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>&gt;&gt; wrote:<br>
<br>
    On 02/22/2012 01:41 PM, Paul Berry wrote:<br>
<br>
          From<br></div>
        <a href="http://www.opengl.org/__registry/specs/ARB/seamless___cube_map.txt" target="_blank">http://www.opengl.org/__<u></u>registry/specs/ARB/seamless___<u></u>cube_map.txt</a><div class="im"><br>
        &lt;<a href="http://www.opengl.org/registry/specs/ARB/seamless_cube_map.txt" target="_blank">http://www.opengl.org/<u></u>registry/specs/ARB/seamless_<u></u>cube_map.txt</a>&gt;:<br>
<br>
             Accepted by the&lt;cap&gt;  parameter of Enable, Disable and<br>
        IsEnabled,<br>
             and by the&lt;pname&gt;  parameter of GetBooleanv, GetIntegerv,<br>
        GetFloatv<br>
             and GetDoublev:<br>
<br>
             TEXTURE_CUBE_MAP_SEAMLESS                   0x884F<br>
<br>
<br>
    Oops.  That was my typo.  You&#39;ll also have to regenerate the various<br>
    files that depend on the XML definitions.  I think this change<br>
    should only cause changes in src/mesa/main/enums.c.<br>
<br>
    Reviewed-by: Ian Romanick &lt;<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a><br></div>
    &lt;mailto:<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.<u></u>com</a>&gt;&gt;<div class="im"><br>
<br>
<br>
Oops.  I didn&#39;t realize that those files weren&#39;t built automatically.<br>
I&#39;ll send an updated patch.<br>
</div></blockquote>
<br>
Dear god, no!  The patch will be huge.</blockquote><div><br>Actually, it&#39;s pretty tiny since it didn&#39;t change any enum names (and hence didn&#39;t move around the offsets in all_enums).  But I can see why the patch would be huge in most cases.  All the same, the diff is not very informative, so I will spare you having to look at it :)<br>
</div></div>