<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 17, 2015 at 12:17 PM, Paul Hoad <span dir="ltr"><<a href="mailto:mydeveloperday@gmail.com" target="_blank" class="cremed">mydeveloperday@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">MyDeveloperDay added a comment.<br>
<span class=""><br>
> The point here that it this space is utterly irrelevant<br>
<br>
<br>
</span>So I totally take the point but if it doesn't matter then why does clang-format automatically add one to my "template<>" why not simply leave it alone?<br></blockquote><div><br></div><div>Because that is something else clang-format (almost) never does. Consistency is beneficial. </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> We never intended clang-format to provide control over every aspect of formatting<br>
<br>
<br>
</span>I know that when new people come along on an open source project they want to change the world, for that I apologize, we weren't here for the previous decisions, but maybe clang-format is turning into something that is different from what was originally intended, maybe it CAN become the code formatting tool for those teams out there that are not quite/can't  do it the google way.<br></blockquote><div><br></div><div>I generally agree and will gladly accept any useful formatting option. I know I have been slow at reviewing your patches and I am sorry about that. I was on vacation and had to catch on other stuff, but I hope we can make progress on those.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> We'd rather have it support a limited set of (important) style options really well<br>
<br>
<br>
</span>And that I understand, but for those of us whose style does not exactly meet one of the 5 what can WE do other than submit patches to give finer control.<br></blockquote><div><br></div><div>One of the 5?? Clang-format has *many* options by now and supports many large open-source projects and publicly available style guides. I have personally spent a lot of time on and implemented many options that are entirely irrelevant to both Google and LLVM.</div><div><br></div><div>And whether or not to add the space is something that nobody really had an opinion. Basically every project I sampled was inconsistent, not a single style guide said what to do. We looked around at the sources we could find (the style guide most heavily) and made a decision. If you had brought up good arguments that the space hurts readability (I really doubt it), you could have made us turn the other way. The point is that this space matter so little that it is not worth having the option.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I don't want to argue but I would like you (who seems to be the gatekeeper for clang-format) to consider.<br></blockquote><div><br></div><div>I have considered and my opinion stands. It is not worth having this option and it is also not worth changing that clang-format adds the space now that it has been in use by many for 2+ years.</div><div><br></div><div>I am sorry if this means you cannot use clang-format for a significant project.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://reviews.llvm.org/D12921" rel="noreferrer" target="_blank" class="cremed">http://reviews.llvm.org/D12921</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>