[PATCH v2 wayland 1/5] tests: sync example.xml with wayland.xml

Pekka Paalanen ppaalanen at gmail.com
Mon Jan 23 14:40:52 UTC 2017


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

> Peter,
> 
> > On Jan 11, 2017, at 4:13 PM, Peter Hutterer <peter.hutterer at 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.hutterer at who-t.net>  
> 
> This brings the test xml files up to date for consistency, and is:
> 
> Reviewed-by: Yong Bakos <ybakos at humanoriented.com>
> Tested-by: Yong Bakos <ybakos at 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.paalanen at 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(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20170123/be5dccba/attachment.sig>


More information about the wayland-devel mailing list