Add new <array> argument modification indicating array usage

The modification can be used to indicate that for example
int* is meant to be used as int[].
This is reflected in the NativePointerAsArrayPattern
usage pattern of AbstractMetaType.

Task-number: PYSIDE-354
Task-number: PYSIDE-516
Change-Id: Icaeb3cce4be9ce06caa2cab628d4e8fc1b684819
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>

1 files changed, 11 insertions, 2 deletions

diff --git a/sources/shiboken2/ApiExtractor/typesystem.cpp b/sources/shiboken2/ApiExtractor/typesystem.cpp
index 9adc5107b..13664c336 100644
--- a/sources/shiboken2/ApiExtractor/typesystem.cpp
+++ b/sources/shiboken2/ApiExtractor/typesystem.cpp
@@ -176,6 +176,7 @@ Handler::Handler(TypeDatabase* database, bool generate)
     tagNames.insert(QLatin1String("no-null-pointer"), StackElement::NoNullPointers);
     tagNames.insert(QLatin1String("reference-count"), StackElement::ReferenceCount);
     tagNames.insert(QLatin1String("parent"), StackElement::ParentOwner);
+    tagNames.insert(QLatin1String("array"), StackElement::Array);
     tagNames.insert(QLatin1String("inject-documentation"), StackElement::InjectDocumentation);
     tagNames.insert(QLatin1String("modify-documentation"), StackElement::ModifyDocumentation);
     tagNames.insert(QLatin1String("add-function"), StackElement::AddFunction);
@@ -1283,6 +1284,9 @@ bool Handler::startElement(const QStringRef &n, const QXmlStreamAttributes &atts
         case StackElement::ParentOwner:
             attributes.insert(QLatin1String("index"), QString());
             attributes.insert(QLatin1String("action"), QString());
+            break;
+        case StackElement::Array:
+            break;
         default:
             { };
         };
@@ -1875,8 +1879,13 @@ bool Handler::startElement(const QStringRef &n, const QXmlStreamAttributes &atts
             m_contextStack.top()->functionMods.last().argument_mods.last().owner = ao;
         }
         break;
-
-
+        case StackElement::Array:
+            if (topElement.type != StackElement::ModifyArgument) {
+                m_error = QLatin1String("array must be child of modify-argument");
+                return false;
+            }
+            m_contextStack.top()->functionMods.last().argument_mods.last().array = true;
+            break;
         case StackElement::InjectCode: {
             if (!(topElement.type & StackElement::ComplexTypeEntryMask)
                 && (topElement.type != StackElement::AddFunction)