On 06/01/2010 04:05 PM, Corentin Chary wrote:
A simple patch would have been to just remove count -= 1, but this
one also replace the while (count--) with a for(i = 0; i< count; i++)
which I believe is more easy to understand.
Signed-off-by: Corentin Chary<corenti...@iksaif.net>
Applied all. Thanks.
Regards,
Anthony Liguori
---
vnc-encoding-tight.c | 9 ++++-----
1 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/vnc-encoding-tight.c b/vnc-encoding-tight.c
index 50be44e..c8effe6 100644
--- a/vnc-encoding-tight.c
+++ b/vnc-encoding-tight.c
@@ -249,18 +249,16 @@ static void print_palette(const char *key, QObject *obj,
void *opaque)
uint##bpp##_t *src; \
uint##bpp##_t rgb; \
uint8_t key[6]; \
- int rep = 0; \
+ int i, rep; \
uint8_t idx; \
\
src = (uint##bpp##_t *) buf; \
\
- count -= 1; \
- while (count--) { \
+ for (i = 0; i< count; i++) { \
rgb = *src++; \
rep = 0; \
- while (count&& *src == rgb) { \
- rep++, src++, count--; \
+ while (i< count&& *src == rgb) { \
+ rep++, src++, i++; \
} \
tight_palette_rgb2buf(rgb, bpp, key); \
if (!qdict_haskey(palette, (char *)key)) { \