<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 23, 2017 at 5:01 PM George Karpenkov via Phabricator via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">george.karpenkov added a comment.<br>
<br>
@dcoughlin the context I was thinking about is that if everyone consistently runs `clang-format` (if we want that), then we never would have discussion.<br>
The alternative is that every run of `clang-format` would be followed by manually reverting changes which were introduced by mistake (in this case, because the file was moved).<br></blockquote><div><br>clang-format has script (git-clang-format) for formatting only a diff, use that or something like it so you're not reformatting unrelated lines. If you're changing so much of the file, or it's so malformatted that local format updates are going to be really inconsistent, reformat the entire file in a standalone commit first, then submit your semantic changes separately.<br><br>(also there's a way to setup clang-format to "format changed lines on-save" in vim, which is what I use - now I basically don't think about formatting :) )<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D39208" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39208</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>