[Libreoffice-commits] core.git: compilerplugins/clang

Stephan Bergmann sbergman at redhat.com
Fri Jun 17 11:58:49 UTC 2016


 compilerplugins/clang/override.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 793fd7e54856216bd58e1d402f63cec208291019
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Fri Jun 17 13:56:58 2016 +0200

    loplugin:override: No more need to request dtor be marked 'virtual'
    
    ...rather than 'override', cf. <https://gerrit.libreoffice.org/#/c/26347/>
    "loplugin:override: No more need for the "MSVC dtor override" workaround".
    
    Change-Id: Ib528938ed4be248117bb7114f34d4e9421bc3275

diff --git a/compilerplugins/clang/override.cxx b/compilerplugins/clang/override.cxx
index 155f4ce..69155d4 100644
--- a/compilerplugins/clang/override.cxx
+++ b/compilerplugins/clang/override.cxx
@@ -67,7 +67,7 @@ bool Override::VisitCXXMethodDecl(CXXMethodDecl const * decl) {
             report(
                 DiagnosticsEngine::Warning,
                 ("overriding destructor declaration not explicitly marked"
-                 " 'virtual'"),
+                 " 'override'"),
                 decl->getLocation())
                 << decl->getSourceRange();
         }


More information about the Libreoffice-commits mailing list