qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@linux.vnet.ibm.com>
To: Alexander Graf <agraf@suse.de>
Cc: Corentin Chary <corentincj@iksaif.net>,
	Qemu-development List <qemu-devel@nongnu.org>
Subject: [Qemu-devel] Re: [PATCH 2/5] vnc: JPEG should be disabled if the client don't set tight quality
Date: Fri, 04 Jun 2010 08:19:01 -0500	[thread overview]
Message-ID: <4C08FD45.8000404@linux.vnet.ibm.com> (raw)
In-Reply-To: <17E76748-6C1D-4A9A-8AD2-97AA128C189E@suse.de>

On 06/04/2010 08:16 AM, Alexander Graf wrote:
> On 04.06.2010, at 15:18, Corentin Chary wrote:
>
>    
>> Disable JPEG compression by default and only enable it if the
>> VNC client has sent the requested quality.
>>
>> Signed-off-by: Corentin Chary<corentincj@iksaif.net>
>> ---
>> vnc.c |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/vnc.c b/vnc.c
>> index ed0e096..9cf38d1 100644
>> --- a/vnc.c
>> +++ b/vnc.c
>> @@ -1644,7 +1644,7 @@ static void set_encodings(VncState *vs, int32_t *encodings, size_t n_encodings)
>>      vs->features = 0;
>>      vs->vnc_encoding = 0;
>>      vs->tight_compression = 9;
>> -    vs->tight_quality = 9;
>> +    vs->tight_quality = -1; /* Lossless by default */
>>      
> So what's the threshold? When do you start to be lossy?
>    

When the client explicitly requests it.

Regards,

Anthony Liguori

> Alex
>
>    

  reply	other threads:[~2010-06-04 13:20 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-04 13:18 [Qemu-devel] [PATCH 0/5] vnc updates and ui move Corentin Chary
     [not found] ` <1275657527-26032-5-git-send-email-corentincj@iksaif.net>
2010-06-04 13:18   ` [Qemu-devel] Re: [PATCH 4/5] ui: move all ui components in ui/ Alexander Graf
2010-06-04 14:23     ` Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 1/5] vnc: tight: add JPEG and gradient subencoding with smooth image detection Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 2/5] vnc: JPEG should be disabled if the client don't set tight quality Corentin Chary
2010-06-04 13:16   ` [Qemu-devel] " Alexander Graf
2010-06-04 13:19     ` Anthony Liguori [this message]
2010-06-04 13:20       ` Alexander Graf
2010-06-04 13:48         ` Corentin Chary
2010-06-04 13:52           ` Alexander Graf
2010-06-04 14:22             ` Corentin Chary
2010-06-04 13:18 ` [Qemu-devel] [PATCH 3/5] vnc: add lossless option Corentin Chary
2010-06-04 13:17   ` [Qemu-devel] " Alexander Graf
2010-06-04 13:20     ` Anthony Liguori
2010-06-04 13:18 ` [Qemu-devel] [PATCH 5/5] vnc: rename vnc-encoding-* vnc-enc-* Corentin Chary
2010-06-04 13:18   ` [Qemu-devel] " Alexander Graf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C08FD45.8000404@linux.vnet.ibm.com \
    --to=aliguori@linux.vnet.ibm.com \
    --cc=agraf@suse.de \
    --cc=corentincj@iksaif.net \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).