On Fri, 20 Jan 2017 06:10:38 -0800
Yong Bakos <j...@humanoriented.com> wrote:

> Peter,
> 
> > On Jan 11, 2017, at 4:13 PM, Peter Hutterer <peter.hutte...@who-t.net> 
> > wrote:
> > 
> > These are the protocol.xml changes from:
> > 66a26aeb2a: protocol: Remove inconsistent line breaks
> > a26ed0949e: protocol: indentation fixes
> > 6a18a87727: protocol: Extend wl_touch with touchpoint shape and orientation
> > 
> > and a few other, smaller ones.
> > 
> > Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>  
> 
> This brings the test xml files up to date for consistency, and is:
> 
> Reviewed-by: Yong Bakos <yba...@humanoriented.com>
> Tested-by: Yong Bakos <yba...@humanoriented.com>

Hi,

that's odd, after this patch I see:

$ diff -u protocol/wayland.xml tests/data/example.xml 
--- protocol/wayland.xml        2017-01-23 16:32:32.847221497 +0200
+++ tests/data/example.xml      2017-01-23 16:32:32.849221480 +0200
@@ -1987,7 +1987,7 @@
        the vertical motion of a device is converted to scroll events while
        a button is held down.
       </description>
-      <entry name="wheel" value="0" summary="a physical wheel" />
+      <entry name="wheel" value="0" summary="a physical wheel rotation" />
       <entry name="finger" value="1" summary="finger on a touch surface" />
       <entry name="continuous" value="2" summary="continuous coordinate 
space"/>
     </enum>

I know it hasn't ever been done before so there is no precedent, but
I'd just mention the commit sha in the commit message from which the
wayland.xml has been copied from.

Anyway, the difference is not significant, and it'll get corrected on
the next sync, so:
Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk>


Thanks,
pq

> > ---
> > Changes to v1:
> > - new in this series
> > 
> > tests/data/example-client.h | 112 +++++++++++++++++++--
> > tests/data/example-code.c   |  12 ++-
> > tests/data/example-server.h |  51 ++++++++--
> > tests/data/example.xml      | 231 
> > ++++++++++++++++++++++++++------------------
> > 4 files changed, 294 insertions(+), 112 deletions(-)

Attachment: pgpB4gLdOuRNr.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to