[PATCH spice-streaming-agent v2] Eliminate signed/unsigned warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Christophe de Dinechin <dinechin@xxxxxxxxxx>

Currently -Wsign-compare is disabled by default in the default settings
(m4/spice-compile-warnings.m4).
However is good and not that expensive to remove this warning.

Signed-off-by: Christophe de Dinechin <dinechin@xxxxxxxxxx>
Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
 src/mjpeg-fallback.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Changes since v1:
- update comment

diff --git a/src/mjpeg-fallback.cpp b/src/mjpeg-fallback.cpp
index 605a4b3..b64762c 100644
--- a/src/mjpeg-fallback.cpp
+++ b/src/mjpeg-fallback.cpp
@@ -47,7 +47,7 @@ private:
     std::vector<uint8_t> frame;
 
     // last frame sizes
-    uint32_t last_width = ~0u, last_height = ~0u;
+    int last_width = -1, last_height = -1;
     // last time before capture
     uint64_t last_time = 0;
 };
@@ -70,7 +70,7 @@ MjpegFrameCapture::~MjpegFrameCapture()
 void MjpegFrameCapture::Reset()
 {
     frame.clear();
-    last_width = last_height = ~0u;
+    last_width = last_height = -1;
 }
 
 FrameInfo MjpegFrameCapture::CaptureFrame()
-- 
2.14.3

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]