On 4 November 2012 09:55, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><div class="gmail_extra"><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">Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> writes:<br>
<br>
> On 11/02/2012 05:13 PM, Ian Romanick wrote:<br>
>> On 11/02/2012 03:01 PM, Paul Berry wrote:<br>
>>> On 2 November 2012 14:11, Ian Romanick <<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a><br>
</div><div class="im">>>>     Oh yuck!  Why not just fix the names of the Mesa functions?  That<br>
>>>     seems much better than carrying the work-around in the code<br>
>>>     generator forever.<br>
>>><br>
>>><br>
>>> The reason I shied away from this approach is that there are a lot of<br>
>>> functions to fix (146 of them by my count), and I was worried it would<br>
>>> take a very long time to rename all references to them.<br>
>><br>
>> That many?  Ugh.  Is it possible to switch the aliasing order?  For the<br>
>> IsProgram case, IsProgramNV would alias IsProgramARB (instead of the<br>
>> other way around).  Is that better or worse?<br>
>><br>
>>> But I didn't actually try it.  I'll try spending a few hours on it and<br>
>>> see how far I get.<br>
><br>
> FWIW, I would like to see Mesa use the core names.  IsProgramNV is<br>
> particularly odd because we don't support the extension it came<br>
> from...the name IsProgramARB makes a lot more sense.<br>
<br>
</div>Seconding this.<br></blockquote><div><br>Ok, by popular demand I'm doing a patch to make all the naming consistent.  It actually wasn't that difficult--just a lot of scripting.  To avoid a rebasing nightmare, I'm going to let this patch series land as is, and then send out a patch for review that does the renaming.  I should be able to get that out by the end of the day.<br>
<br>That means we'll have to leave the "mesa_name" ugliness in for now, but we'll be able to remove it again in a few days.<br></div></div></div>