shiboken: Refactor code involved in inheriting template classes

- Add a helper template to conveniently search for a MetaClass by name
  and convenience function to search in lists.
- Remove parameter bool *ok from
  AbstractMetaBuilderPrivate::inheritTemplateType() and check on returned
  pointer instead.
- In the inheritTemplate*() functions, use QScopedPointer to ensure
  the result is deleted on failure.
- Search for a shadowing function first before creating the copy
  in inheritTemplate().

Task-number: PYSIDE-725
Change-Id: I3183087fb42b22a2189c17b94eaafdb4c1151f0e
Reviewed-by: Christian Tismer <tismer@stackless.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/sources/shiboken2/ApiExtractor/abstractmetabuilder_p.h b/sources/shiboken2/ApiExtractor/abstractmetabuilder_p.h
index 7f4d222a3..9fcad26ad 100644
--- a/sources/shiboken2/ApiExtractor/abstractmetabuilder_p.h
+++ b/sources/shiboken2/ApiExtractor/abstractmetabuilder_p.h
@@ -142,9 +142,8 @@ public:
     bool inheritTemplate(AbstractMetaClass *subclass,
                          const AbstractMetaClass *templateClass,
                          const TypeInfo &info);
-    AbstractMetaType *inheritTemplateType(const QVector<AbstractMetaType *> &templateTypes,
-                                          const AbstractMetaType *metaType,
-                                          bool *ok = Q_NULLPTR);
+    AbstractMetaType *inheritTemplateType(const AbstractMetaTypeList &templateTypes,
+                                          const AbstractMetaType *metaType);
 
     bool isQObject(const FileModelItem &dom, const QString &qualifiedName);
     bool isEnum(const FileModelItem &dom, const QStringList &qualifiedName);