<div dir="ltr">I'm on it. Sorry that it got lost for so loing.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 21, 2013 at 6:01 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Copying Daniel, Manuel and Reid.<div class="HOEnZb"><div class="h5"><br>
<br>
On 06/21/2013 08:27 AM, Tobias Grosser wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 05/08/2013 03:58 PM, David Blaikie wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, May 6, 2013 at 1:27 AM, Daniel Jasper <<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: djasper<br>
Date: Mon May  6 03:27:33 2013<br>
New Revision: 181187<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=181187&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=181187&view=rev</a><br>
Log:<br>
Change indentation when breaking after a type.<br>
<br>
clang-format did not indent any declarations/definitions when breaking<br>
after the type. With this change, it indents for all declarations but<br>
does not indent for function definitions, i.e.:<br>
<br>
Before:<br>
const SomeLongTypeName&<br>
some_long_variable_name;<br>
typedef SomeLongTypeName<br>
SomeLongTypeAlias;<br>
const SomeLongReturnType*<br>
SomeLongFunctionName();<br>
const SomeLongReturnType*<br>
SomeLongFunctionName() { ... }<br>
<br>
After:<br>
const SomeLongTypeName&<br>
     some_long_variable_name;<br>
typedef SomeLongTypeName<br>
     SomeLongTypeAlias;<br>
const SomeLongReturnType*<br>
     SomeLongFunctionName();<br>
</blockquote>
<br>
No judgment on how good/bad this is, but this particular case seems<br>
inconsistent with LLVM's defacto standard style. Function declarations<br>
still don't tend to have the indentation for the function name. (I'm<br>
looking at llvm/include/llvm/DIBuilder.h at the moment)<br>
</blockquote></blockquote>
<br>
<br>
This was also reported as <a href="http://llvm.org/PR16157" target="_blank">llvm.org/PR16157</a>.<br>
<br>
The change is in fact really disturbing as all code in LLVM uses the<br>
previous style and we get now a lot of noise and inconsitency in files<br>
that are formatted with the after version. Making clang-format a whole<br>
less useful for me and other people [1].<br>
<br>
This is now in since six weeks without feedback to the people that<br>
raised the very same concern right at the point when this commit<br>
happened. Daniel or Manuel, would it be possible to revert this change<br>
for LLVM style?<br>
<br>
Cheers,<br>
Tobi<br>
[1] <a href="http://permalink.gmane.org/gmane.comp.compilers.clang.scm/74975" target="_blank">http://permalink.gmane.org/<u></u>gmane.comp.compilers.clang.<u></u>scm/74975</a><br>
</div></div></blockquote></div><br></div>