From 008798b0e3e99235d3912d885b8322710c41338b Mon Sep 17 00:00:00 2001
From: Hugo Parente Lima <hugo.pl@gmail.com>
Date: Fri, 11 Mar 2011 17:42:21 -0300
Subject: [PATCH] Fix bug 718 - "PySide/PyQt4 QByteArray incompatibilities:
 setNum method" Fix bug 719 - "PySide/PyQt4 QByteArray incompatibilities:
 appendByte method"

---
 PySide/QtCore/typesystem_core.xml | 148 ++++--------------------------
 tests/QtCore/qbytearray_test.py   |  19 ++++
 2 files changed, 35 insertions(+), 132 deletions(-)

diff --git a/PySide/QtCore/typesystem_core.xml b/PySide/QtCore/typesystem_core.xml
index 4c8c3e3..6b3721d 100644
--- a/PySide/QtCore/typesystem_core.xml
+++ b/PySide/QtCore/typesystem_core.xml
@@ -1600,26 +1600,27 @@
     <modify-function signature="operator>=(const char*,QByteArray)" remove="all" />
     <modify-function signature="operator[](int)const" remove="all"/>
     <modify-function signature="operator[](uint)const" remove="all"/>
+    <!-- Those types have the same representation in Python, an overload would be useless. -->
+    <modify-function signature="setNum(uint,int)" remove="all"/>
+    <modify-function signature="setNum(ushort,int)" remove="all"/>
+    <modify-function signature="setNum(float,char,int)" remove="all"/>
+    <modify-function signature="setNum(short,int)" remove="all"/>
+    <modify-function signature="setNum(qulonglong,int)" remove="all"/>
+
+    <!--### -->
+
+    <modify-function signature="operator const char *()const" remove="all"/>
+    <modify-function signature="operator const void *()const" remove="all"/>
+
+    <!--### STL compatibility functions not supported by PySide -->
     <modify-function signature="push_back(char)" remove="all"/>
     <modify-function signature="push_back(const QByteArray&amp;)" remove="all"/>
     <modify-function signature="push_back(const char*)" remove="all"/>
     <modify-function signature="push_front(char)" remove="all"/>
     <modify-function signature="push_front(const QByteArray&amp;)" remove="all"/>
     <modify-function signature="push_front(const char*)" remove="all"/>
-    <modify-function signature="setNum(uint,int)" remove="all"/>
-    <modify-function signature="setNum(qlonglong,int)" remove="all"/>
-    <modify-function signature="setNum(ushort,int)" remove="all"/>
-    <modify-function signature="split(char)const">
-        <modify-argument index="1">
-            <replace-type modified-type="PyString" />
-        </modify-argument>
-        <inject-code class="target" position="beginning">
-            char c = PyString_AS_STRING(%PYARG_1)[0];
-            if (c) {
-               %PYARG_0 = %CONVERTTOPYTHON[%RETURN_TYPE](%CPPSELF.%FUNCTION_NAME(c));
-            }
-        </inject-code>
-    </modify-function>
+    <!--### -->
+
     <modify-function signature="toLong(bool*, int) const">
       <modify-argument index="1">
         <remove-argument/>
@@ -1686,81 +1687,8 @@
     </modify-function>
     <!-- QByteArray(const char *) do the job of this constructor -->
     <modify-function signature="QByteArray(const char*,int)" remove="all"/>
+    <!-- QByteArray(const char*, const char*) do the job of this function -->
     <modify-function signature="replace(const char*,int,const char*,int)" remove="all"/>
-    <modify-function signature="append(QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="append(QString)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="append(const char *)" remove="all">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="append(char)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="appendByte"/>
-    </modify-function>
-    <modify-function signature="count(const char *)const" remove="all"/>
-    <modify-function signature="data()" remove="all"/>
-    <modify-function signature="endsWith(const char *)const" remove="all"/>
-    <modify-function signature="fill(char,int)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="indexOf(const char*,int)const" remove="all"/>
-    <modify-function signature="indexOf(char,int)const">
-      <rename to="indexOfByte"/>
-    </modify-function>
-    <modify-function signature="insert(int,QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="insert(int,QString)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="insert(int,const char *)" remove="all"/>
-    <modify-function signature="insert(int,char)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="insertByte"/>
-    </modify-function>
-    <modify-function signature="lastIndexOf(const char*,int)const" remove="all"/>
-    <modify-function signature="lastIndexOf(char,int)const">
-      <rename to="lastIndexOfByte"/>
-    </modify-function>
-    <modify-function signature="prepend(QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="prepend(const char *)" remove="all"/>
-    <modify-function signature="prepend(char)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="prependByte"/>
-    </modify-function>
-    <modify-function signature="remove(int,int)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(int,int,QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(int,int,const char *)" remove="all"/>
-    <modify-function signature="replace(QByteArray,QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(const char*,QByteArray)" remove="all"/>
-    <modify-function signature="replace(QByteArray,const char *)" remove="all"/>
-    <modify-function signature="replace(QString,QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(QString,const char *)" remove="all"/>
-    <modify-function signature="replace(const char *,const char *)" remove="all"/>
-    <modify-function signature="replace(char,QByteArray)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(char,QString)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="replace(char,const char *)" remove="all"/>
-    <modify-function signature="replace(char,char)">
-      <modify-argument index="0" replace-value="this"/>
-    </modify-function>
-    <modify-function signature="startsWith(const char *)const" remove="all"/>
     <modify-function signature="fromRawData(const char*,int)">
         <modify-argument index="2">
             <remove-argument />
@@ -1770,50 +1698,6 @@
             %PYARG_0 = %CONVERTTOPYTHON[QByteArray](QByteArray(%1));
         </inject-code>
     </modify-function>
-    <modify-function signature="number(int,int)">
-      <rename to="fromInt"/>
-    </modify-function>
-    <modify-function signature="number(uint,int)">
-      <rename to="fromUInt"/>
-    </modify-function>
-    <modify-function signature="number(qlonglong,int)">
-      <rename to="fromLongLong"/>
-    </modify-function>
-    <modify-function signature="number(qulonglong,int)">
-      <rename to="fromULongLong"/>
-    </modify-function>
-    <modify-function signature="setNum(int,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setInt"/>
-    </modify-function>
-    <modify-function signature="setNum(uint,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setUInt"/>
-    </modify-function>
-    <modify-function signature="setNum(short,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setShort"/>
-    </modify-function>
-    <modify-function signature="setNum(ushort,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setUShort"/>
-    </modify-function>
-    <modify-function signature="setNum(qlonglong,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setLongLong"/>
-    </modify-function>
-    <modify-function signature="setNum(qulonglong,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setULongLong"/>
-    </modify-function>
-    <modify-function signature="setNum(double,char,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setDouble"/>
-    </modify-function>
-    <modify-function signature="setNum(float,char,int)">
-      <modify-argument index="0" replace-value="this"/>
-      <rename to="setFloat"/>
-    </modify-function>
     <modify-function signature="toDouble(bool*)const">
       <modify-argument index="1">
         <remove-argument/>
diff --git a/tests/QtCore/qbytearray_test.py b/tests/QtCore/qbytearray_test.py
index fa838c1..4a600ba 100644
--- a/tests/QtCore/qbytearray_test.py
+++ b/tests/QtCore/qbytearray_test.py
@@ -28,6 +28,25 @@ class QByteArrayTestToNumber(unittest.TestCase):
         self.assertEqual((ctypes.c_double(37.109).value, True),
                          obj.toDouble())
 
+    def testSetNum(self):
+        b = QByteArray()
+        b.setNum(-124124L)
+        self.assertEqual(b, "-124124")
+        b = QByteArray()
+        b.setNum(-124124)
+        self.assertEqual(b, "-124124")
+        b = QByteArray()
+        b.setNum(-0.5)
+        self.assertEqual(b, "-0.5")
+
+    def testAppend(self):
+        b = QByteArray()
+        b.append("A")
+        self.assertEqual(b.size(), 1)
+        b.append("AB")
+        self.assertEqual(b.size(), 3)
+
+
 class QByteArraySplit(unittest.TestCase):
     '''Test case for QByteArray.split'''