diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2014-07-31 09:58:52 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2014-07-31 09:58:52 +0000 |
commit | f97701d490f91403b66427b5730074b9d7428395 (patch) | |
tree | 635ed77e7779b91d2917af0ec8c06704d73a8df5 /clang-query | |
parent | 866dd4699ef71322ea8280910c4566fc82d688d7 (diff) |
[clang-tidy] Add a checker for code that looks like a delegate constructors but doesn't delegate.
Summary:
class Foo {
Foo() {
Foo(42); // oops
}
Foo(int);
};
This is valid code but it does nothing and we can't emit a warning in clang
because there might be side effects. The checker emits a warning for this
pattern and also for base class initializers written in this style.
There is some overlap with the unused-rtti checker but they follow different
goals and fire in different places most of the time.
Reviewers: alexfh, djasper
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D4667
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@214397 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'clang-query')
0 files changed, 0 insertions, 0 deletions