[cfe-commits] r170337 - in /cfe/trunk: lib/Format/Format.cpp unittests/Format/FormatTest.cpp

Daniel Jasper djasper at google.com
Mon Dec 17 14:43:20 PST 2012


I can. Although for most of the patches so far, it would be

>>>
We used to format
...
as
<horrible mess>
but now we format it as
<what you can see in the tests>
<<<

Which does not really help.. ;-)


On Mon, Dec 17, 2012 at 11:01 PM, Sean Silva <silvas at purdue.edu> wrote:

> In the future, in your commit messages, could you include something like:
>
> >>>
> We used to format
> ...
> as
> ...
> but now we format it as
> ...
> <<<
>
> That way, it's easier to understand your changes. Your test cases only
> show the "but now we format it as ..." part.
>
> -- Sean Silva
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121217/70a5f2a2/attachment.html>


More information about the cfe-commits mailing list