Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Rémi Denis-Courmont
Hello, Le 2014-10-06 03:29, Jason Ekstrand a écrit : Remi, While this would probably be nice, your approach isnt going to work.  Yeah, I figured that much after I submitted the patch. And then I rediscovered that the client library does not actually know the version of the project

Re: [RFC libwayland] Track protocol object versions inside wl_proxy.

2014-10-06 Thread Pekka Paalanen
On Sun, 5 Oct 2014 18:48:27 -0700 Jasper St. Pierre jstpie...@mecheye.net wrote: One of my issues is what happens for objects that have multiple parents, like wl_buffer or wl_callback. What do we do if another person wants to introduce another (like the recently proposed pointer lock

Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Pekka Paalanen
Re-adding CCs and some more... On Mon, 06 Oct 2014 09:17:41 +0300 Rémi Denis-Courmont r...@remlab.net wrote: Hello, Le 2014-10-06 03:29, Jason Ekstrand a écrit : Remi, While this would probably be nice, your approach isnt going to work.  Yeah, I figured that much after I submitted

Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Pekka Paalanen
On Mon, 6 Oct 2014 10:45:14 +0300 Pekka Paalanen ppaala...@gmail.com wrote: Re-adding CCs and some more... On Mon, 06 Oct 2014 09:17:41 +0300 Rémi Denis-Courmont r...@remlab.net wrote: Hello, Le 2014-10-06 03:29, Jason Ekstrand a écrit : Remi, While this would probably be

Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Rémi Denis-Courmont
Le 2014-10-06 11:26, Pekka Paalanen a écrit : 4 or higher as a wl_surface version 3? If not, I cannot rely on 'wl_proxy_get_version(surface) = 2'... Changing the semantics[5] of wl_surface.damage is a good example where this would fail indeed. If a library only knows up to version 3, but

Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Rémi Denis-Courmont
Le 2014-10-06 12:52, Pekka Paalanen a écrit : Could this be solved (completely?) by saying that libraries that take existing wl_proxy objects such that they a) set up listeners, or b) create new wl_proxies from it and set their listeners; need to also offer API to tell which versions of the

Re: [PATCH libinput] cosmetic: convert evdev_need_touch_frame from int to bool

2014-10-06 Thread Peter Hutterer
On Fri, Oct 03, 2014 at 03:01:35PM -0500, Derek Foreman wrote: We use stdbool elsewhere, so use it here too for consistency. --- src/evdev.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/evdev.c b/src/evdev.c index d8e3e5a..1b4ce10 100644 ---

Re: [PATCH libinput 0/8] Add libinput_device_suspend() to disable devices

2014-10-06 Thread Peter Hutterer
On Tue, Sep 30, 2014 at 11:36:30AM -0700, Bill Spitzak wrote: On 09/11/2014 07:34 AM, Bastien Nocera wrote: I don't like the Smart disable as a name because the consumer of the API might only see one device for both the trackpoint and the touchpad in which case you want to disable

Re: [RFC libwayland] Track protocol object versions inside wl_proxy.

2014-10-06 Thread Jason Ekstrand
On Mon, Oct 6, 2014 at 12:20 AM, Pekka Paalanen ppaala...@gmail.com wrote: On Sun, 5 Oct 2014 18:48:27 -0700 Jasper St. Pierre jstpie...@mecheye.net wrote: One of my issues is what happens for objects that have multiple parents, like wl_buffer or wl_callback. What do we do if another

Re: [PATCH wayland] client: add wl_proxy_get_version function

2014-10-06 Thread Jason Ekstrand
On Oct 6, 2014 12:45 AM, Pekka Paalanen ppaala...@gmail.com wrote: Re-adding CCs and some more... On Mon, 06 Oct 2014 09:17:41 +0300 Rémi Denis-Courmont r...@remlab.net wrote: Hello, Le 2014-10-06 03:29, Jason Ekstrand a écrit : Remi, While this would probably be nice, your

Re: [PATCH libinput 0/8] Add libinput_device_suspend() to disable devices

2014-10-06 Thread Bill Spitzak
On 10/06/2014 05:59 AM, Peter Hutterer wrote: FORCE_ENABLED - Bypass anything that makes the device stop reporting events. why would you want that setting? if it's disabled when it shouldn't be then this is a bug we need to fix. It is what you are calling enabled. My complaint is that the