[PATCH] D69764: [clang-format] Add Left/Right Const (East/West , Before/After) fixer capability
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 3 00:24:42 PST 2019
MyDeveloperDay marked an inline comment as done.
MyDeveloperDay added inline comments.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:1378
+**ConstStyle** (``ConstAlignmentStyle``)
+ Different ways to arrange const.
----------------
klimek wrote:
> Personally, I'm somewhat against having 3 different aliases for the options. I'd chose one, even though it doesn't make everybody happy, and move on. I'm fine with East/West as long as the documentation makes it clear what it is.
If I have to drop the other options, I think I'd want to go with East/West const as I feel it has more momentum, just letting people know before I change the code back (to my original patch ;-) )
https://www.youtube.com/watch?v=gRmI_gsNqcI
{F10954065}
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69764/new/
https://reviews.llvm.org/D69764
More information about the cfe-commits
mailing list