Re: I'd like to rearrange the stream_t API slightly

[Sam Roberts]

Quoting Alain Magloire <address@hidden>, who wrote:
> > Allo!
> 
> Bonjour,
>   under the snow, and freezing rain 8(.

That sounds worse than Sept Iles!

> Busy .. Busy

Well.. sounds interesting at least.

> One of the problem with using stdin, is that you can not seek.
> You can get away with this if you use a temporary message_t;
> message_create (&msg, NULL); But the parsing for mailbox insists
> that the stream be seekable.

Are you sure? I'm doing OK now with an unseekable stream.

> And it seems to that your trying to use stdin directly, it
> may be simpler to use a temporary stream/file to isolate.

If I have to, I will.

> The original idea was that stream_t is an interface or virtual class,
> it does not carry an implementation.  The idea is more enforced
> in mailbox2 where there is no stream_create () but rather if
> you want an object that has the signature of stream_t
> you'll have to get it from a factory:
>       stream_file_create (&stream, ...)
>       stream_stdio_create (&stream, ...)

These don't take enough information in them, even in mailbox2, so
you have to pass info to the stream_open() function, which makes
it no longer generic.

Julie had a half-hour meeting when I was in Sept Iles, so attached
is proposal #2.

>       stream_tcp_create (&stream, ...)
>       stream_fd_create (&stream, ...)
>       stream_buffer_create (&stream, ...)


> > ps1 I've implemented the sendmail mailer as described in the docs
> > I committed, next is smtp, but I wanted some comment on how to
> > approach this stream issue as I write/test the smtp mailer.
> 
> There is a sendmail.c and smtp.c that you may want to look
> at first.  The annoying thing with the old code was to
> keep the state to be async safe, a real pain.

I read them, I'm modifying them.

> Is smtp mean the nullmailer that you propose last time?

No, I mean that the mailer_t API now forces the user into sendmail "-t" mode,
making it impossible to bounce messages or send DSNs.

The mailer.texi I committed describes the interface change I
need, I'm modifying sendmail.c and smtp.c to implment those
changes. sendmail.c is done. smtp.c is harder, I will work on it
tonight.

> Using SMTP on port 25(default) to deliver mail to a mailhub,
> no MX lookup was involved right?

Different utility.... I've about 9 on my wishlist... and no time.

Cheers.

Sam

-- 
Sam Roberts <address@hidden> (Vivez sans temps mort!)

_stream.txt
Description: Text document