[PATCH] D14484: Formatting constructor initializer lists by putting them always on different lines

Nikolai Hlubek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 21 00:29:21 PDT 2019


nhlubek added a comment.

Is it possible to discuss about this feature again?

This is only a corner case if you allow only 80 characters per line.
If you allow more characters (e.g. you use doxygen for inline formatting and want some space on the right) this quickly becomes very relevant, because then each initializer list gets very unreadable.

Currently it is even not possible, to reproduce the mock up example in the original implementation 
(only by setting the line limit at 40 or so which is a bit unreasonable).
https://reviews.llvm.org/D32479

  // When ConstructorInitializerAllOnOneLineOrOnePerLine = true:
  Constructor() :
      initializer1(),
      initializer2()
  {}


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D14484/new/

https://reviews.llvm.org/D14484





More information about the cfe-commits mailing list