<div dir="ltr">I'm sending patches to fix all of these things up in the current code.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 1 June 2013 13:50, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@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><div>Another couple of related things<br><br></div>* We should always pass an Error object to the callback, not a string.<br>
</div>* The name of the callback in the functions signature should just be "callback".<br>
* I also think there should only be one callback per function, need to change a bit the ds API to avoid multiple callbacks.<br></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On 31 May 2013 14:24, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2013/5/30 Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>>:<br>
<div><div>> Hello,<br>
><br>
> in the async API we have written so far we have not been handling errors<br>
> very consistently. I think we should just adopt the nodejs convention. The<br>
> first parameter passed to all the async callbacks is an error or null if the<br>
> call succeeded.<br>
<br>
</div></div>I have seen this convention used in other JS projects too, so let's go<br>
ahead with it.<br>
<br>
--<br>
.. manuq ..<br>
</blockquote></div><br><br clear="all"><br></div></div><span class="HOEnZb"><font color="#888888">-- <br>Daniel Narvaez<br>
</font></span></div>
</blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div>