[PATCH] Ensure consecutive patch header lines
Jeremy Kerr
jk at ozlabs.org
Mon Feb 28 12:29:13 EST 2011
Hi Dirk,
> Reset the state if we can't proceed after the optional extended header
> line.
I was originally intending for patchwork to be fairly permissive with what it
accepts as part of the patch, as patch(1) is happy to ignore stuff that isn't
part of the actual patch. Looks like it's being *too* permissive though.
I think a nicer fix would be to drop the current parse buffer (ie, make it
part of the comment) if we see '^diff' again, rather than allowing for only
one line between '^diff' and '^--- '. This way we should only see entire hunks
ending up in the patch bufffer.
Cheers,
Jeremy
More information about the Patchwork
mailing list