Use QT_NO_CAST_FROM_ASCII for debugger plugin.

Fix gdmi escape routine by using a traits class to adapt
QByteArray/QString.

Change-Id: Ic6d2a0c927e4613aec9f69095bbb04a393e9edb9
Reviewed-by: hjk <qthjk@ovi.com>

src/plugins/debugger/debugger.pro

@@ -8,7 +8,7 @@ TARGET = Debugger
 include(../../qtcreatorplugin.pri)
 include(debugger_dependencies.pri)
 
-DEFINES += DEBUGGER_LIBRARY
+DEFINES += DEBUGGER_LIBRARY QT_NO_CAST_FROM_ASCII
 
 INCLUDEPATH += $$PWD/../../libs/utils
 

src/plugins/debugger/debuggeractions.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "debuggeractions.h"
 #include "debuggerconstants.h"
 

src/plugins/debugger/debuggerplugin.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "debuggerplugin.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/gdb/coregdbadapter.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "coregdbadapter.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/gdb/gdbengine.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "gdbengine.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/gdb/gdbmi.cpp

@@ -246,9 +246,26 @@ void GdbMi::dumpChildren(QByteArray * str, bool multiline, int indent) const
     }
 }
 
-class MyString : public QString {
+template <class String, typename CharType>
-public:
+struct EscapeTraits
-    ushort at(int i) const { return constData()[i].unicode(); }
+{
+    static CharType at(const String &s, int  i)
+        { return s.at(i); }
+    static void appendChar(CharType c, String *s)
+        { s->append(c); }
+    static void appendString(const char *c, String *s)
+        { s->append(c); }
+};
+
+template <>
+struct EscapeTraits<QString, ushort>
+{
+    static ushort at(const QString &s, int  i)
+        { return s.at(i).unicode(); }
+    static void appendChar(ushort c, QString *s)
+        { s->append(QLatin1Char(c)); }
+    static void appendString(const char *c, QString *s)
+        { s->append(QLatin1String(c)); }
 };
 
 template<class ST, typename CT>
@@ -257,25 +274,25 @@ inline ST escapeCStringTpl(const ST &ba)
     ST ret;
     ret.reserve(ba.length() * 2);
     for (int i = 0; i < ba.length(); ++i) {
-        CT c = ba.at(i);
+        const CT c = EscapeTraits<ST, CT>::at(ba, i);
         switch (c) {
-            case '\\': ret += "\\\\"; break;
+            case '\\': EscapeTraits<ST, CT>::appendString("\\\\", &ret); break;
-            case '\a': ret += "\\a"; break;
+            case '\a': EscapeTraits<ST, CT>::appendString("\\a", &ret); break;
-            case '\b': ret += "\\b"; break;
+            case '\b': EscapeTraits<ST, CT>::appendString("\\b", &ret); break;
-            case '\f': ret += "\\f"; break;
+            case '\f': EscapeTraits<ST, CT>::appendString("\\f", &ret); break;
-            case '\n': ret += "\\n"; break;
+            case '\n': EscapeTraits<ST, CT>::appendString("\\n", &ret); break;
-            case '\r': ret += "\\r"; break;
+            case '\r': EscapeTraits<ST, CT>::appendString("\\r", &ret); break;
-            case '\t': ret += "\\t"; break;
+            case '\t': EscapeTraits<ST, CT>::appendString("\\t", &ret); break;
-            case '\v': ret += "\\v"; break;
+            case '\v': EscapeTraits<ST, CT>::appendString("\\v", &ret); break;
-            case '"': ret += "\\\""; break;
+            case '"': EscapeTraits<ST, CT>::appendString("\\\"", &ret); break;
             default:
                 if (c < 32 || c == 127) {
-                    ret += '\\';
+                    EscapeTraits<ST, CT>::appendChar('\\', &ret);
-                    ret += '0' + (c >> 6);
+                    EscapeTraits<ST, CT>::appendChar('0' + (c >> 6), &ret);
-                    ret += '0' + ((c >> 3) & 7);
+                    EscapeTraits<ST, CT>::appendChar('0' + ((c >> 3) & 7), &ret);
-                    ret += '0' + (c & 7);
+                    EscapeTraits<ST, CT>::appendChar('0' + (c & 7), &ret);
                 } else {
-                    ret += c;
+                    EscapeTraits<ST, CT>::appendChar(c, &ret);
                 }
         }
     }
@@ -284,7 +301,7 @@ inline ST escapeCStringTpl(const ST &ba)
 
 QString GdbMi::escapeCString(const QString &ba)
 {
-    return escapeCStringTpl<MyString, ushort>(static_cast<const MyString &>(ba));
+    return escapeCStringTpl<QString, ushort>(ba);
 }
 
 QByteArray GdbMi::escapeCString(const QByteArray &ba)

src/plugins/debugger/lldb/lldbenginehost.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "lldbenginehost.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/loadcoredialog.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "loadcoredialog.h"
 
 #include "debuggerconstants.h"

src/plugins/debugger/namedemangler/namedemangler.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "namedemangler.h"
 
 #include "demanglerexceptions.h"

src/plugins/debugger/namedemangler/parsetreenodes.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "parsetreenodes.h"
 
 #include "demanglerexceptions.h"

src/plugins/debugger/pdb/pdbengine.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "pdbengine.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/qml/qmlinspectoradapter.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "qmlinspectoradapter.h"
 
 #include "debuggeractions.h"

src/plugins/debugger/qml/qmlinspectoragent.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "qmlinspectoragent.h"
 
 #include "debuggeractions.h"

src/plugins/debugger/qml/qmllivetextpreview.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "qmllivetextpreview.h"
 
 #include "qmlinspectoradapter.h"

src/plugins/debugger/qml/qmlv8debuggerclient.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "qmlv8debuggerclient.h"
 #include "qmlv8debuggerclientconstants.h"
 #include "debuggerstringutils.h"

src/plugins/debugger/script/scriptengine.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "scriptengine.h"
 
 #include "debuggerstartparameters.h"

src/plugins/debugger/watchhandler.cpp

@@ -28,8 +28,6 @@
 **
 **************************************************************************/
 
-#define QT_NO_CAST_FROM_ASCII
-
 #include "watchhandler.h"
 
 #include "breakhandler.h"