david.venhoff/QEMU-Nyx-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

vnc: tight: don't forget last pixel in tight_encode_indexed_rect

Browse Source 
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 <corentincj@iksaif.net>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
Corentin Chary 2010-06-01 23:05:42 +02:00 committed by Anthony Liguori
parent 637503d122
commit 54d43eacc5
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
vnc-encoding-tight.c
View File

@ -249,17 +249,16 @@ static void print_palette(const char *key, QObject *obj, void *opaque)
uint##bpp##_t *src; \ uint##bpp##_t *src; \
uint##bpp##_t rgb; \ uint##bpp##_t rgb; \
uint8_t key[6]; \ uint8_t key[6]; \
int rep = 0; \ int i, rep; \
uint8_t idx; \ uint8_t idx; \
\ \
src = (uint##bpp##_t *) buf; \ src = (uint##bpp##_t *) buf; \
\ \
count -= 1; \ for (i = 0; i < count; i++) { \
while (count--) { \
rgb = *src++; \ rgb = *src++; \
rep = 0; \ rep = 0; \
while (count && *src == rgb) { \ while (i < count && *src == rgb) { \
rep++, src++, count--; \ rep++, src++, i++; \
} \ } \
tight_palette_rgb2buf(rgb, bpp, key); \ tight_palette_rgb2buf(rgb, bpp, key); \
if (!qdict_haskey(palette, (char *)key)) { \ if (!qdict_haskey(palette, (char *)key)) { \