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 the > feature.
I like ALR too but Pierre did not seem to agree. So I shortened the text and I felt that dropping "lossless" was better than dropping "automatic". > That means that it needs lossy region tracking As stated earlier, this will happen at a later point in time. I believe in incremental software development and consider that a little something is better than nothing at all, especially since the patch I've sent is able to cover a few use cases already. > and mathematically lossless needs to be the default. I feel otherwise but do not care at all. I can make it the default if people feel like it. However the current patch shouldn't be used with mathematically lossless as it sends updates that are too big. My proposal is to change the default in a future patch when I get around to implementing lossy region tracking, because at this point it will make sense technically. > you don't have to have the clunky "use quality 10 as mathematical lossless" > rule. You > just set alrQuality to -1 by default, and the user overrides that to 0-9. I can replace the 10 with a -1 pretty easily, and you're right that it's better. Will do (unless the committer wants to do it directly). I'm rather more annoyed by the 0-9 vs. 0-100 scale, which is a big difference between Turbo and Tiger and makes it difficult to move flawlessly from one to the other (but I don't have any plans to deal with it as I suspect it would already have been dealt with if it was easy and worth it). Thanks for the comments -- A. Huillet ------------------------------------------------------------------------------ Try before you buy = See our experts in action! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-dev2 _______________________________________________ Tigervnc-devel mailing list Tigervnc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tigervnc-devel
