[PATCH] parser: Don't extract diffs from replies
Stephen Finucane
stephen at that.guru
Sat Apr 15 10:35:41 AEST 2017
On Sun, 2017-04-09 at 16:23 +0100, Stephen Finucane wrote:
> In '2a915efd', a check was added to ensure mails prefixed with 'RE:'
> or
> similar would not be parsed as patches. By time this check actually
> happens, any patches had already been extracted from the mail thus
> these
> patches were re-added to the mail content before saving the comment.
> Unfortunately, this didn't take into account cases where a patch or
> diff
> was not the last part of a mail but rather located somewhere in the
> middle of the content:
>
> Introduction content
> Diff or patch content ***
> Additional content
>
> This would result in mangling of the mail as the patch would _always_
> be
> appended to the end:
>
> Introduction content
> Additional content
> Diff or patch content ***
>
> Handle this by only breaking a mail into a comment and a diff if
> there
> is any possibility that we might want to use that diff.
>
> Signed-off-by: Stephen Finucane <stephen at that.guru>
> Fixes: 2a915efd ("parser: fix wrong parsing of diff comments")
> Closes-bug: #95
> Cc: Ralf Baechle <ralf at linux-mips.org>
I'm pretty confident with this fix. Applied so I can work on finishing
touches for 2.0.
Stephen
More information about the Patchwork
mailing list