<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 22, 2014 at 2:52 PM,  <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank" class="cremed">jingham@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">But touching a file doesn't mean you should also "fix up" its formatting with such a tool.  As long as THAT is part of our coding conventions, I guess I can't object.</blockquote>
</div><br>From my experience in Clang and LLVM the only way folks go in and do extensive reformatting is when they're about to make absolutely *sweeping* changes to the code. As in, rewrite the overwhelming majority of it. At that point, doing more comprehensive updates to the formatting makes this more consistent and well laid out in almost every case. This typically isn't a blind application of the tool either, it is just a concerted effort to bring all of the formatting in the file into a state that will be consistent with the subsequent extensive changes.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">And even then, they usually do it *first* so that those commits are clearly known to be no-functionality-changed and the diffs which contain their substantial rewrite can be reviewed reasonably without any noise from reformatting code (yuck).</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I don't know if or when that would ever be relevant for LLDB.</div></div>