On 20 August 2013 11:30, Paul Berry <stereotype...@gmail.com> wrote:

> ---
>  src/mesa/drivers/dri/i965/brw_program.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_program.c
> b/src/mesa/drivers/dri/i965/brw_program.c
> index fdb2848..c40d506 100644
> --- a/src/mesa/drivers/dri/i965/brw_program.c
> +++ b/src/mesa/drivers/dri/i965/brw_program.c
> @@ -102,6 +102,17 @@ static struct gl_program *brwNewProgram( struct
> gl_context *ctx,
>          return NULL;
>     }
>
> +   case MESA_GEOMETRY_PROGRAM: {
> +      struct brw_geometry_program *prog =
> CALLOC_STRUCT(brw_geometry_program);
> +      if (prog) {
> +         prog->id = get_new_program_id(brw->intelScreen);
> +
> +         return _mesa_init_geometry_program(ctx, &prog->program, target,
> id);
> +      } else {
> +         return NULL;
> +      }
> +   }
> +
>     default:
>        return _mesa_new_program(ctx, target, id);
>     }
>

>From our in-person code review yesterday:

As a follow up patch, make the default case assert.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to