<div dir="ltr"><div><div><div>Hi Pekka,<br>I agree with --disable-multitouch.<br></div>For the comparison between the use of ifdef and no-op functions. I think the second solution is ambiguous, we disable the feature in one side and we use the API in the other side.<br>
</div>I look if it's possible to have a third solution which satisfies every body...<br></div>Marc.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/10 Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, 7 Jun 2013 12:11:14 +0200<br>
Marc Chalain <<a href="mailto:marc.chalain@gmail.com">marc.chalain@gmail.com</a>> wrote:<br>
<br>
> mtdev is not necessary on some device (desktop or set-top-box). For<br>
> embedded solution the minimum of dependencies is required. "configure"<br>
> accepts the --disable-mtdev option.<br>
<br>
</div></div>Hi Marc,<br>
<br>
with this patch, if weston is compiled with --disable-mtdev, it looks<br>
like it will still happily open multitouch devices, expose them to<br>
Wayland clients, and feed them garbage or fail or whatever. Right?<br>
<br>
I think the proper approach is to change the configure option into<br>
--disable-multitouch or --disable-touchscreen, which would not only<br>
drop the dependency to mtdev, but also make Weston ignore all<br>
touchscreen devices since it will not be able to handle them.<br>
<br>
As the cherry on top, when a touchscreen device gets ignored during<br>
input device detection, it would be nice to print a log message saying<br>
it was ignored because touchscreen support was not built, instead of<br>
the usual log message.<br>
<br>
Also, there are not that many mtdev functions, so to minimize the<br>
number of #ifdefs is the code, you could define no-op functions for a<br>
few mtdev entry points when the real mtdev is not used.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div>