Pierre Ossman <oss...@cendio.se> wrote on 09/11/2011 12:36:24:

> Looks good. But I'm thinking that going via ConnParams is
> probably overkill. Just put the configuration object at the top of
> CMsgReader.cxx, the same way it is for SMsgReader.cxx. A name change
> for both of the parameters might be in order though to avoid conflicts.
> Note that the external name can be the same, but the object name should
> be different.

Thank you

In order to expose it in the Options dialog, I've published the parameter
as a static method of CMsgReader, rather than exposing it as a global
variable.

Not sure this is the best solution though. So?

-- 
Matthieu Lochegnies
Altis Semiconductor - http://www.altissemiconductor.com/

Attachment: tigervnc_cutnpaste.patch
Description: Binary data

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
Tigervnc-devel mailing list
Tigervnc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-devel

Reply via email to