On 5 March 2018 at 16:53, Brian Paul <bri...@vmware.com> wrote: > Print GL_MAX_CONVOLUTION_WIDTH, HEIGHT on separate lines. > Print GL_MAX_COLOR_MATRIX_STACK_DEPTH in same section. > --- > src/xdemos/glinfo_common.c | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/src/xdemos/glinfo_common.c b/src/xdemos/glinfo_common.c > index f84d1a4..508fce1 100644 > --- a/src/xdemos/glinfo_common.c > +++ b/src/xdemos/glinfo_common.c > @@ -464,7 +464,6 @@ print_limits(const char *extensions, const char > *oglstring, int version, > { 1, GL_MAX_ATTRIB_STACK_DEPTH, "GL_MAX_ATTRIB_STACK_DEPTH", NULL }, > { 1, GL_MAX_CLIENT_ATTRIB_STACK_DEPTH, > "GL_MAX_CLIENT_ATTRIB_STACK_DEPTH", NULL }, > { 1, GL_MAX_CLIP_PLANES, "GL_MAX_CLIP_PLANES", NULL }, > - { 1, GL_MAX_COLOR_MATRIX_STACK_DEPTH, > "GL_MAX_COLOR_MATRIX_STACK_DEPTH", "GL_ARB_imaging" }, > { 1, GL_MAX_ELEMENTS_VERTICES, "GL_MAX_ELEMENTS_VERTICES", NULL }, > { 1, GL_MAX_ELEMENTS_INDICES, "GL_MAX_ELEMENTS_INDICES", NULL }, > { 1, GL_MAX_EVAL_ORDER, "GL_MAX_EVAL_ORDER", NULL }, > @@ -716,12 +715,16 @@ print_limits(const char *extensions, const char > *oglstring, int version, > > /* these don't fit into the above mechanism, unfortunately */ > if (extension_supported("GL_ARB_imaging", extensions)) { > + GLint d; > + printf(" GL_ARB_imaging:\n");
With the small suggestions, the series is Reviewed-by: Emil Velikov <emil.veli...@collabora.com> Aside: Mesa is not exposing the extension, even though it seems implemented. A dummy_true entry in src/mesa/main/extensions_table.h should be all that's needed. -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev