<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Oct 30, 2017, at 8:40 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Mon, Oct 23, 2017 at 5:01 PM George Karpenkov via Phabricator via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:<br class=""></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 class="">
<br class="">
@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 class="">
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 class=""></blockquote><div class=""><br class="">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 class=""></div></div></div></div></blockquote><div><br class=""></div><div>But that’s what I was using: trouble is, the file got renamed, and clang-format then decided to reformat all of it.</div><div>(and another question is whether it is a reasonable thing to do)</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><div class=""><br class="">(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 class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
<br class="">
<a href="https://reviews.llvm.org/D39208" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/D39208</a><br class="">
<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
cfe-commits mailing list<br class="">
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" class="">cfe-commits@lists.llvm.org</a><br class="">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="">
</blockquote></div></div>
</div></blockquote></div><br class=""></body></html>