[Mesa-dev] [PATCH 2/2] gallium/util: fix up inaccurate behavior of util_framebuffer_state_equal
Ilia Mirkin
imirkin at alum.mit.edu
Sun Mar 27 22:40:03 UTC 2016
Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
I would personally move the for loop down below the "simple" checks.
Purely optional.
On Sun, Mar 27, 2016 at 2:25 PM, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> ---
> src/gallium/auxiliary/util/u_framebuffer.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/util/u_framebuffer.c b/src/gallium/auxiliary/util/u_framebuffer.c
> index 2e0ef74..3798d9b 100644
> --- a/src/gallium/auxiliary/util/u_framebuffer.c
> +++ b/src/gallium/auxiliary/util/u_framebuffer.c
> @@ -55,7 +55,7 @@ util_framebuffer_state_equal(const struct pipe_framebuffer_state *dst,
> dst->height != src->height)
> return FALSE;
>
> - for (i = 0; i < Elements(src->cbufs); i++) {
> + for (i = 0; i < src->nr_cbufs; i++) {
> if (dst->cbufs[i] != src->cbufs[i]) {
> return FALSE;
> }
> --
> 2.5.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list