<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 30, 2015 at 6:40 AM, Marek Chalupa <span dir="ltr"><<a href="mailto:mchqwerty@gmail.com" target="_blank">mchqwerty@gmail.com</a>></span> wrote:<br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-        This request only takes effect if the seat has the pointer<br>
-        capability.<br>
+       This request always takes effect, but the created wl_pointer object<br>
+       will only emit events as long as the seat has the pointer capability.<br></blockquote></div></div></blockquote><div><br></div><div>I would delete all the stuff about "takes effect" as it is confusing. How about:<br><br></div><div>    The created wl_pointer object will only emit events as long as the seat has the pointer capability.<br></div><div><br>> To have it comprehensive, you could mention that even the requests to such object will be ignored.<br><br></div><div>I don't think "release" is ignored. And the only other request I see is "set_cursor" which says it is ignored if the caller does not have the pointer focus, which it won't when there is no pointer, so that is already true. So anyway it seems that what happens when there is no capability should be described in the request, and probably only when it is not obvious.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
And more unspecified things: client should destroy the (let's say) wl_touch once it gets capabilities without touch (from the formulation<br>
is not clear that after the seat gains the touch capability back,<br>
the old object won't start functioning again).<br></blockquote><div><br></div><div>I would greatly prefer the object to just start/stop working when the capabilities change. This is probably easier for the compositor to implement and a lot easier on the clients, who would like to create the wl_seat and wl_pointer and wl_keyboard all at once and assume these work without changes until the seat itself is destroyed.<br><br></div></div></div></div>