On Fri, 13 Jul 2012 12:50:00 +0200
Pierre Ossman <oss...@cendio.se> wrote:

> On Fri, 13 Jul 2012 05:18:16 -0500
> DRC <dcomman...@users.sourceforge.net> wrote:
> 
> > 
> > Maybe I'm missing something, but isn't the goal to hook only drawing
> > commands that actually produce visible information?  That being the
> > case, isn't the relevant data already being hooked when the pixmap gets
> > copied to the window using XCopyArea()?
> > 
> 
> That was the assumption. Something is wrong about that assumption
> though as we're seeing missing updates. Only speculation as to what so
> far.
> 

Mystery somewhat clearer. The reason we're missing the update, and why
the pixmap hack works, is because the borders are drawn directly on the
_screen_ pixmap.

That explains why the coordinates still make sense, even though we're
hooking a pixmap instead of a window.

Not sure what the fix is. Starting to activate hooks even for just the
screen pixmap still means we have to audit the code for assumptions
about windows. Not terribly appealing. I'll dig around and see if I can
find something cleaner...

Rgds
-- 
Pierre Ossman           Software Development
Cendio AB               http://cendio.com
Teknikringen 8          http://twitter.com/ThinLinc
583 30 Linköping        http://facebook.com/ThinLinc
Phone: +46-13-214600    http://plus.google.com/112509906846170010689

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Attachment: signature.asc
Description: PGP signature

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Tigervnc-devel mailing list
Tigervnc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-devel

Reply via email to