<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:11 PM Todd Fiala via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Dec 11, 2015 at 2:02 PM, Eugene Zelenko <span dir="ltr"><<a href="mailto:eugene.zelenko@gmail.com" target="_blank">eugene.zelenko@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi, Todd!<br>
<br>
As far as I know Clang-format doesn't leave spaces. Somebody pointed<br>
to old discussion on this matter, but I forgot it locations :-(<br>
<br>
I think will be good idea to implement automatic run of Clang-format<br>
as pre-commit hook, since LLDB code have too much diversity in code<br>
formatting for single project.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>I like the pre-commit hook idea.  Have we attempted to pursue this in the past?</div></div></div></div></blockquote><div><br></div><div>Biggest problem with commit hook is that it will format lines that you specifically don't want formatted.  See for example all the option tables in Command*.cpp files </div></div></div>