diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-11-14 19:17:07 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-11-15 07:49:28 +0100 |
commit | cab6e6836973a9ddfc5ed9df757e07138328c1c3 (patch) | |
tree | e8f5f6036897f3564ea6aecd777b8ba692ee6087 /compilerplugins/clang/test/unnecessaryoverride.cxx | |
parent | 4283092eb219968870e23bd0d600e1ef521dab89 (diff) |
Make checkIdenticalDefaultArguments more precise
...when creating objects involves copy/move constructors
Change-Id: I0c7ccb85b7dcb584502a48817d7d2abfde25aaf2
Reviewed-on: https://gerrit.libreoffice.org/44733
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins/clang/test/unnecessaryoverride.cxx')
-rw-r--r-- | compilerplugins/clang/test/unnecessaryoverride.cxx | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/compilerplugins/clang/test/unnecessaryoverride.cxx b/compilerplugins/clang/test/unnecessaryoverride.cxx index 7abcf971986f..1f5c1f9fb4a7 100644 --- a/compilerplugins/clang/test/unnecessaryoverride.cxx +++ b/compilerplugins/clang/test/unnecessaryoverride.cxx @@ -94,6 +94,7 @@ struct Base2 { void default1(Base const& = SimpleDerived()); void default2(Base const& = SimpleDerived()); + void default3(Base = Base()); }; struct Derived2 : Base2 @@ -105,6 +106,12 @@ struct Derived2 : Base2 { Base2::default2(x); } + void + default3( // expected-error {{public function just calls public parent [loplugin:unnecessaryoverride]}} + Base x = Base()) + { + (Base2::default3(x)); + } }; /* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */ |