[Spice-devel] [spice-common v2 6/6] quic: Use i == 0 rather than !i

Christophe Fergeau cfergeau at redhat.com
Tue Aug 1 13:55:09 UTC 2017


This helps to make quic_rgb_tmpl.c and quic_tmpl.c more consistent.

Signed-off-by: Christophe Fergeau <cfergeau at redhat.com>
---
 common/quic_rgb_tmpl.c | 8 ++++----
 common/quic_tmpl.c     | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/common/quic_rgb_tmpl.c b/common/quic_rgb_tmpl.c
index 15d14b9..23dea39 100644
--- a/common/quic_rgb_tmpl.c
+++ b/common/quic_rgb_tmpl.c
@@ -237,7 +237,7 @@ static void FNAME(compress_row0_seg)(Encoder *encoder, int i,
 
     spice_assert(end - i > 0);
 
-    if (!i) {
+    if (i == 0) {
         unsigned int codeword, codewordlen;
 
         COMPRESS_ONE_ROW0_0(r);
@@ -349,7 +349,7 @@ static void FNAME(compress_row_seg)(Encoder *encoder, int i,
 
     spice_assert(end - i > 0);
 
-    if (!i) {
+    if (i == 0) {
         unsigned int codeword, codewordlen;
 
         COMPRESS_ONE_0(r);
@@ -492,7 +492,7 @@ static void FNAME(uncompress_row0_seg)(Encoder *encoder, int i,
 
     spice_assert(end - i > 0);
 
-    if (!i) {
+    if (i == 0) {
         unsigned int codewordlen;
 
         UNCOMPRESS_PIX_START(&cur_row[i]);
@@ -613,7 +613,7 @@ static void FNAME(uncompress_row_seg)(Encoder *encoder,
 
     spice_assert(end - i > 0);
 
-    if (!i) {
+    if (i == 0) {
         unsigned int codewordlen;
 
         UNCOMPRESS_PIX_START(&cur_row[i]);
diff --git a/common/quic_tmpl.c b/common/quic_tmpl.c
index b77dc07..b9056bb 100644
--- a/common/quic_tmpl.c
+++ b/common/quic_tmpl.c
@@ -255,7 +255,7 @@ static void FNAME(compress_row_seg)(Encoder *encoder, Channel *channel, int i,
 
     spice_assert(end - i > 0);
 
-    if (!i) {
+    if (i == 0) {
         unsigned int codeword, codewordlen;
 
         decorrelate_drow[0] = family.xlatU2L[(unsigned)((int)cur_row->a -
-- 
2.13.3



More information about the Spice-devel mailing list