On Sun, Jun 19, 2011 at 3:03 PM, jdrewsen <span dir="ltr"><<a href="mailto:jdrewsen@nospam.com">jdrewsen@nospam.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Den 19-06-2011 17:57, Jimmy Cao skrev:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sun, Jun 19, 2011 at 5:29 AM, jdrewsen <<a href="mailto:jdrewsen@nospam.com" target="_blank">jdrewsen@nospam.com</a><div class="im"><br>
<mailto:<a href="mailto:jdrewsen@nospam.com" target="_blank">jdrewsen@nospam.com</a>>> wrote:<br>
<br></div><div class="im">
    Very nice. A couple of things I believe would help:<br>
    1, Get rid of MailMessageData and use curl.onSend() and a delegate<br>
    that keeps a reference to the message. That way you don't have to<br>
    use the lower level Curl.set(infile/readfunction) calls as well.<br>
<br>
<br></div><div class="im">
Ah, that makes it much better.<br>
<br></div><div class="im">
    2, It would be nice if the static sendMail(...) function worked like<br>
    the Http/Ftp counterparts. They return a Result object that you can<br>
    change before performing the actual task. That way you can easily<br>
    set timeouts etc. If there shouldn't be support for async smtp then<br>
    this is probably not important though.<br>
<br>
<br></div><div class="im">
There should be support for async SMTP.<br>
The problem is this:<br>
SMTP.sendMailAsync(...).<u></u>connectTimeout(dur!"seconds"(<u></u>60)).localPort(25).?<br>
<br>
byLine, byChunk, etc don't make much sense there.<br>
<br>
I think it would be better to get rid of the static sendMail function,<br>
and write a performAsync method.<br>
I'm not sure though.<br>
Something like this:<br>
<a href="https://gist.github.com/1034433" target="_blank">https://gist.github.com/<u></u>1034433</a><br>
</div></blockquote>
<br>
Maybe i doesn't make sense to provide the async interface at all? Users needing to do it async could just as well create a delegate and call spawn themselves.<br>
<br>
/Jonas<br>
</blockquote></div><br><div>Yep.</div><div>I think there isn't a need for a static convenience function either.</div>