Re: [Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Kenneth Graunke
On 01/01/2013 05:06 PM, Vinson Lee wrote: Fixes missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee --- 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/

Re: [Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Paul Berry
This looks correct to me. Reviewed-by: Paul Berry On 1 January 2013 17:06, Vinson Lee wrote: > Fixes missing break in switch defect reported by Coverity. > > Signed-off-by: Vinson Lee > --- > src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sr

[Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Vinson Lee
Fixes missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee --- 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 --