This looks correct to me.

Reviewed-by: Paul Berry <stereotype...@gmail.com>


On 1 January 2013 17:06, Vinson Lee <v...@freedesktop.org> wrote:

> Fixes missing break in switch defect reported by Coverity.
>
> Signed-off-by: Vinson Lee <v...@freedesktop.org>
> ---
>  src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
> b/src/mesa/drivers/dri/i965/brw_eu_emit.c
> index 8a93ced..bd9c182 100644
> --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
> +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c
> @@ -2397,6 +2397,7 @@ brw_set_uip_jip(struct brw_compile *p)
>
>          assert(insn->bits3.break_cont.uip != 0);
>          assert(insn->bits3.break_cont.jip != 0);
> +        break;
>
>        case BRW_OPCODE_ENDIF:
>           if (block_end_ip == 0)
> --
> 1.8.0.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to