diff options
| author | Jeremy Huddleston <jeremyhu@apple.com> | 2015-10-18 23:12:49 -0700 |
|---|---|---|
| committer | Adam Jackson <ajax@redhat.com> | 2015-10-19 11:51:52 -0400 |
| commit | 5dc415048e4091b18cd7d123ebeae8f95ed5a4f5 (patch) | |
| tree | d68a7c5348e536b931c1b117f6729e0dbd63bcdb | |
| parent | 9a2a05a9a7ba02921fa29844c4cad41243c41326 (diff) | |
randr: Silence -Wshift-negative-value warnings
rrtransform.c:199:23: warning: shifting a negative signed value is undefined [-Wshift-negative-value,Semantic Issue]
rot_cos = F(-1);
^~~~~
rrtransform.c:114:14: note: expanded from macro 'F'
^~~~~~~~~~~~~~
../render/picture.h:200:24: note: expanded from macro 'IntToxFixed'
^~~~~~~~~~~~~~~~~~~~~~
/opt/X11/include/pixman-1/pixman.h:130:56: note: expanded from macro 'pixman_int_to_fixed'
~~~ ^
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
| -rw-r--r-- | randr/rrtransform.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/randr/rrtransform.c b/randr/rrtransform.c index 26b0649c9..d32b43aa3 100644 --- a/randr/rrtransform.c +++ b/randr/rrtransform.c @@ -196,7 +196,7 @@ RRTransformCompute(int x, f_rot_sin = 0; f_rot_dx = width; f_rot_dy = height; - rot_cos = F(-1); + rot_cos = F(~0u); rot_sin = F(0); rot_dx = F(width); rot_dy = F(height); @@ -207,7 +207,7 @@ RRTransformCompute(int x, f_rot_dx = 0; f_rot_dy = width; rot_cos = F(0); - rot_sin = F(-1); + rot_sin = F(~0u); rot_dx = F(0); rot_dy = F(width); break; @@ -230,7 +230,7 @@ RRTransformCompute(int x, scale_dy = 0; if (rotation & RR_Reflect_X) { f_scale_x = -1; - scale_x = F(-1); + scale_x = F(~0u); if (rotation & (RR_Rotate_0 | RR_Rotate_180)) { f_scale_dx = width; scale_dx = F(width); @@ -242,7 +242,7 @@ RRTransformCompute(int x, } if (rotation & RR_Reflect_Y) { f_scale_y = -1; - scale_y = F(-1); + scale_y = F(~0u); if (rotation & (RR_Rotate_0 | RR_Rotate_180)) { f_scale_dy = height; scale_dy = F(height); |