On Thu, Jan 10, 2013 at 4:09 PM, Eric Anholt <[email protected]> wrote:
> With the change to ban deprecated-format SNORM renderbuffers in Mesa,
> these now throw GL_INVALID_ENUM.  Catch it and skip those subtests.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59072
> ---
>  .../spec/arb_framebuffer_object/get-renderbuffer-internalformat.c |    5 
> +++++
>  1 file changed, 5 insertions(+)
>
> diff --git 
> a/tests/spec/arb_framebuffer_object/get-renderbuffer-internalformat.c 
> b/tests/spec/arb_framebuffer_object/get-renderbuffer-internalformat.c
> index 88fb65c..5dca392 100644
> --- a/tests/spec/arb_framebuffer_object/get-renderbuffer-internalformat.c
> +++ b/tests/spec/arb_framebuffer_object/get-renderbuffer-internalformat.c
> @@ -66,6 +66,11 @@ test_format(const struct format_desc *format)
>         glGenRenderbuffers(1, &rb);
>         glBindRenderbuffer(GL_RENDERBUFFER, rb);
>         glRenderbufferStorage(GL_RENDERBUFFER, format->internalformat, 1, 1);
> +       if (glGetError() == GL_INVALID_ENUM) {
> +               printf("SKIP\n");
> +               return PIGLIT_SKIP;
> +       }
> +
>         glGetRenderbufferParameteriv(GL_RENDERBUFFER,
>                                      GL_RENDERBUFFER_INTERNAL_FORMAT,
>                                      &internalformat);
> --
> 1.7.10.4

Reviewed-by: Matt Turner <[email protected]>
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to