<div dir="ltr"><br><br><div class="gmail_quote">On Mon Feb 09 2015 at 11:56:44 AM <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I actually do think that having the space between the complex visual noise of the argument list and the function name makes it easier to detect functions when scanning code, which was why we did it this way to start.  That and from years of working on FSF code, Jason & I were used to it.  It was a hard-and-fast rule for FSF code, but then for C++ this sort of thing:<br>
<br>
target.GetProcess ().GetSelectedThread ().GetStackFrameAtIndex (0)<br>
<br>
just looks stupid.  So we don't use it there.  I have a Quixotic desire to have some not hard-and-fast rules in the coding conventions, and rather to decide based on what looks good, because "I'm a free man, dammit!".<br></blockquote><div><br></div><div>That's fine, and that's why I'm proposing just removing it from the style guide.  Because leaving it in, and saying "We do this, but actually... not everyone does, but you can do it if you want!" is kind of redundant and unnecessary.  Unfortunately clang-format, being a tool, does have hard and fast rules.  So it seems like we can just say "do it if you want, but clang-format will remove it for you, and hopefully you use clang-format (because it has many other benefits as well)"</div></div></div>