On 9 January 2014 20:03, Chris Forbes <[email protected]> wrote:
> This is a slightly odd construction (although copied from the existing
> code):
>
> > + if ((shProg == NULL) || (shProg->_LinkedShaders[stage] == NULL))
> > + shProg = NULL;
>
You're right. This would be much better as:
if ((shProg != NULL) && (shProg->_LinkedShaders[stage] == NULL))
shProg = NULL;
I'll update it in a follow-up patch.
_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
