diff options
author | Sergio Martins <smartins@kde.org> | 2017-10-05 00:56:26 +0100 |
---|---|---|
committer | Sergio Martins <iamsergio@gmail.com> | 2017-10-05 00:56:26 +0100 |
commit | d64c73841f73cd68865ce88bfd18336f1954b3a7 (patch) | |
tree | edc371789b001158920a3e5e76696ccf5a4698e1 | |
parent | 5631e0772f65ddac821bfe1571dc1c74a9f3c147 (diff) | |
parent | 5e8c4a8647179b056b1cee0600f900987ebdc889 (diff) |
Merge branch '1.2' into master
-rw-r--r-- | src/checks/level0/unused-non-trivial-variable.cpp | 1 | ||||
-rw-r--r-- | tests/unused-non-trivial-variable/main.cpp | 4 | ||||
-rw-r--r-- | tests/unused-non-trivial-variable/other.h | 1 |
3 files changed, 5 insertions, 1 deletions
diff --git a/src/checks/level0/unused-non-trivial-variable.cpp b/src/checks/level0/unused-non-trivial-variable.cpp index 1450224b..a05ee2a3 100644 --- a/src/checks/level0/unused-non-trivial-variable.cpp +++ b/src/checks/level0/unused-non-trivial-variable.cpp @@ -90,6 +90,7 @@ void UnusedNonTrivialVariable::handleVarDecl(VarDecl *varDecl) return; SourceLocation locStart = varDecl->getLocStart(); + locStart = sm().getExpansionLoc(locStart); auto declRefs = HierarchyUtils::getStatements<DeclRefExpr>(body, &sm(), locStart); auto pred = [varDecl] (DeclRefExpr *declRef) { diff --git a/tests/unused-non-trivial-variable/main.cpp b/tests/unused-non-trivial-variable/main.cpp index 8bb50d94..12febdab 100644 --- a/tests/unused-non-trivial-variable/main.cpp +++ b/tests/unused-non-trivial-variable/main.cpp @@ -3,7 +3,7 @@ #include <QtCore/QVector> #include <QtCore/QByteArray> #include <QtCore/QRect> - +#include "other.h" @@ -50,4 +50,6 @@ void test4() QVector<int> v; //Warn QByteArray b; //Warn QRect r; // Warn + FOO(QRect) r2; // OK + r2.setX(0); } diff --git a/tests/unused-non-trivial-variable/other.h b/tests/unused-non-trivial-variable/other.h new file mode 100644 index 00000000..36798a4e --- /dev/null +++ b/tests/unused-non-trivial-variable/other.h @@ -0,0 +1 @@ +#define FOO(X) ::X |