diff options
| author | Vinson Lee <vlee@freedesktop.org> | 2013-09-09 20:14:28 -0700 |
|---|---|---|
| committer | Vinson Lee <vlee@freedesktop.org> | 2013-09-10 10:44:09 -0700 |
| commit | 49f2ba2cb07202b93955d15bc2bce2d4ae77bbf9 (patch) | |
| tree | 56befeba21afbe88f688ca2878051ee6817bde5e | |
| parent | d93e23ba25cbfd8f77a4b3a113ca7ed3dece1e68 (diff) | |
radeonsi: Add parentheses around '|' operands.
Fixes GCC parentheses warning.
r600_texture.c: In function 'si_texture_create':
r600_texture.c:518:20: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses]
!(templ->bind & PIPE_BIND_CURSOR | PIPE_BIND_LINEAR)) {
^
Fixes "Wrong operator used" defect reported by Coverity.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
| -rw-r--r-- | src/gallium/drivers/radeonsi/r600_texture.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/radeonsi/r600_texture.c b/src/gallium/drivers/radeonsi/r600_texture.c index ef4e8653ae..e6e8e200e5 100644 --- a/src/gallium/drivers/radeonsi/r600_texture.c +++ b/src/gallium/drivers/radeonsi/r600_texture.c @@ -515,7 +515,7 @@ struct pipe_resource *si_texture_create(struct pipe_screen *screen, int r; if (!(templ->flags & R600_RESOURCE_FLAG_TRANSFER) && - !(templ->bind & PIPE_BIND_CURSOR | PIPE_BIND_LINEAR)) { + !(templ->bind & (PIPE_BIND_CURSOR | PIPE_BIND_LINEAR))) { if (templ->flags & R600_RESOURCE_FLAG_FORCE_TILING || templ->nr_samples > 1) { array_mode = V_009910_ARRAY_2D_TILED_THIN1; |