<div dir="ltr"><div>I don't see how this belongs in libwayland.  Sure, we use strtol twice, but I don't think that warrants adding 100 lines of wrapper functions and test cases.<br></div>--Jason Ekstrand<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 15, 2014 at 6:16 AM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 2014-10-15 16:14, Imran Zaman a écrit :<span class=""><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi<br>
<br>
The patch is used to replace strtol and strtoul with wl_strtol and<br>
wl_strtoul with inputs and result checks.<br>
</blockquote>
<br></span>
I don't know where Wayland developers stand on this, but I would rather the client library function calls not clobber errno to zero.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Rémi Denis-Courmont<br>
______________________________<u></u>_________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.<u></u>freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/wayland-devel</a><br>
</font></span></blockquote></div><br></div>