<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"><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">dwnarvaez@gmail.com</a>>:<br>
<div><div class="h5">> 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>-- <br>Daniel Narvaez<br>
</div>