<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 11, 2015 at 2:10 PM, Todd Fiala <span dir="ltr"><<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Dec 11, 2015 at 2:04 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">No, sadly.  We still have 4 space indentation, allman style braces, 120 line character limit, return-type-on-separate-line, and a <a href="http://lldb.llvm.org/lldb-coding-conventions.html" target="_blank">few other differences</a>.  I've tried on a couple of occasions to remove some of these differences and move closer to LLDB's style, but it has never gone over well.  I plan to keep trying until I'm dead</div></blockquote><div><br></div></span><div>Eek, hopefully we can solve that before then!</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"> or working on something else, </div></blockquote><div><br></div></span><div>I vote for us fixing it rather than that :-)</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">so if there's anything you can do to help, it would be great :)</div></blockquote><div><br></div></span><div>Okay.  I thought I heard rumor we were adopting LLVM coding style.  </div></div></div></div></blockquote><div><br></div><div>Er, rather that we already did.</div><div> </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"><div>I'll find out what our position is on this.  I just thought the LLVM standard did include the spaces, but only because of the prolific use of it I saw over here.</div><div><br></div><div>I'll do some digging here.</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:01 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.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_extra"><div class="gmail_quote">On Fri, Dec 11, 2015 at 1:59 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><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"><span><div dir="ltr">On Fri, Dec 11, 2015 at 1:55 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">Hey Eugene and Greg,<div><br></div><div>I thought we were doing spaces before the open parens in places like this:</div><div><br></div><div></div></div><div dir="ltr"><div><span style="font-size:13px">-    BreakpointResolverSP resolver_sp(new BreakpointResolverFileLine (NULL,</span><br style="font-size:13px"></div></div><div dir="ltr"><div><span style="font-size:13px">...</span></div><div><span style="font-size:13px">+    BreakpointResolverSP resolver_sp(new BreakpointResolverFileLine(</span><span style="font-size:13px">nullptr,</span><br style="font-size:13px"></div><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">(see the removal of the space after BreakpointResolverFileLine from the clang-tidy settings I presume).</span></div><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">Did I misunderstand that?</span></div></div></blockquote><div><br></div></span><div>This was officially removed from the coding standard some months ago,</div></div></div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Okay.  Are we 100% in sync with LLVM coding standard guidelines?  If so I can just look there to see what we're supposed to be doing.</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </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> but not everyone has adopted this unfortunately.  See r228860.  It pains me to no end that we differ from LLVM, because it leads to exactly these type of problems where people aren't sure what the exact set of rules are.</div></div></div>
</blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">-Todd</div></div>
</div></div></blockquote></div>
</div></div></blockquote></span></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">-Todd</div></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div></div>