[PATCH 04/10] bin/parsemail: fix wrong parsing of diff comments

Stephen Finucane stephenfinucane at hotmail.com
Fri Sep 2 04:35:37 AEST 2016


On 31 Aug 10:04, Daniel Axtens wrote:
> > On Tue, 2016-08-23 at 17:03 +0800, WEN Pingbo wrote:
> >> If the subject of a submission is prefixed by 'Re:', then it can't be
> >> a patch or cover letter.
> >> 
> > Some (very stupid) email clients, I think some versions of Outlook or
> > Outlook Express, internationalize/localize the "Re:" string [1].
> >
> > Should we attempt to handle that here as well?
> >
> > johannes
> >
> > [1] which leads to stupid prefixing when somebody I email with uses the
> > German version - "AW: Re: AW: Re: AW: Re: [...]" since it's not even
> > smart enough to detect the "Re:" and replace it with "AW:" or similar.
> >
> > I have another thread here that just kept growing "AW:" since it *did*
> > remove the "Re:", but *didn't* find the existing "AW:" afterwards, that
> > was with "Microsoft Outlook 15.0".
> 
> I suppose we could have a configurable list of these stored either in
> code or in the database... Let's get basic Re: support in and then grow
> it from there.

Agreed. If you could follow this up with a patch that added additional
prefixes, I'd be more than happy to accept it.

Stephen


More information about the Patchwork mailing list