<div dir="ltr">Linux kernel uses a lot of function pointers. And I'm looking this solution.<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="im">On Mon, 10 Jun 2013 09:47:32 +0200<br>
Marc Chalain <<a href="mailto:marc.chalain@gmail.com">marc.chalain@gmail.com</a>> wrote:<br>
<br>
> Hi Pekka,<br>
> I agree with --disable-multitouch.<br>
> For the comparison between the use of ifdef and no-op functions. I think<br>
> the second solution is ambiguous, we disable the feature in one side and we<br>
> use the API in the other side.<br>
> I look if it's possible to have a third solution which satisfies every<br>
> body...<br>
<br>
</div>In this case, the no-op functions would never be called, anyway.<br>
<br>
#ifdefs are harder to read than function calls, when functions are well<br>
named, and live up to their names. The linux kernel favours this, too.<br>
<br>
<br>
Cheers,<br>
pq<br>
<div class="HOEnZb"><div class="h5"><br>
> 2013/6/10 Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>><br>
><br>
> > 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>
> > 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>
> ><br>
<br>
</div></div></blockquote></div><br></div>