Debugger: Fix transport of negative signed chars

Change-Id: I9b25d4b8e4b68d9cef674581d1a42eb75e21ec56
Task-number: QTCREATORBUG-16353
Reviewed-by: David Schulz <david.schulz@theqtcompany.com>
Reviewed-by: hjk <hjk@theqtcompany.com>

share/qtcreator/debugger/gdbbridge.py

@@ -1025,7 +1025,7 @@ class Dumper(DumperBase):
             self.putType(typeName)
             if typeobj.sizeof == 1:
                 # Force unadorned value transport for char and Co.
-                self.putValue(int(value) & 0xff)
+                self.putValue(int(value))
             else:
                 self.putValue(value)
             self.putNumChild(0)

tests/auto/debugger/tst_dumpers.cpp

@@ -4758,6 +4758,24 @@ void tst_Dumpers::dumper_data()
                + Check("a.#1.i", "42", "int") % LldbEngine
                + Check("a.#2.f", ff, "float") % LldbEngine;
 
+    QTest::newRow("Chars")
+            << Data("#include <qglobal.h>\n",
+                    "char c = -12;\n"
+                    "signed char sc = -12;\n"
+                    "unsigned char uc = -12;\n"
+                    "qint8 qs = -12;\n"
+                    "quint8 qu  = -12;\n"
+                    "unused(&c, &sc, &uc, &qs, &qu);\n")
+
+               + Check("c", "-12", "char")  // on all our platforms char is signed.
+               + Check("sc", "-12", "signed char") % NoCdbEngine
+               + Check("sc", "-12", "char") % CdbEngine
+               + Check("uc", "244", "unsigned char")
+               + Check("qs", "-12", "@qint8") % NoCdbEngine
+               + Check("qs", "-12", "char") % CdbEngine
+               + Check("qu", "244", "@quint8") % NoCdbEngine
+               + Check("qu", "244", "unsigned char") % CdbEngine;
+
 
     QTest::newRow("CharArrays")
             << Data("char s[] = \"aöa\";\n"