shiboken: Code cleanup

Remove code working around a Qt bug and a modification
on a type entry name which was apparently intended to strip
off qualifiers using a regex, but has no effect since the QString
overload was called.

Remove check for null in AbstractMetaBuilderPrivate::addAbstractMetaClass()
and modify call sites accordingly.

Change-Id: I7ab2a163fe558af09f2c7989bdec1561e9bbb203
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>

1 files changed, 4 insertions, 10 deletions

diff --git a/sources/shiboken2/ApiExtractor/abstractmetabuilder.cpp b/sources/shiboken2/ApiExtractor/abstractmetabuilder.cpp
index 73928cd6f..b0c212f89 100644
--- a/sources/shiboken2/ApiExtractor/abstractmetabuilder.cpp
+++ b/sources/shiboken2/ApiExtractor/abstractmetabuilder.cpp
@@ -476,11 +476,8 @@ void AbstractMetaBuilderPrivate::traverseDom(const FileModelItem &dom)
     for (const ClassModelItem &item : typeValues) {
         ReportHandler::progress(QStringLiteral("Generating class model (%1)...")
                                 .arg(typeValues.size()));
-        AbstractMetaClass *cls = traverseClass(dom, item);
-        if (!cls)
-            continue;
-
-        addAbstractMetaClass(cls);
+        if (AbstractMetaClass *cls = traverseClass(dom, item))
+            addAbstractMetaClass(cls);
     }
 
     // We need to know all global enums
@@ -513,8 +510,8 @@ void AbstractMetaBuilderPrivate::traverseDom(const FileModelItem &dom)
     for (const TypeDefModelItem &typeDef : typeDefs) {
         ReportHandler::progress(QStringLiteral("Resolving typedefs (%1)...")
                                 .arg(typeDefs.size()));
-        AbstractMetaClass* cls = traverseTypeDef(dom, typeDef);
-        addAbstractMetaClass(cls);
+        if (AbstractMetaClass *cls = traverseTypeDef(dom, typeDef))
+            addAbstractMetaClass(cls);
     }
 
     for (const ClassModelItem &item : typeValues)
@@ -768,9 +765,6 @@ void AbstractMetaBuilder::setLogDirectory(const QString& logDir)
 
 void AbstractMetaBuilderPrivate::addAbstractMetaClass(AbstractMetaClass *cls)
 {
-    if (!cls)
-        return;
-
     cls->setOriginalAttributes(cls->attributes());
     if (cls->typeEntry()->isContainer()) {
         m_templates << cls;