diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2012-06-04 12:08:11 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2012-06-04 12:08:11 +0100 |
| commit | 99129c369c16d9ff7588a0e59f9285cc339b7eb3 (patch) | |
| tree | 444af9d1f514b2972ed6719cd4d924f03dba3889 /src/sna/gen6_render.c | |
| parent | fae9c054d47203b68a6c09647945d23074ea4df9 (diff) | |
sna/gen4+: Use an explicit invalid value for alu
Since the static analyser also doesn't like comparing a uint8_t against
~0.
Reported-by: Zdenek Kabelac <zkabelac@redhat.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'src/sna/gen6_render.c')
| -rw-r--r-- | src/sna/gen6_render.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/sna/gen6_render.c b/src/sna/gen6_render.c index cc6cc508..87a2c9c2 100644 --- a/src/sna/gen6_render.c +++ b/src/sna/gen6_render.c @@ -3661,7 +3661,7 @@ gen6_render_fill_boxes(struct sna *sna, (prefer_blt_fill(sna, dst_bo) || too_large(dst->drawable.width, dst->drawable.height) || !gen6_check_dst_format(format))) { - uint8_t alu = ~0; + uint8_t alu = GXinvalid; pixel = 0; if (op == PictOpClear) @@ -3674,7 +3674,7 @@ gen6_render_fill_boxes(struct sna *sna, format)) alu = GXcopy; - if (alu != ~0 && + if (alu != GXinvalid && sna_blt_fill_boxes(sna, alu, dst_bo, dst->drawable.bitsPerPixel, pixel, box, n)) |