<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jan 5, 2014 at 3:58 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank" class="cremed">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On 01/05/2014 02:13 PM, Daniel Jasper wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Tobias, could you submit the two small formatting changes that are<br>
required? I am not entirely certain whether it would be ok for me to just<br>
submit patches to polly. Also I don't think it would be a good idea for<br>
clang-format developers to be required to ensure other projects' adherence<br>
to what clang-format deems to be correct style.<br>
</blockquote>
<br>
Hi Daniel,<br>
<br>
thanks for letting me know. You are not required to fix Polly, but if there is an obvious and easy fix, I would be more than happy if you do so (it also keeps the buildbot noise down). For non-obvious changes I would prefer a short pre-commit mail (or just an headsup as today). However, if the style change is non-obvious it would be great if you could make it off by default for LLVM and give a short headsup on the mailing list such that I can prepare a patch for Polly.<br>

<br>
Regarding the change today, I wonder how you figured out that the new formatting is 'the predominant choice' in LLVM/clang? I just checked what formatting is more common in LLVM/clang by counting the one line enums in the current LLVM and clang code and after running the latest clang-format over all .cpp and .h files.<br>

<br>
I use the following grep call to count:<br>
<br>
grep -R 'enum.*{.*}.*;' tools/clang/lib/ lib/ include/ \<br>
                        tools/clang/include | wc<br>
<br>
I get 259 cases for the current formatting and 531 for the reformatted code, which means 272 cases use currently a multi line formatting.<br></blockquote><div><br></div><div>I did a slightly different analysis. I searched for "enum.*}" and there seem to be 432 files with in LLVM/Clang containing these. This compares to a search (with multiline regexs) for "\n\ *enum[^\}]{0,73}\n\}" of which I find 80 files containing them. The latter should be all enums that can be written on a single line but aren't.</div>
<div><br></div><div>Without paths containing "test" these numbers change to 219 vs. 47 files. The 5:1 ratio seems quite consistent. The regular expressions might have errors, I only hacked at them until I was satisfied and the individual findings were what I was looking for.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
It seems the one line formatting is actually slightly less common in the LLVM/clang files. I also don't believe that the new style buys us anything in readability.<br></blockquote><div><br></div><div>It still makes sense to build clang-format close to the more frequent style. Plus, the people I talked to before-hand (Manuel, Alex and a few other users prefer single lines).</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I would prefer clang-format to remain consistent between versions,<br>
as long as there is no good reason to break this consistency.</blockquote><div><br></div><div>I think the above (preferences of users + more similarity with existing code) constitutes a good reason.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Consequently, I wonder if it would not make sense to use the old style in LLVM mode?<br></blockquote><div><br></div><div>a) Not from the numbers I had. We can do a more thorough analysis, though.</div><div>b) If there is an even split inside LLVM (as your numbers would suggest) then I'd prefer to have fewer options.</div>
<div><br></div><div>Btw. this is not only Google Style. GNU style has this explicitly written down, in WebKit it seems to be common.</div><div><br></div><div>Cheers,</div><div>Daniel</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Cheers,<br>
Tobias<br>
</blockquote></div><br></div></div>