On 2 November 2012 14:09, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></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 11/01/2012 03:19 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Future patches will use this annotation when code generating<br>
_mesa_create_exec_table(), to determine which functions should be<br>
dispatched to ES-specific implementations.  exec="es" indicates that<br>
the ES-specific implementation has a name beginning with "_es_"<br>
(e.g. _es_QueryMatrixxOES), and exec="check" indicates that the<br>
ES-specific implementation has a name beginning with "_check_"<br>
(e.g. _check_GetTexGenxvOES).<br>
</blockquote>
<br></div>
All of the _check_ nonsense should just go away.  In api_exec_es1.c, every single instance of _check_Foo is just a #define to _es_Foo.<br>
<br>
All of the fixed-point functions should be renamed from _es_Foox to _mesa_Foox.  The single-precision matrix functions (e.g., _es_Orthof) should similar treatment.<br>
<br>
As far as I can tell, that just leaves _es_RenderbufferStorageEXT, _es_TexGenf, _es_TexGenfv, and _es_GetTexGenfv with mangled ES1 names.<br>
<br>
The single special-case for GL_RGB565 in _es_RenderbufferStorageEXT should just be handled in renderbuffer_storage.<br>
<br>
The TexGen functions should just have an 'if (ctx->API == API_OPENGL_ES)' block at the top the defers to the special ES1 implementation.</blockquote><div><br>So, in other words, if I'm understanding you correctly, if we rename all the _es_ and _check_ functions to _mesa_..., and do some refactoring in the case of RenderbufferStorageEXT, TexGenf, TexGenfv, and GetTexGenfv, we could get rid of the exec="es" and exec="check" flavors entirely?<br>
<br>That seems fairly reasonable.  I'll give it a shot and see how it goes.<br></div></div>