Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: Indent break statements and add a missing one.

2015-03-03 Thread Matt Turner
On Tue, Mar 3, 2015 at 6:01 PM, Emil Velikov wrote: > Seems like you've added 10.4 to the list before pushing. Upon closer > looks it seems that this addresses Iago and others' formats rework. > The latter of which never made it in 10.4. Can you confirm if we want > this for 10.4 ? Ah, nope, the

Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: Indent break statements and add a missing one.

2015-03-03 Thread Emil Velikov
On 28 February 2015 at 19:02, Matt Turner wrote: > Always indenting break statements makes spotting missing ones easier. > > Cc: 10.5 > --- > src/mesa/main/pack.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/src/mesa/main/pack.c b/src/mesa/main/pack.c > in

Re: [Mesa-dev] [Mesa-stable] [PATCH] mesa: Indent break statements and add a missing one.

2015-02-28 Thread Brian Paul
Reviewed-by: Brian Paul On 02/28/2015 12:02 PM, Matt Turner wrote: Always indenting break statements makes spotting missing ones easier. Cc: 10.5 --- src/mesa/main/pack.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/mesa/main/pack.c b/src/mesa/main/pa