Re: [Tigervnc-devel] [PATCH] Add an automatic lossless refresh mechanism.

2012-02-27 Thread Pierre Ossman
On Mon, 27 Feb 2012 12:31:20 +0100 Arthur Huillet arthur.huil...@free.fr wrote: It sends a full update of the screen, in high quality JPEG format, after a programmable idle time. This is disabled by default. --- common/rfb/ServerCore.cxx |8 common/rfb/ServerCore.h

Re: [Tigervnc-devel] [PATCH] Add an automatic lossless refresh mechanism.

2012-02-27 Thread DRC
Since I developed the original ALR mechanism for TurboVNC, I have a couple of comments/critiques of this, based on the real-world usage of the feature in TurboVNC. The first is that people use TurboVNC's ALR feature in the medical viz industry, where mathematical losslessness is important, so

[Tigervnc-devel] [PATCH] [TRIVIAL] Comment explaining why we pretend there are 65535 rectangles in a FB update.

2012-02-27 Thread Arthur Huillet
--- Hello, I wasn't sure whether to send this or not, but since it took me a trip to the protocol spec to understand, I think it's worth adding some information. Please apply as you see fit. common/rfb/VNCSConnectionST.cxx |4 1 file changed, 4

Re: [Tigervnc-devel] [PATCH] Add an automatic lossless refresh mechanism.

2012-02-27 Thread DRC
I still disagree with several things in general: -- I didn't have a problem with the long variable names, but I would prefer that the abbreviation ALR or alr be used instead of automatic refresh. Automatic refresh is not a good name for the feature. -- For my purposes, the feature isn't useful

Re: [Tigervnc-devel] [PATCH] Add an automatic lossless refresh mechanism.

2012-02-27 Thread Arthur HUILLET
On Mon, Feb 27, 2012 at 02:19:20PM -0600, DRC wrote: I still disagree with several things in general: -- I didn't have a problem with the long variable names, but I would prefer that the abbreviation ALR or alr be used instead of automatic refresh. Automatic refresh is not a good name for