<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On 17 November 2015 at 11:09, Jonas Ã…dahl <span dir="ltr"><<a href="mailto:jadahl@gmail.com" target="_blank">jadahl@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">The wp_relative_pointer interface is an extension to the wl_pointer<br>
interface created from wl_seat. It has the same focus, but adds the<br>
functionality of sending relative pointer motions unhindered by<br>
constraints such as monitor edges or other barriers. It also contains<br>
unaccelerated pointer motion information.<br>
<br>
Signed-off-by: Jonas Ã…dahl <<a href="mailto:jadahl@gmail.com">jadahl@gmail.com</a>><br>
---<br>
<br>
There has alreay been RBs on this protocol, but I'd like to get them<br>
explicitly re-RB:ed considering that the protocol has changed some, even<br>
though the changes have not been major ones.<br>
<br>
<br>
Changes since the last version sent as weston patches:<br>
<br>
Interfaces and protocol renamed according to wayland-protocols naming<br>
conventions.<br>
<br>
Accelerated and unaccelerated motion deltas have the data type<br>
wl_fixed_t now again. 64 bit fixed point deltas was deemed unnecessary,<br>
and wl_double_fixed_t was dropped.<br>
<br>
64 bit timestamp argument name suffix changed from msb/lsb to hi/lo.<br>
<br>
Various other minor changes.<br>
<br>
<br>
Jonas<br>
<br>
<br>
 Makefile.am  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  |  Â 1 +<br>
 unstable/relative-pointer/README  Â  Â  Â  Â  Â  Â  Â  Â  Â |  Â 4 +<br>
 .../relative-pointer-unstable-v1.xml  Â  Â  Â  Â  Â  Â  Â | 122 +++++++++++++++++++++<br>
 3 files changed, 127 insertions(+)<br>
 create mode 100644 unstable/relative-pointer/README<br>
 create mode 100644 unstable/relative-pointer/relative-pointer-unstable-v1.xml<br>
<br>
diff --git a/Makefile.am b/Makefile.am<br>
index f1bac16..2db6bd5 100644<br>
--- a/Makefile.am<br>
+++ b/Makefile.am<br>
@@ -5,6 +5,7 @@ nobase_dist_pkgdata_DATA =  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  \<br>
  Â  Â  Â  unstable/text-input/text-input-unstable-v1.xml  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  \<br>
  Â  Â  Â  unstable/input-method/input-method-unstable-v1.xml  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  \<br>
  Â  Â  Â  unstable/xdg-shell/xdg-shell-unstable-v5.xml  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  \<br>
+  Â  Â  Â unstable/relative-pointer/relative-pointer-unstable-v1.xml  Â  Â  Â  Â  Â  Â  \<br>
  Â  Â  Â  $(NULL)<br>
<br>
 pkgconfigdir = $(libdir)/pkgconfig<br>
diff --git a/unstable/relative-pointer/README b/unstable/relative-pointer/README<br>
new file mode 100644<br>
index 0000000..64c42a1<br>
--- /dev/null<br>
+++ b/unstable/relative-pointer/README<br>
@@ -0,0 +1,4 @@<br>
+Relative pointer protocol<br>
+<br>
+Maintainers:<br>
+Jonas Ã…dahl <<a href="mailto:jadahl@gmail.com">jadahl@gmail.com</a>><br>
diff --git a/unstable/relative-pointer/relative-pointer-unstable-v1.xml b/unstable/relative-pointer/relative-pointer-unstable-v1.xml<br>
new file mode 100644<br>
index 0000000..0411d1f<br>
--- /dev/null<br>
+++ b/unstable/relative-pointer/relative-pointer-unstable-v1.xml<br>
@@ -0,0 +1,122 @@<br>
+<?xml version="1.0" encoding="UTF-8"?><br>
+<protocol name="relative_pointer_unstable_v1"><br>
+<br>
+  <copyright><br>
+  Â  Copyright Â© 2014  Â  Â  Jonas Ã…dahl<br>
+  Â  Copyright Â© 2015  Â  Â  Red Hat Inc.<br>
+<br>
+  Â  Permission is hereby granted, free of charge, to any person obtaining a<br>
+  Â  copy of this software and associated documentation files (the "Software"),<br>
+  Â  to deal in the Software without restriction, including without limitation<br>
+  Â  the rights to use, copy, modify, merge, publish, distribute, sublicense,<br>
+  Â  and/or sell copies of the Software, and to permit persons to whom the<br>
+  Â  Software is furnished to do so, subject to the following conditions:<br>
+<br>
+  Â  The above copyright notice and this permission notice (including the next<br>
+  Â  paragraph) shall be included in all copies or substantial portions of the<br>
+  Â  Software.<br>
+<br>
+  Â  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
+  Â  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
+  Â  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL<br>
+  Â  THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
+  Â  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING<br>
+  Â  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER<br>
+  Â  DEALINGS IN THE SOFTWARE.<br>
+  </copyright><br>
+<br>
+  <interface name="zwp_relative_pointer_manager_v1" version="1"><br>
+  Â  <description summary="get relative pointer objects"><br>
+  Â  Â  A global interface used for getting the relative pointer object for a<br>
+  Â  Â  given seat.<br>
+<br>
+  Â  Â  Warning! The protocol described in this file is experimental and backward<br>
+  Â  Â  incompatible changes may be made. Backward compatible changes may be added<br>
+  Â  Â  together with the corresponding interface version bump.  Backward<br>
+  Â  Â  incompatible changes are done by bumping the version number in the<br>
+  Â  Â  protocol and interface names and resetting the interface version.  Once<br>
+  Â  Â  the protocol is to be declared stable, the 'z' prefix and the version<br>
+  Â  Â  number in the protocol and interface names are removed and the interface<br>
+  Â  Â  version number is reset.<br>
+  Â  </description><br></blockquote><div><br></div><div>I think that it would be better to link to README in wayland-protocols repository that describes this procedure in detail, rather than duplicating this in each protocol file. Also README doesn't define experimental protocols - instead it uses unstable term. </div><div>Therefore, something like this might be better:</div><div><br></div><div>"Warning! The protocol described in this file is unstable and backward incompatible changes may be made. See <span style="color:rgb(0,0,0);white-space:pre-wrap"> </span><a href="http://cgit.freedesktop.org/wayland/wayland-protocols/plain/README">http://cgit.freedesktop.org/wayland/wayland-protocols/plain/README</a> for details." </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
+<br>
+  Â  <request name="destroy" type="destructor"><br>
+  Â  Â  <description summary="destroy the relative pointer manager object"><br>
+  Â  Â  Â Used by the client to notify the server that it will no longer use this<br>
+  Â  Â  Â relative pointer manager object.<br>
+  Â  Â  </description><br>
+  Â  </request><br>
+<br>
+  Â  <request name="get_relative_pointer"><br>
+  Â  Â  <description summary="get a relative pointer object"><br>
+  Â  Â  Â Create a relative pointer interface given a wl_pointer object. See the<br>
+  Â  Â  Â wp_relative_pointer interface for more details.<br>
+  Â  Â  </description><br>
+<br>
+  Â  Â  <arg name="id" type="new_id" interface="zwp_relative_pointer_v1"/><br>
+  Â  Â  <arg name="pointer" type="object" interface="wl_pointer"/><br>
+  Â  </request><br>
+  </interface><br>
+<br>
+  <interface name="zwp_relative_pointer_v1" version="1"><br>
+  Â  <description summary="relative pointer object"><br>
+  Â  Â  A wp_relative_pointer object is an extension to the wl_pointer interface<br>
+  Â  Â  used for emitting relative pointer events. It shares the same focus as<br>
+  Â  Â  wl_pointer objects of the same seat and will only emit events when it has<br>
+  Â  Â  focus.<br>
+  Â  </description><br>
+<br>
+  Â  <request name="destroy" type="destructor"><br>
+  Â  Â  <description summary="release the relative pointer object"/><br>
+  Â  </request><br>
+<br>
+  Â  <event name="relative_motion"><br>
+  Â  Â  <description summary="relative pointer motion"><br>
+  Â  Â  Â Relative x/y pointer motion from the pointer of the seat associated with<br>
+  Â  Â  Â this object.<br>
+<br>
+  Â  Â  Â A relative motion is in the same dimension as regular wl_pointer motion<br>
+  Â  Â  Â events, except they do not represent an absolute position. For example,<br>
+  Â  Â  Â moving a pointer from (x, y) to (x', y') would have the equivalent<br>
+  Â  Â  Â relative motion (x' - x, y' - y). If a pointer motion caused the<br>
+  Â  Â  Â absolute pointer position to be clipped by for example the edge of the<br>
+  Â  Â  Â monitor, the relative motion is unaffected by the clipping and will<br>
+  Â  Â  Â represent the unclipped motion.<br>
+<br>
+  Â  Â  Â This event also contains non-accelerated motion deltas. The<br>
+  Â  Â  Â non-accelerated delta is, when applicable, the regular pointer motion<br>
+  Â  Â  Â delta as it was before having applied motion acceleration and other<br>
+  Â  Â  Â transformations such as normalization.<br>
+<br>
+  Â  Â  Â Note that the non-accelerated delta does not represent 'raw' events as<br>
+  Â  Â  Â they were read from some device. Pointer motion acceleration is device-<br>
+  Â  Â  Â and configuration-specific and non-accelerated deltas and accelerated<br>
+  Â  Â  Â deltas may have the same value on some devices.<br>
+<br>
+  Â  Â  Â Relative motions are not coupled to wl_pointer.motion events, and can be<br>
+  Â  Â  Â sent in combination with such events, but also independently. There may<br>
+  Â  Â  Â also be scenarious where wl_pointer.motion is sent, but there is no<br>
+  Â  Â  Â relative motion. The order of an absolute and relative motion event<br>
+  Â  Â  Â originating from the same physical motion is not guaranteed.<br>
+<br>
+  Â  Â  Â If the client needs button events or focus state, it can receive them<br>
+  Â  Â  Â from a wl_pointer object of the same seat that the wp_relative_pointer<br>
+  Â  Â  Â object is associated with.<br>
+  Â  Â  </description><br>
+<br>
+  Â  Â  <arg name="utime_hi" type="uint"<br>
+  Â  Â  Â  Â  summary="high 32 bits of a 64 bit timestamp with microsecond granularity"/><br>
+  Â  Â  <arg name="utime_lo" type="uint"<br>
+  Â  Â  Â  Â  summary="low 32 bits of a 64 bit timestamp with microsecond granularity"/><br>
+  Â  Â  <arg name="dx" type="fixed"<br>
+  Â  Â  Â  Â  summary="the x component of the motion vector"/><br>
+  Â  Â  <arg name="dy" type="fixed"<br>
+  Â  Â  Â  Â  summary="the y component of the motion vector"/><br>
+  Â  Â  <arg name="dx_unaccel" type="fixed"<br>
+  Â  Â  Â  Â  summary="the x component of the unaccelerated motion vector"/><br>
+  Â  Â  <arg name="dy_unaccel" type="fixed"<br>
+  Â  Â  Â  Â  summary="the y component of the unaccelerated motion vector"/><br>
+  Â  </event><br>
+  </interface><br>
+<br>
+</protocol><br>
<span class=""><font color="#888888">--<br>
2.4.3<br>
<br></font></span></blockquote><div> </div></div></div><div class="gmail_extra">Mariusz Ceier</div></div>