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