virgl: make use of local variable
authorEric Engestrom <eric.engestrom@intel.com>
Sat, 22 Jun 2019 18:40:04 +0000 (19:40 +0100)
committerEric Engestrom <eric.engestrom@intel.com>
Wed, 31 Jul 2019 08:41:05 +0000 (09:41 +0100)
Otherwise that variable is only used in an assert() and would need an
ASSERTED to avoid the warning.

Signed-off-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/gallium/drivers/virgl/virgl_screen.c

index 35becc1bd6aa62a24bac86f7046015c67f1944b5..e1a9f4ef3855c3e157991bbd535f095e3520b910 100644 (file)
@@ -550,7 +550,7 @@ has_format_bit(struct virgl_supported_format_mask *mask,
    unsigned idx = val / 32;
    unsigned bit = val % 32;
    assert(idx < ARRAY_SIZE(mask->bitmask));
-   return (mask->bitmask[val / 32] & (1u << bit)) != 0;
+   return (mask->bitmask[idx] & (1u << bit)) != 0;
 }
 
 bool