On Thu, 07 May 2009 09:23:32 +0200
Peter Rosin <p...@lysator.liu.se> wrote:

> 
> I think this is accurate.
> 
> Signed-off-by: Peter Rosin <p...@lysator.liu.se>
> 

I'm not sure that Constantin is subscribed unfortunately, but perhaps
there are other Tight experts here that can have a look.

I have one question though:

> +where ``CAPABILITY`` is
> +
> +=============== =============================== =======================
> +No. of bytes    Type                            Description
> +=============== =============================== =======================
> +4               ``U32``                         *code*
> +4               ``U8`` array                    *vendor*
> +8               ``U8`` array                    *signature*
> +=============== =============================== =======================

Why is there both a numerical code, and a string pair used to identify
the extension? It seems a bit redundant.

> +
> +The following *encoding* capabilities are registered:
> +
> +=========== ========= ============= ===================================
> +Code        Vendor    Signature     Description
> +=========== ========= ============= ===================================
> +0           ``STDV``  ``RAW_____``  `Raw Encoding`_
> +1           ``STDV``  ``COPYRECT``  `CopyRect Encoding`_
> +2           ``STDV``  ``RRE_____``  `RRE Encoding`_
> +4           ``STDV``  ``CORRE___``  CoRRE Encoding
> +5           ``STDV``  ``HEXTILE_``  `Hextile Encoding`_
> +6           ``TRDV``  ``ZLIB____``  ZLib Encoding
> +7           ``TGHT``  ``TIGHT___``  Tight Encoding
> +8           ``TRDV``  ``ZLIBHEX_``  ZLibHex Encoding
> +-32         ``TGHT``  ``JPEGQLVL``  JPEG Quality Level 0
> +-223        ``TGHT``  ``NEWFBSIZ``  New FB Size
> +-224        ``TGHT``  ``LASTRECT``  Last Rect
> +-232        ``TGHT``  ``POINTPOS``  Pointer Position
> +-239        ``TGHT``  ``RCHCURSR``  Rich Cursor
> +-240        ``TGHT``  ``X11CURSR``  X Cursor
> +-256        ``TGHT``  ``COMPRLVL``  Compression Level 0
> +-305        ``GGI_``  ``GII_____``  `gii (General Input Interface)
> +                                    Pseudo-encoding`_
> +=========== ========= ============= ===================================

Nit-picking, but code was defined as U32, not S32. :)

Rgds
-- 
Pierre Ossman            OpenSource-based Thin Client Technology
System Developer         Telephone: +46-13-21 46 00
Cendio AB                Web: http://www.cendio.com

Attachment: signature.asc
Description: PGP signature

------------------------------------------------------------------------------
The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your
production scanning environment may not be a perfect world - but thanks to
Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700
Series Scanner you'll get full speed at 300 dpi even with all image 
processing features enabled. http://p.sf.net/sfu/kodak-com
_______________________________________________
tigervnc-rfbproto mailing list
tigervnc-rfbproto@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-rfbproto

Reply via email to