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

Chris Forbes chrisf at ijw.co.nz
Thu Jun 18 16:27:27 PDT 2015


Oh, how silly :)

Reviewed-by: Chris Forbes <chrisf at ijw.co.nz>

- Chris

On Fri, Jun 19, 2015 at 11:19 AM, Matt Turner <mattst88 at gmail.com> 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 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c b/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c
> index c0c8dfa..49f2e3e 100644
> --- a/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c
> +++ b/src/mesa/drivers/dri/i965/brw_meta_fast_clear.c
> @@ -339,12 +339,13 @@ is_color_fast_clear_compatible(struct brw_context *brw,
>                                 mesa_format format,
>                                 const union gl_color_union *color)
>  {
> -   if (_mesa_is_format_integer_color(format))
> +   if (_mesa_is_format_integer_color(format)) {
>        if (brw->gen >= 8) {
>           perf_debug("Integer fast clear not enabled for (%s)",
>                      _mesa_get_format_name(format));
>        }
>        return false;
> +   }
>
>     for (int i = 0; i < 4; i++) {
>        if (color->f[i] != 0.0 && color->f[i] != 1.0 &&
> --
> 2.3.6
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list