Introduce a new built-in type: signed char.

C++ distinguish between "char", "signed char" and
"unsigned char", they are three independent types.

Fix QVariant behavior on ARM. On ARM "char" may mean
"unsigned char", but we depends on the sign during
a numerical conversions.

Change-Id: I610ce3fb88ed5964b67f3ae442d264fe16b2d261
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>

1 files changed, 7 insertions, 0 deletions

diff --git a/src/corelib/kernel/qmetatype.cpp b/src/corelib/kernel/qmetatype.cpp
index 2756dd5241..448b6e3938 100644
--- a/src/corelib/kernel/qmetatype.cpp
+++ b/src/corelib/kernel/qmetatype.cpp
@@ -161,6 +161,7 @@ struct DefinedTypesFilter {
     \value ULong \c{unsigned long}
     \value ULongLong ULongLong
     \value UShort \c{unsigned short}
+    \value SChar \c{signed char}
     \value UChar \c{unsigned char}
     \value Float \c float
     \value QObjectStar QObject *
@@ -718,6 +719,9 @@ bool QMetaType::save(QDataStream &stream, int type, const void *data)
     case QMetaType::UShort:
         stream << *static_cast<const ushort *>(data);
         break;
+    case QMetaType::SChar:
+        stream << *static_cast<const signed char *>(data);
+        break;
     case QMetaType::UChar:
         stream << *static_cast<const uchar *>(data);
         break;
@@ -937,6 +941,9 @@ bool QMetaType::load(QDataStream &stream, int type, void *data)
     case QMetaType::UShort:
         stream >> *static_cast<ushort *>(data);
         break;
+    case QMetaType::SChar:
+        stream >> *static_cast<signed char *>(data);
+        break;
     case QMetaType::UChar:
         stream >> *static_cast<uchar *>(data);
         break;