Eugene.Zelenko added a comment.
Clang-tidy also has //modernize-use-default-member-init//. Will be good idea to
mention this check in documentation and otherwise as well as draw distinction
(C++ version, coding guidelines, etc) in use cases.
Comment at:
clang-tools-extra/cla
whisperity added inline comments.
Comment at:
clang-tools-extra/test/clang-tidy/checkers/readability-prefer-initialization-list.cpp:221
+m = 1;
+// NO-MESSAGES: initialization of m follows a conditional expression
+ }
Comment diverged from what's actual
whisperity added a comment.
Can you refresh my memory on whether a rule for "if init expr is constant,
initialise in class body instead" exists for init list members? If so, this
will be a funny "two pass needed to fix" kind of check.
Comment at:
clang-tools-extra/clang-tidy
sylvestre.ledru added a comment.
> affects positively its performance.
Interesting! Do you have some numbers you could share? thanks
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71199/new/
https://reviews.llvm.org/D71199
___
baloghadamsoftware created this revision.
baloghadamsoftware added reviewers: alexfh, aaron.ballman.
baloghadamsoftware added a project: clang-tools-extra.
Herald added subscribers: mgehre, Szelethus, rnkovacs, xazax.hun, mgorny.
Herald added a project: clang.
Finds member initializations in the c
