[Mesa-dev] [PATCH] i965: Don't try to fall back when creating unrecognized program targets.

Anuj Phogat anuj.phogat at gmail.com
Fri Aug 23 12:33:31 PDT 2013


On Fri, Aug 23, 2013 at 11:50 AM, Paul Berry <stereotype441 at gmail.com> wrote:
> If brwNewProgram is asked to create a program for an unrecognized
> target, don't bother falling back on _mesa_new_program().  That just
> hides bugs.
> ---
>  src/mesa/drivers/dri/i965/brw_program.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_program.c b/src/mesa/drivers/dri/i965/brw_program.c
> index c40d506..b683681 100644
> --- a/src/mesa/drivers/dri/i965/brw_program.c
> +++ b/src/mesa/drivers/dri/i965/brw_program.c
> @@ -114,7 +114,8 @@ static struct gl_program *brwNewProgram( struct gl_context *ctx,
>     }
>
>     default:
> -      return _mesa_new_program(ctx, target, id);
> +      _mesa_problem(ctx, "Unsupported target %d in brwNewProgram()\n", target);
> +      return NULL;
>     }
>  }
>
> --
> 1.8.3.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>


More information about the mesa-dev mailing list