<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 8, 2014 at 3:23 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">> RRCrtcChangeNotify is a event subtype of RRNotify, since that's the only<br>
> one we can catch here, no need to filter futher.<br>
<br>
</span>well, let's use the more specific type then, this is the one which was<br>
used before so it makes things easier to follow, and RRCrtcChangeNotify<br>
is documented in<br>
<a href="http://cgit.freedesktop.org/xorg/proto/randrproto/tree/randrproto.txt" target="_blank">http://cgit.freedesktop.org/xorg/proto/randrproto/tree/randrproto.txt</a><br>
while RRNotify is not.</blockquote></div><br></div><div class="gmail_extra">So you want that?<br><br>         case RRNotify: {<br>+            XRRNotifyEvent *ne = (XRRNotifyEvent *) &event;<br>+            g_assert(ne->subtype == RRNotify_CrtcChange);<br><br>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.<br><br><br></div><div class="gmail_extra"><br>-- <br>Marc-André Lureau
</div></div>