Re: [Mesa-dev] [PATCH] i965: Add missing braces around if-statement.

2015-06-18 Thread Kenneth Graunke
On Thursday, June 18, 2015 04:28:25 PM Ben Widawsky wrote: > > On Thu, Jun 18, 2015 at 04:19:36PM -0700, Matt Turner wrote: > > Fixes a performance problem caused by commit b639ed2f. > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90895 > > Ken spotted this in review. > /me hides

Re: [Mesa-dev] [PATCH] i965: Add missing braces around if-statement.

2015-06-18 Thread Ben Widawsky
On Thu, Jun 18, 2015 at 04:19:36PM -0700, Matt Turner wrote: > Fixes a performance problem caused by commit b639ed2f. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90895 Ken spotted this in review. /me hides Reviewed-by: Ben Widawsky > --- > src/mesa/drivers/dri/i965/brw_meta_fa

Re: [Mesa-dev] [PATCH] i965: Add missing braces around if-statement.

2015-06-18 Thread Jason Ekstrand
Wow... Reviewed-by: Jason Ekstrand On Thu, Jun 18, 2015 at 4:19 PM, Matt Turner wrote: > Fixes a performance problem caused by commit b639ed2f. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90895 > --- > src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 3 ++- > 1 file changed, 2 i

Re: [Mesa-dev] [PATCH] i965: Add missing braces around if-statement.

2015-06-18 Thread Chris Forbes
Oh, how silly :) Reviewed-by: Chris Forbes - Chris On Fri, Jun 19, 2015 at 11:19 AM, Matt Turner wrote: > Fixes a performance problem caused by commit b639ed2f. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90895 > --- > src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 3 ++- >

[Mesa-dev] [PATCH] i965: Add missing braces around if-statement.

2015-06-18 Thread Matt Turner
Fixes a performance problem caused by commit b639ed2f. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90895 --- src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c b/src/mesa