On Mon, Jan 7, 2013 at 1:08 PM, Jorge Ramirez Ortiz,  HCL Europe
<jorge.ramirez-or...@hcl.com> wrote:
> You are absolutely wrong. NULL is a valid return value. Please read the code.

I'm not sure if you missed Ian's point or you're just being pedantic,
but let me try to explain. The glxinfo code does this:

if (glXMakeCurrent(dpy, win, ctx)) {
    ...
    const char *glVersion = (const char *) glGetString(GL_VERSION);

For glGetString(GL_VERSION) to be called, you have a context. So,
you've got a context and you're supplying a proper argument. It cannot
return NULL. As a result, glxinfo does not need to check for NULL.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to