On Sun, Jun 19, 2011 at 5:29 AM, jdrewsen <[email protected]> wrote:
> Very nice. A couple of things I believe would help: > 1, Get rid of MailMessageData and use curl.onSend() and a delegate that > keeps a reference to the message. That way you don't have to use the lower > level Curl.set(infile/readfunction) calls as well. Ah, that makes it much better. > 2, It would be nice if the static sendMail(...) function worked like the > Http/Ftp counterparts. They return a Result object that you can change > before performing the actual task. That way you can easily set timeouts etc. > If there shouldn't be support for async smtp then this is probably not > important though. There should be support for async SMTP. The problem is this: SMTP.sendMailAsync(...).connectTimeout(dur!"seconds"(60)).localPort(25).? byLine, byChunk, etc don't make much sense there. I think it would be better to get rid of the static sendMail function, and write a performAsync method. I'm not sure though. Something like this: https://gist.github.com/1034433
