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

Reply via email to