[PATCH] Adds IndentationWidth to FormatStyle (clang-format)

Manuel Klimek klimek at google.com
Mon May 13 06:23:40 PDT 2013


Implemented in r181690. Sorry I missed your patch, but testing this was
actually harder than it looks, and your patch missed one place where the
indent width was used. Also, the implemented solution is aligned with our
strategy to put all options into configuration files instead of creating
loads of command line options.


On Sun, May 5, 2013 at 7:44 PM, Nitram Aw <nitramaw at gmail.com> wrote:

> Clang-format seems to be hard coded to use two space width indentation.
> To make it useful to a broader audience I suggest getting rid of this
> limitation
> by letting it be part of the FormatStyle.
>
> I added IndentationWidth to FormatStyle.
> I used IndentationWidth instead of 2 the places that seem most important.
> I added -iw as a switch to clang-format to override selected style.
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130513/ea3f23a1/attachment.html>


More information about the cfe-commits mailing list