Code review comment for lp:~mgorven/ibid/smtp-fixes

Revision history for this message
Michael Gorven (mgorven) wrote :

On Thursday 07 January 2010 10:15:25 Michael Gorven wrote:
> On Thursday 07 January 2010 00:23:45 Jonathan Hitchcock wrote:
> > Line 40: isn't "References" supposed to be the previous mail's
> > References + the previous mail's Message-ID?
>
> Technically yes, but then you also need proper length handling and whatnot.
> Since most of the time the bot will be responding to the first message in a
> thread it would hardly be used.

r837. I haven't done the length handling -- RFC 2822 doesn't require it.

« Back to merge proposal