On Wed, Oct 08, 2014 at 03:49:29PM +0200, Marc-André Lureau wrote: > On Wed, Oct 8, 2014 at 3:23 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> > wrote: > > > > RRCrtcChangeNotify is a event subtype of RRNotify, since that's the only > > > one we can catch here, no need to filter futher. > > > > well, let's use the more specific type then, this is the one which was > > used before so it makes things easier to follow, and RRCrtcChangeNotify > > is documented in > > http://cgit.freedesktop.org/xorg/proto/randrproto/tree/randrproto.txt > > while RRNotify is not. > > > So you want that? > > case RRNotify: { > + XRRNotifyEvent *ne = (XRRNotifyEvent *) &event; > + g_assert(ne->subtype == RRNotify_CrtcChange); Oh sorry, I assumed there was a RRCrtcChangeNotify similar to RRScreenChangeNotify but this does not exist. I'd just add a comment about the kind of event we handle here /* RRCrtcChangeNotify */
Attachment:
pgpeyQ8su5RZi.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel