<div dir="ltr"><div>Daniel! </div><div><br></div><div>As per your logic, I see wl_list APIs exposed etc, which shouldn't be part of libwayland as well. <br></div><div><div>similarly, wl_fixed_to_double and wl_array shouldn't be part of the window system. Isnt it?</div><div>I can make inline functions if that helps.</div><div><br></div><div>Btw here is an API patch, which has not be reviewed till now.</div><div><a href="http://lists.freedesktop.org/archives/wayland-devel/2014-October/017833.html">http://lists.freedesktop.org/archives/wayland-devel/2014-October/017833.html</a><br></div><div><br></div><div>BR</div><div>imran</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 6:36 PM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 28 October 2014 15:40, Imran Zaman <span dir="ltr"><<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>You guys should check the reason why the patch is there rather than throwing out random thoughts or blunt comments.</div><div><br></div><div>I did this patch mainly because weston/wayland has been using strtol/strtoul functions in number of places with buggy error checks, and duplicate code everywhere. Weston and wayland go together; so in bigger picture, its a very useful patch IMO.. I hardly find any patches with proper tests, but I did it so to make it more effective. But I guess in wayland/weston community, only maintainers are allowed to push patches others are strongly discouraged to do so. I guess its better to encourage people/community for giving helping hand.</div><div><br></div><div>Anyways we will now only push patches (including multi-seat support) in Tizen weston/wayland rather than wasting time in upstreamn weston/wayland as it seems to be long bureaucratic process to overcome with virtually no success.</div></div></blockquote><div><br></div></span><div>That's not what we've said. No-one said 'don't take the patch'; all we said is 'please don't expose it as part of libwayland-*'s _public_ API'.</div><div><br></div><div>I like the idea of the patch, I like how it's caught a number of buggy spots where we've open-coded the same thing, and I like that it's well-tested. For internal usage, it's great! I just don't want to expose string manipulation functions in the public API of a window system.</div><div><br></div><div>You're right that the test infrastructure is in a pretty bad state. Anything which helps that is more than welcome, and you may have seen a bunch of patches from Derek Foreman (not a maintainer) on this list, which are progressing well and go a long way towards improving our test suite into something that will be really useful day to day. Any further contributions along those lines - from anyone - are totally welcome.</div><div><br></div><div>As for your multiseat patches, the last discussions I remember involved some pretty fundamental disagreements about the whole architecture, particularly input support. I haven't seen any more patches or discussion since the last IRC chat, though.</div><div><br></div><div>Cheers,</div><div>Daniel</div><div><br></div></div></div></div></blockquote></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 6:36 PM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 28 October 2014 15:40, Imran Zaman <span dir="ltr"><<a href="mailto:imran.zaman@gmail.com" target="_blank">imran.zaman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>You guys should check the reason why the patch is there rather than throwing out random thoughts or blunt comments.</div><div><br></div><div>I did this patch mainly because weston/wayland has been using strtol/strtoul functions in number of places with buggy error checks, and duplicate code everywhere. Weston and wayland go together; so in bigger picture, its a very useful patch IMO.. I hardly find any patches with proper tests, but I did it so to make it more effective. But I guess in wayland/weston community, only maintainers are allowed to push patches others are strongly discouraged to do so. I guess its better to encourage people/community for giving helping hand.</div><div><br></div><div>Anyways we will now only push patches (including multi-seat support) in Tizen weston/wayland rather than wasting time in upstreamn weston/wayland as it seems to be long bureaucratic process to overcome with virtually no success.</div></div></blockquote><div><br></div></span><div>That's not what we've said. No-one said 'don't take the patch'; all we said is 'please don't expose it as part of libwayland-*'s _public_ API'.</div><div><br></div><div>I like the idea of the patch, I like how it's caught a number of buggy spots where we've open-coded the same thing, and I like that it's well-tested. For internal usage, it's great! I just don't want to expose string manipulation functions in the public API of a window system.</div><div><br></div><div>You're right that the test infrastructure is in a pretty bad state. Anything which helps that is more than welcome, and you may have seen a bunch of patches from Derek Foreman (not a maintainer) on this list, which are progressing well and go a long way towards improving our test suite into something that will be really useful day to day. Any further contributions along those lines - from anyone - are totally welcome.</div><div><br></div><div>As for your multiseat patches, the last discussions I remember involved some pretty fundamental disagreements about the whole architecture, particularly input support. I haven't seen any more patches or discussion since the last IRC chat, though.</div><div><br></div><div>Cheers,</div><div>Daniel</div></div></div></div>
</blockquote></div><br></div>