Re: [PATCH wayland] Add enum attribute to arg elements

2015-04-22 Thread Pekka Paalanen
On Tue, 21 Apr 2015 16:32:40 + Jeroen Bollen jbin...@gmail.com wrote: I think I have thought out a solution. The scanner should be able to parse multiple specification files. There would be the default wayland.xml, and for example a gnome.xml, which is an extension to the wayland.xml.

Re: [PATCH wayland] Add enum attribute to arg elements

2015-04-22 Thread Bill Spitzak
I think this looks like the correct patch. Only correction is that I would put the enum right after the type=int consistently. Some of your cases you put the summary between them. On 04/19/2015 01:30 PM, Auke Booij wrote: On 19 April 2015 at 14:51, Jeroen Bollen jbin...@gmail.com wrote:

Re: [PATCH 5/6] compositor-drm: Fix refresh rate selection in drm_output_switch_mode

2015-04-22 Thread Derek Foreman
On 22/04/15 12:42 AM, Mario Kleiner wrote: On 04/16/2015 07:14 PM, Derek Foreman wrote: On 02/04/15 12:10 AM, Mario Kleiner wrote: The matching logic in choose_mode() compared refresh rate of a drm_mode candidate mode expressed in Hz against the requested refresh rate of the target

[PATCH libinput] evdev: add support for LIBINPUT_MODEL_* udev tags

2015-04-22 Thread Peter Hutterer
Some devices need specific configuration or different defaults. Push that into udev rules and a hwdb file, that's where detection is the easiest. The LIBINPUT_MODEL_ prefix is used to determine some type of device model. Note that this property is a private API and subject to change at any time

[PATCH 2/2] doc: Use enum argument type to make links in protocol documentation

2015-04-22 Thread Bill Spitzak
--- doc/publican/protocol-to-docbook.xsl | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/doc/publican/protocol-to-docbook.xsl b/doc/publican/protocol-to-docbook.xsl index 7b45969..3c4e140 100644 --- a/doc/publican/protocol-to-docbook.xsl +++

[PATCH 1/2] Clarify the enum argument type

2015-04-22 Thread Bill Spitzak
This improvement to the protocol allows you to refer to the kind of enum you are expecting. It also introduces a distinction between enums that are bitfields, ie that can be OR'ed together. --- protocol/wayland.dtd |2 ++ protocol/wayland.xml | 32 2 files

[PATCH] doc: use enum information to make documentation links

2015-04-22 Thread Bill Spitzak
This is based on (and includes unchanged) Auke Booij's patch to protocol.xml to add enum type information. This ignores whether an enumeration is a bitmap, but that seems to be clear from the summary text. ___ wayland-devel mailing list

Re: [PATCH] test-touchpad: Adjust touchpad_edge_scroll_no_2fg test for gesture support

2015-04-22 Thread Peter Hutterer
On Wed, Apr 22, 2015 at 10:32:38AM +0200, Hans de Goede wrote: Unlike all the other 2fg scroll tests the touchpad_edge_scroll_no_2fg test puts the 2 fingers down quite far apart, this makes the pinch vs scroll gesture detection code in the gestures branch detect a pinch causing the test to

[PATCH v2 weston] window: Use wl_cursor_frame_and_duration() for mouse cursor updates

2015-04-22 Thread Derek Foreman
Some animated cursor sets use very long delays, but until now we'd use the frame callback and update the cursor at the display framerate anyway. Now we use a timerfd to drive cursor animation if the delay is longer than 100ms, or the old method for short delays. Signed-off-by: Derek Foreman

Re: Wayland Relative Pointer API Progress

2015-04-22 Thread x414e54
On Wed, Apr 22, 2015 at 1:52 PM, Peter Hutterer peter.hutte...@who-t.net wrote: The real problem regarding the mouse position is that you now rely on the client and the compositor to calculate the cursor position exactly the same way. If not, you may end up leaving the window when the cursor as

Re: [PATCH libinput v4 2/2] udev: add a custom udev rule for X230 touchpads

2015-04-22 Thread Peter Hutterer
On Mon, Apr 20, 2015 at 02:29:45PM -0400, Benjamin Tissoires wrote: X230 touchpads should be tagged as LIBINPUT_MODEL_LENOVO_X230 by udev to apply a different acceleration profile. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@gmail.com Reviewed-by: Peter Hutterer

Re: Wayland Relative Pointer API Progress

2015-04-22 Thread x414e54
On Thu, Apr 23, 2015 at 1:51 PM, x414e54 x414...@linux.com wrote: On Wed, Apr 22, 2015 at 1:52 PM, Peter Hutterer peter.hutte...@who-t.net wrote: The real problem regarding the mouse position is that you now rely on the client and the compositor to calculate the cursor position exactly the

[PATCH] test-touchpad: Adjust touchpad_edge_scroll_no_2fg test for gesture support

2015-04-22 Thread Hans de Goede
Unlike all the other 2fg scroll tests the touchpad_edge_scroll_no_2fg test puts the 2 fingers down quite far apart, this makes the pinch vs scroll gesture detection code in the gestures branch detect a pinch causing the test to fail. This commit brings the finger placement in line with the other

[PATCH libinput] Add libinput_device_keyboard_has_key()

2015-04-22 Thread Peter Hutterer
Similar to libinput_device_pointer_has_button(), this function returns whether a given device has a specific keycode. This enables a caller to determine if the device is really a keyboard (check for KEY_A-KEY_Z) or just a media key device (check for KEY_PLAY or somesuch), depending on the context

Re: Wayland Relative Pointer API Progress

2015-04-22 Thread Peter Hutterer
On Tue, Apr 21, 2015 at 06:05:02PM -0700, Bill Spitzak wrote: Interesting. It does seem like a good idea to do remote by providing identical device api's. This probably applies to sound output too. There will have to be simple and obvious methods to figure out the remote machine so that all

Re: [PATCH libinput] Add libinput_device_keyboard_has_key()

2015-04-22 Thread Hans de Goede
Hi, On 22-04-15 06:44, Peter Hutterer wrote: Similar to libinput_device_pointer_has_button(), this function returns whether a given device has a specific keycode. This enables a caller to determine if the device is really a keyboard (check for KEY_A-KEY_Z) or just a media key device (check for

Re: [PATCH wayland] Add enum attribute to arg elements

2015-04-22 Thread Auke Booij
On 22 April 2015 at 08:34, Pekka Paalanen ppaala...@gmail.com wrote: I also think this discussion is going off-topic. You wanted to add annotations to the XML, so you could find out about enum and bitfield arguments, so let's keep to that. There is value in simplicity. How about this: Add