<div dir="ltr"><div><div>Boyan,<br></div>Thanks for picking this up!  I'm making some comments on the patches themselves.  Unfortunately, I don't have touch hardware to test it with, but I hope my comments are helpful none the less.<br>
</div>--Jason Ekstrand<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 6, 2014 at 9:46 PM, Boyan Ding <span dir="ltr"><<a href="mailto:stu_dby@126.com" target="_blank">stu_dby@126.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The following two patches add touch support to weston's nested wayland<br>
backend to facilitate testing. The first one adds motion support to<br>
cairo-util and the second actually add the touch listeners in the<br>
compositor.<br>
<br>
I'm new to wayland and weston and this is my first work. So any advice<br>
is welcomed.<br>
<br>
Boyan Ding (2):<br>
  cairo-util: Add frame_touch_motion support<br>
  compositor-wayland: Add touch support<br>
<br>
 shared/cairo-util.h      |  3 ++<br>
 shared/frame.c           | 24 ++++++++++++<br>
 src/compositor-wayland.c | 95 ++++++++++++++++++++++++++++++++++++++++++++++++<br>
 3 files changed, 122 insertions(+)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.9.2<br>
<br>
<br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</font></span></blockquote></div><br></div>