[Mesa-dev] [PATCH v2 28/29] gallium/aux/util/u_surface.c: Silence warnings and remove unneeded MAYBE_UNUSED
Gert Wollny
gw.fossdev at gmail.com
Thu Nov 16 15:10:01 UTC 2017
* Explicitely convert values to int in comparison.
* Remove one MAYBE_UNUSED that is actually not needed.
Signed-off-by: Gert Wollny <gw.fossdev at gmail.com>
---
src/gallium/auxiliary/util/u_surface.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_surface.c b/src/gallium/auxiliary/util/u_surface.c
index c66f279dc2..5f07eb1cda 100644
--- a/src/gallium/auxiliary/util/u_surface.c
+++ b/src/gallium/auxiliary/util/u_surface.c
@@ -259,7 +259,7 @@ util_resource_copy_region(struct pipe_context *pipe,
struct pipe_transfer *src_trans, *dst_trans;
uint8_t *dst_map;
const uint8_t *src_map;
- MAYBE_UNUSED enum pipe_format src_format;
+ enum pipe_format src_format;
enum pipe_format dst_format;
struct pipe_box src_box, dst_box;
unsigned src_bs, dst_bs, src_bw, dst_bw, src_bh, dst_bh;
@@ -328,10 +328,10 @@ util_resource_copy_region(struct pipe_context *pipe,
assert(dst_box.y % dst_bh == 0);
/* check that region boxes are not out of bounds */
- assert(src_box.x + src_box.width <= u_minify(src->width0, src_level));
- assert(src_box.y + src_box.height <= u_minify(src->height0, src_level));
- assert(dst_box.x + dst_box.width <= u_minify(dst->width0, dst_level));
- assert(dst_box.y + dst_box.height <= u_minify(dst->height0, dst_level));
+ assert(src_box.x + src_box.width <= (int)u_minify(src->width0, src_level));
+ assert(src_box.y + src_box.height <= (int)u_minify(src->height0, src_level));
+ assert(dst_box.x + dst_box.width <= (int)u_minify(dst->width0, dst_level));
+ assert(dst_box.y + dst_box.height <= (int)u_minify(dst->height0, dst_level));
/* check that total number of src, dest bytes match */
assert((src_box.width / src_bw) * (src_box.height / src_bh) * src_bs ==
--
2.13.6
More information about the mesa-dev
mailing list