[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vnc: set the right prefered encoding
From: |
Corentin Chary |
Subject: |
Re: [Qemu-devel] [PATCH] vnc: set the right prefered encoding |
Date: |
Thu, 6 May 2010 09:00:58 +0200 |
On Tue, May 4, 2010 at 2:01 PM, Corentin Chary <address@hidden> wrote:
> From RFB specs: "The order of the encoding types given in this
> message is a hint by the client as to its preference (the first
> encoding specified being most preferred)"
>
> Signed-off-by: Corentin Chary <address@hidden>
> ---
> vnc.c | 14 ++++++++++----
> 1 files changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/vnc.c b/vnc.c
> index 5241a6a..2d05d8f 100644
> --- a/vnc.c
> +++ b/vnc.c
> @@ -1594,7 +1594,7 @@ static void set_encodings(VncState *vs, int32_t
> *encodings, size_t n_encodings)
>
> vnc_zlib_init(vs);
> vs->features = 0;
> - vs->vnc_encoding = 0;
> + vs->vnc_encoding = -1;
> vs->tight_compression = 9;
> vs->tight_quality = 9;
> vs->absolute = -1;
> @@ -1603,18 +1603,24 @@ static void set_encodings(VncState *vs, int32_t
> *encodings, size_t n_encodings)
> enc = encodings[i];
> switch (enc) {
> case VNC_ENCODING_RAW:
> - vs->vnc_encoding = enc;
> + if (vs->vnc_encoding != -1) {
> + vs->vnc_encoding = enc;
> + }
hum patch is broken, sending a fixed patch today, sorry
should be if (vs->vnc_encoding == -1)
--
Corentin Chary
http://xf.iksaif.net