From a8174d258d4ad1f151a245b80e5a9bbddc731f2a Mon Sep 17 00:00:00 2001
From: Stephan Bergmann <sbergman@redhat.com>
Date: Thu, 30 Apr 2015 15:05:23 +0200
Subject: streamline containsWindowSubclass

Change-Id: I56d61b577df00855a49dd618e9dafcdb86cd7ca4
---
 compilerplugins/clang/vclwidgets.cxx | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

(limited to 'compilerplugins')

diff --git a/compilerplugins/clang/vclwidgets.cxx b/compilerplugins/clang/vclwidgets.cxx
index f984dcb2ec7b..a2f4ee15855e 100644
--- a/compilerplugins/clang/vclwidgets.cxx
+++ b/compilerplugins/clang/vclwidgets.cxx
@@ -80,20 +80,18 @@ bool isDerivedFromWindow(const CXXRecordDecl *decl) {
 bool containsWindowSubclass(const Type* pType0);
 
 bool containsWindowSubclass(const QualType& qType) {
-    if (startsWith(qType.getAsString(), "VclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "const VclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "class VclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "const class VclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "ScopedVclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "class ScopedVclPtr"))
-        return false;
-    if (startsWith(qType.getAsString(), "const class ScopedVclPtr"))
-        return false;
+    auto t = qType->getAs<RecordType>();
+    if (t != nullptr) {
+        auto d = dyn_cast<ClassTemplateSpecializationDecl>(t->getDecl());
+        if (d != nullptr) {
+            std::string name(d->getQualifiedNameAsString());
+            if (name == "ScopedVclPtr" || name == "ScopedVclPtrInstance"
+                || name == "VclPtr" || name == "VclPtrInstance")
+            {
+                return false;
+            }
+        }
+    }
     return containsWindowSubclass(qType.getTypePtr());
 }
 
-- 
cgit