<div dir="ltr">Hi Emil,<div><br></div><div>Do you mean adding the cast only if __APPLE__ is defined ?</div><div><br></div><div>Actually when reading comment #2 we should get the build error for <span style="color:rgb(0,0,0);white-space:pre-wrap">src/mesa/main/uniform_query.cpp too but it is not the case anymore. So with git blame I can see that the build error has been fixed by commit </span>d96ed5c0 for the file <span style="color:rgb(0,0,0);white-space:pre-wrap">uniform_query.cpp only.</span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap">But can we apply the same for </span>_mesa_BindAttribLocation in shader_query.cpp ? I can see in the spec that non-ARB version of glBindAttribLocation 's first param is GLuint  <a href="https://www.opengl.org/sdk/docs/man3/xhtml/glBindAttribLocation.xml">https://www.opengl.org/sdk/docs/man3/xhtml/glBindAttribLocation.xml</a> </div><div>(gles2: <a href="https://www.khronos.org/opengles/sdk/docs/man/xhtml/glBindAttribLocation.xml">https://www.khronos.org/opengles/sdk/docs/man/xhtml/glBindAttribLocation.xml</a> )<br></div><div><br></div><div>What is the purpose of src/mapi/glapi/gen/ARB_robustness.xml ?</div>































<div><br></div><div>Thx</div><div>Julien<br><div class="gmail_extra"><br><div class="gmail_quote">On 18 June 2015 at 19:16, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On 18 June 2015 at 06:53, Julien Isorce <<a href="mailto:julien.isorce@gmail.com">julien.isorce@gmail.com</a>> wrote:<br>
> From: Jon TURNEY <<a href="mailto:jon.turney@dronecode.org.uk">jon.turney@dronecode.org.uk</a>><br>
><br>
> On darwin, GLhandleARB is defined as a void *, not the unsigned int it is on<br>
> linux.<br>
><br>
> For the moment, apply a cast to supress the warning<br>
><br>
> Possibly this is safe, as for the mesa software renderer the shader program<br>
> handle is not a real pointer, but a integer handle<br>
><br>
> Probably this is not the right thing to do, and we should pay closer attention<br>
> to how the GLhandlerARB type is used.<br>
><br>
> Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=66346" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=66346</a><br>
</span>Imho one's goal should not be to find the quickest solution to shut<br>
warnings up, but to understand them.<br>
As mentioned by Jeremy Huddleston in comment 11, this approach only<br>
masks the issue.<br>
<br>
There was a lengthy discussion(s) on the topic a while back and one of<br>
the objections on the proposed fixes (from Ian iirc) was that it will<br>
break the libglapi ABI. As Jeremy did not see that as a concern<br>
perhaps we can revive them and make sure that the ABI change does not<br>
happen in the non-Mac world.<br>
<span class=""><font color="#888888"><br>
-Emil<br>
</font></span></blockquote></div><br></div></div></div>