<div dir="ltr">It would be a great start if we could just get everyone to actually run clang-format manually.  There are two outstanding issues with clang-format that dont' support LLDB style.  Those two issues would go away if: 1) LLDB abandoned those styles and adoped LLVM's style (preferred), or 2) I fix clang-format to support them.  I keep promising to do #2, but I haven't yet because it seems like I'm the only using clang-format anyway.  If there were assurances that more people would start using it, I would re-prioritize.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:12 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</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_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" target="_blank">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></div><div dir="ltr"><div class="gmail_quote"><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></blockquote></div>