Remove createIndex method that uses PyObject*

We have a proper `void*` treatment, so the extra
function does not seems to be necessary.

The injected code used for the interalPointer method was
also removed since is better to leave on shiboken hands
the conversion between void* and python types.

A couple of test were adapted since wrongly used ModelIndex
with Python objects.

Change-Id: I082637c7d1dd1bb0f50a3c453513e5d722f4b783
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>

1 files changed, 0 insertions, 34 deletions

diff --git a/sources/pyside2/PySide2/QtCore/typesystem_core_common.xml b/sources/pyside2/PySide2/QtCore/typesystem_core_common.xml
index cff7e0af0..3b5d9f520 100644
--- a/sources/pyside2/PySide2/QtCore/typesystem_core_common.xml
+++ b/sources/pyside2/PySide2/QtCore/typesystem_core_common.xml
@@ -1827,11 +1827,6 @@
     </add-function>
   </value-type>
   <value-type name="QPersistentModelIndex" hash-function="qHash">
-    <modify-function signature="internalPointer()const">
-        <inject-code class="target" position="beginning">
-            <insert-template name="return_internal_pointer" />
-        </inject-code>
-    </modify-function>
     <modify-function signature="operator const QModelIndex&amp;()const">
         <modify-argument index="return">
             <parent index="this" action="add"/>
@@ -2101,36 +2096,12 @@
   <object-type name="QAbstractItemModel">
     <enum-type name="CheckIndexOption" flags="CheckIndexOptions" since="5.11"/>
     <enum-type name="LayoutChangeHint" />
-    <!-- This function was replaced by a added function -->
-    <modify-function signature="createIndex(int,int,void*)const" remove="all"/>
     <!-- This function is the same as createIndex(int, int, int)const -->
     <modify-function signature="createIndex(int,int,quintptr)const">
         <modify-argument index="3">
             <replace-default-expression with="0" />
         </modify-argument>
     </modify-function>
-    <add-function signature="createIndex(int,int,PyObject*)const" return-type="QModelIndex">
-        <modify-argument index="1">
-            <rename to="row"/>
-        </modify-argument>
-        <modify-argument index="2">
-            <rename to="column"/>
-        </modify-argument>
-        <modify-argument index="3">
-            <rename to="ptr"/>
-        </modify-argument>
-        <inject-code class="target" position="beginning">
-        %RETURN_TYPE %0 = %CPPSELF.%FUNCTION_NAME(%1, %2, %PYARG_3);
-        %PYARG_0 = %CONVERTTOPYTHON[%RETURN_TYPE](%0);
-        </inject-code>
-        <inject-documentation mode="append" format="target">
-            Creates a model index for the given row and column with the internal pointer ptr.
-            When using a QSortFilterProxyModel, its indexes have their own internal pointer. It is not advisable to access this internal pointer outside of the model. Use the data() function instead.
-            This function provides a consistent interface that model subclasses must use to create model indexes.
-
-            .. warning:: Because of some Qt/Python itegration rules, the ptr argument do not get the reference incremented during the QModelIndex life time. So it is necessary to keep the object used on ptr argument alive during the whole process. Do not destroy the object if you are not sure about that.
-        </inject-documentation>
-    </add-function>
     <inject-code class="target" position="end">
         qRegisterMetaType&lt;QVector&lt;int&gt; &gt;("QVector&lt;int&gt;");
     </inject-code>
@@ -3890,11 +3861,6 @@
     </modify-function>
   </object-type>
   <value-type name="QModelIndex" hash-function="qHash">
-    <modify-function signature="internalPointer()const">
-        <inject-code class="target" position="beginning">
-            <insert-template name="return_internal_pointer" />
-        </inject-code>
-    </modify-function>
     <modify-function signature="model()const">
       <modify-argument index="return">
         <define-ownership class="target" owner="default"/>