<div dir="ltr"><div class="gmail_quote"><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">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><div>This was officially removed from the coding standard some months ago, 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>