On Wed, Oct 8, 2014 at 3:23 PM, Christophe Fergeau <[email protected]>
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);

It's superflous, RRNotify_CrtcChange is a subtype of RRNotify, the only one
we can catch here.  Even if we were to receive other RRNotify events, it
wouldn't harm.



-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to