[PATCH] clang-format-diff: Don't reformat unmodified lines
djasper at google.com
Wed Oct 2 07:02:26 PDT 2013
Submitted as r191820.
I absolutely agree that tests would be nice, but I have not gotten around
to writing them yet (and probably won't for several more weeks) :-(.
Thanks for working on this!
On Wed, Oct 2, 2013 at 11:14 AM, Alp Toker <alp at nuanti.com> wrote:
> clang-format-diff incorrectly modifies unchanged lines due to an error
> in diff parsing.
> The unified diff format has a default line change count of 1, and 0 may
> be specified to indicate that no lines have been added. This patch
> updates the parser to accurately reflect the diff specification.
> This also has the benefit of stabilising the operation so it will
> produce the same output when run multiple times on the same changeset,
> which was previously not the case.
> No tests added because this script is not currently tested (though we
> should look into that!)
> the browser experts
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits