<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><br></div><div>BR</div><div>imran</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 4:50 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 11:21, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-10-27 18:51 GMT+02:00 Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net" target="_blank">jstpierre@mecheye.net</a>>:<br>
<span>> Can I also suggest that we don't make this public API? These are internal<br>
> helpers for libwayland, not designed for any consumers. We've been burned by<br>
> making too much internal helper API public before.<br>
<br>
</span>+1<br>
I don't think this belongs in the wayland API at all. That means<br>
duplicating them in weston, but they will hardly need modifications<br>
anyway.<br></blockquote><div><br></div></span><div>AOL. We're a window system, not a replacement libc.</div><div><br></div><div>Cheers,</div><div>Dan </div></div></div></div>
</blockquote></div><br></div>