<div dir="ltr">Please read <a href="http://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options">http://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options</a><div><br></div><div>Does your style option qualify?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Nov 8, 2015 at 8:04 AM, JVApen <span dir="ltr"><<a href="mailto:jvapen@gmail.com" target="_blank">jvapen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JVApen created this revision.<br>
JVApen added a reviewer: djasper.<br>
JVApen added a subscriber: cfe-commits.<br>
JVApen set the repository for this revision to rL LLVM.<br>
Herald added a subscriber: klimek.<br>
<br>
Hi all,<br>
<br>
I've been playing around with the clang for a while now and really enjoy it. Unfortunately clang-format does not yet do what I like it to do, so I started hacking it. So here is my first successful attempt to get something working.<br>
<br>
The issue: ConstructorInitializerAllOnOneLineOrOnePerLine only works if 'If the constructor initializers don’t fit on a line', while I prefer it to always work. In other words, I use the following formatting:<br>
<br>
```<br>
Constructor()<br>
  : a(a)<br>
  , b(b)<br>
```<br>
<br>
Since everyone can benefit from upstreaming, I like to share my changes and get some feedback.<br>
Here is already some of the stuff which I was uncertain about:<br>
<br>
  - Should I keep ConstructorInitializerAllOnOneLineOrOnePerLine or rename it (currently the second one)<br>
  - How to name the values, currently: Compact (old: false), BestFit (old: true), OnePerLine (new)<br>
  - Is the back-ward compatibility in ScalarEnumerationTraits a good idea? (On rename most likely not)<br>
<br>
JVApen<br>
<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14484" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14484</a><br>
<br>
Files:<br>
  docs/ClangFormatStyleOptions.rst<br>
  include/clang/Format/Format.h<br>
  lib/Format/ContinuationIndenter.cpp<br>
  lib/Format/Format.cpp<br>
  lib/Format/TokenAnnotator.cpp<br>
  unittests/Format/FormatTest.cpp<br>
<br>
</blockquote></div><br></div>