Den 2009-06-16 13:33 skrev Pierre Ossman:
> On Thu, 11 Jun 2009 13:29:42 +0200
> Peter Rosin <p...@lysator.liu.se> wrote:
> 
>> +Compression Level Pseudo-encoding
>> +---------------------------------
>> +
>> +Specifies the desired compression level. Encoding number -247 implies
>> +high compression level, -255 implies low compression level. Low
>> +compression level can be useful to get low latency in medium to high
>> +bandwidth situations and high compression level can be useful in low
>> +bandwidth situations.
>> +
> 
> Perhaps we should specify what low and high means a little more
> clearly. Like that fact that this controls lossless compression, so
> it's primarily a tradeoff between CPU time and bandwidth.

Right.

> And should we reserve these settings to only Tight or encourage others
> to use it? I think this can be used more broadly, but I think we need
> to be more verbose in its description for that.

I see little reason to restrict to Tight only. But nothing uses it
to my knowledge so it would be a little bit out of place to mention
that something else /could/ use it...

> And I also think this should just be a hint to the server, not a
> requirement. Since we do not specify exactly what the levels mean, I
> don't see how it can be anything other than a hint.

I've added a note about this. Will send an updated patch series
with all updates...

> Either way, I think these things should be mentioned.

Cheers,
Peter

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
tigervnc-rfbproto mailing list
tigervnc-rfbproto@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-rfbproto

Reply via email to