From a246830d3feaab033a16b2332716cff05a7ec92c Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Thu, 24 Oct 2019 08:34:25 +0200 Subject: This is fixed now on Clang trunk ...since "Include leading attributes in DeclStmt's SourceRange" Change-Id: I21859dfdcb454a9dd4444ce299548bf75043b1ab Reviewed-on: https://gerrit.libreoffice.org/81424 Tested-by: Jenkins Reviewed-by: Stephan Bergmann --- compilerplugins/clang/indentation.cxx | 10 +++++++--- compilerplugins/clang/test/indentation.cxx | 9 +++++++++ 2 files changed, 16 insertions(+), 3 deletions(-) (limited to 'compilerplugins/clang') diff --git a/compilerplugins/clang/indentation.cxx b/compilerplugins/clang/indentation.cxx index 57fbd1b382a0..1b94eac3ec1a 100644 --- a/compilerplugins/clang/indentation.cxx +++ b/compilerplugins/clang/indentation.cxx @@ -120,9 +120,12 @@ bool Indentation::VisitCompoundStmt(CompoundStmt const* compoundStmt) // these are always weirdly indented if (isa(stmt)) continue; - // At least until Clang 9.0.0, getBeginLoc of a VarDecl DeclStmt with an UnusedAttr points - // after the attr (and getLocation of the attr would point at "maybe_unused", not at the - // leading "[["), so just ignore those at least for now: +#if CLANG_VERSION < 100000 + // Before + // + // "Include leading attributes in DeclStmt's SourceRange", getBeginLoc of a VarDecl DeclStmt + // with an UnusedAttr pointed after the attr (and getLocation of the attr pointed at + // "maybe_unused", not at the leading "[["), so just ignore those in old compiler versions: if (auto const declStmt = dyn_cast(stmt)) { if (declStmt->decl_begin() != declStmt->decl_end()) @@ -136,6 +139,7 @@ bool Indentation::VisitCompoundStmt(CompoundStmt const* compoundStmt) } } } +#endif auto stmtLoc = compat::getBeginLoc(stmt); diff --git a/compilerplugins/clang/test/indentation.cxx b/compilerplugins/clang/test/indentation.cxx index 3a76c2c97e2f..27e858fb319c 100644 --- a/compilerplugins/clang/test/indentation.cxx +++ b/compilerplugins/clang/test/indentation.cxx @@ -7,6 +7,8 @@ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ +#include "config_clang.h" + int foo(); int foo2(int); @@ -49,4 +51,11 @@ void attr() { foo(); } +#if CLANG_VERSION >= 100000 +void attr_bad() { + [[maybe_unused]] int i = foo(); // expected-note {{measured against this one [loplugin:indentation]}} + foo(); // expected-error {{statement mis-aligned compared to neighbours [loplugin:indentation]}} +} +#endif + /* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */ -- cgit v1.2.3