shiboken6: TypeEntry: Use a pointer to the target lang API type entry

Replace the string m_targetLangApiName by a pointer to the type
entry; allowing to retrieve the check function.

Similarly, in TargetToNativeConversion::sourceTypeCheck(),
use the existing type entry to retrieve the check function,
allowing for removing some heuristics.

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

5 files changed, 47 insertions, 11 deletions

diff --git a/sources/shiboken6/ApiExtractor/messages.cpp b/sources/shiboken6/ApiExtractor/messages.cpp
index d851a02e0..00d9dd816 100644
--- a/sources/shiboken6/ApiExtractor/messages.cpp
+++ b/sources/shiboken6/ApiExtractor/messages.cpp
@@ -826,3 +826,9 @@ QString msgDuplicateTypeEntry(const QString &name)
 {
     return u"Duplicate type entry: '"_qs + name + u"'."_qs;
 }
+
+QString msgInvalidTargetLanguageApiName(const QString &name)
+{
+    return u"Invalid target language API name \""_qs
+           + name + u"\"."_qs;
+}
diff --git a/sources/shiboken6/ApiExtractor/messages.h b/sources/shiboken6/ApiExtractor/messages.h
index 8c924aa86..7e6fd87a5 100644
--- a/sources/shiboken6/ApiExtractor/messages.h
+++ b/sources/shiboken6/ApiExtractor/messages.h
@@ -237,5 +237,6 @@ QString msgUnknownTypeInArgumentTypeReplacement(const QString &typeReplaced,
 
 QString msgDuplicateBuiltInTypeEntry(const QString &name);
 QString msgDuplicateTypeEntry(const QString &name);
+QString msgInvalidTargetLanguageApiName(const QString &name);
 
 #endif // MESSAGES_H
diff --git a/sources/shiboken6/ApiExtractor/typesystem.cpp b/sources/shiboken6/ApiExtractor/typesystem.cpp
index 298baa7d7..6ad87a736 100644
--- a/sources/shiboken6/ApiExtractor/typesystem.cpp
+++ b/sources/shiboken6/ApiExtractor/typesystem.cpp
@@ -71,7 +71,6 @@ public:
 
     const TypeEntry *m_parent;
     QString m_name; // C++ fully qualified
-    QString m_targetLangApiName;
     mutable QString m_cachedShortName; // C++ excluding inline namespaces
     QString m_entryName;
     QString m_targetLangPackage;
@@ -89,6 +88,7 @@ public:
     SourceLocation m_sourceLocation; // XML file
     TypeEntry::CodeGeneration m_codeGeneration = TypeEntry::GenerateCode;
     TypeEntry *m_viewOn = nullptr;
+    CustomTypeEntry *m_targetLangApiType = nullptr;
     int m_revision = 0;
     int m_sbkIndex = 0;
     TypeEntry::Type m_type;
@@ -440,15 +440,25 @@ QString TypeEntry::qualifiedCppName() const
     return m_d->m_name;
 }
 
-QString TypeEntry::targetLangApiName() const
+const CustomTypeEntry *TypeEntry::targetLangApiType() const
+{
+    return m_d->m_targetLangApiType;
+}
+
+bool TypeEntry::hasTargetLangApiType() const
 {
-    return m_d->m_targetLangApiName.isEmpty()
-        ? m_d->m_name : m_d->m_targetLangApiName;
+    return m_d->m_targetLangApiType != nullptr;
 }
 
-void TypeEntry::setTargetLangApiName(const QString &t)
+void TypeEntry::setTargetLangApiType(CustomTypeEntry *cte)
 {
-    m_d->m_targetLangApiName = t;
+    m_d->m_targetLangApiType = cte;
+}
+
+QString TypeEntry::targetLangApiName() const
+{
+    return m_d->m_targetLangApiType != nullptr
+           ? m_d->m_targetLangApiType->name() : m_d->m_name;
 }
 
 QString TypeEntry::targetLangName() const
@@ -2067,7 +2077,16 @@ QString CustomConversion::TargetToNativeConversion::sourceTypeName() const
 
 QString CustomConversion::TargetToNativeConversion::sourceTypeCheck() const
 {
-    return m_d->sourceTypeCheck;
+    if (!m_d->sourceTypeCheck.isEmpty())
+        return m_d->sourceTypeCheck;
+
+    if (m_d->sourceType != nullptr && m_d->sourceType->isCustom()) {
+        const auto *cte = static_cast<const CustomTypeEntry *>(m_d->sourceType);
+        if (cte->hasCheckFunction())
+            return cte->checkFunction() + u"(%in)"_qs;
+    }
+
+    return {};
 }
 
 QString CustomConversion::TargetToNativeConversion::conversion() const
diff --git a/sources/shiboken6/ApiExtractor/typesystem.h b/sources/shiboken6/ApiExtractor/typesystem.h
index fcd0abe99..fc77fc6ca 100644
--- a/sources/shiboken6/ApiExtractor/typesystem.h
+++ b/sources/shiboken6/ApiExtractor/typesystem.h
@@ -43,6 +43,7 @@ class FlagsTypeEntry;
 class SourceLocation;
 class PrimitiveTypeEntry;
 class TypeSystemTypeEntry;
+class CustomTypeEntry;
 
 class TypeEntryPrivate;
 class TemplateArgumentEntryPrivate;
@@ -185,8 +186,10 @@ public:
     /// be a JNI name, for Python it should represent the CPython type name.
     /// \return string representing the target language API name
     /// Currently used only for PrimitiveTypeEntry (attribute "target").
+    const CustomTypeEntry *targetLangApiType() const;
+    bool hasTargetLangApiType() const;
+    void setTargetLangApiType(CustomTypeEntry *cte);
     QString targetLangApiName() const;
-    void setTargetLangApiName(const QString &t);
 
     // The type's name in TargetLang
     QString targetLangName() const; // "Foo.Bar"
diff --git a/sources/shiboken6/ApiExtractor/typesystemparser.cpp b/sources/shiboken6/ApiExtractor/typesystemparser.cpp
index fe1761dd8..a3730ef29 100644
--- a/sources/shiboken6/ApiExtractor/typesystemparser.cpp
+++ b/sources/shiboken6/ApiExtractor/typesystemparser.cpp
@@ -1334,6 +1334,7 @@ PrimitiveTypeEntry *
     if (!checkRootElement())
         return nullptr;
     auto *type = new PrimitiveTypeEntry(name, since, currentParentTypeEntry());
+    QString targetLangApiName;
     if (!applyCommonAttributes(reader, type, attributes))
         return nullptr;
     for (int i = attributes->size() - 1; i >= 0; --i) {
@@ -1341,7 +1342,7 @@ PrimitiveTypeEntry *
         if (name == targetLangNameAttribute()) {
             type->setTargetLangName(attributes->takeAt(i).value().toString());
         } else if (name == QLatin1String("target-lang-api-name")) {
-            type->setTargetLangApiName(attributes->takeAt(i).value().toString());
+            targetLangApiName = attributes->takeAt(i).value().toString();
         } else if (name == preferredConversionAttribute()) {
             qCWarning(lcShiboken, "%s",
                       qPrintable(msgUnimplementedAttributeWarning(reader, name)));
@@ -1354,8 +1355,14 @@ PrimitiveTypeEntry *
         }
     }
 
-    if (type->targetLangApiName().isEmpty())
-        type->setTargetLangApiName(type->name());
+    if (!targetLangApiName.isEmpty()) {
+        auto *e = m_database->findType(targetLangApiName);
+        if (e == nullptr || !e->isCustom()) {
+               m_error = msgInvalidTargetLanguageApiName(targetLangApiName);
+               return nullptr;
+        }
+        type->setTargetLangApiType(static_cast<CustomTypeEntry *>(e));
+    }
     type->setTargetLangPackage(m_defaultPackage);
     return type;
 }