diff options
| author | Thiago Macieira <[email protected]> | 2013-12-05 23:29:37 -0800 |
|---|---|---|
| committer | Thiago Macieira <[email protected]> | 2014-01-10 15:40:22 +0100 |
| commit | 58a4e601ffa1262841534c547bf8bb7cd075c3dd (patch) | |
| tree | cbac4603f2d8cab1ad050b2525d2cc91a477f20d /src/plugins/debugger/debuggerprotocol.cpp | |
| parent | 1ff3a8287d7f830de362d49b4042544c6af3a754 (diff) | |
Fix change-of-sign warning found by ICC
-1 is not a valid unsigned value.
registerhandler.cpp(354): warning #68: integer conversion resulted in a change of sign
return createIndex(row, col, TopLevelId);
^
Change-Id: I41935255704f19724d6cec16ea470e2f8f8a16a1
Reviewed-by: Kurt Pattyn <[email protected]>
Reviewed-by: Nikolai Kosjar <[email protected]>
Reviewed-by: hjk <[email protected]>
Diffstat (limited to 'src/plugins/debugger/debuggerprotocol.cpp')
| -rw-r--r-- | src/plugins/debugger/debuggerprotocol.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/plugins/debugger/debuggerprotocol.cpp b/src/plugins/debugger/debuggerprotocol.cpp index 5e5ab738214..086e47fae50 100644 --- a/src/plugins/debugger/debuggerprotocol.cpp +++ b/src/plugins/debugger/debuggerprotocol.cpp @@ -50,7 +50,7 @@ uchar fromhex(uchar c) return 10 + c - 'a'; if (c >= 'A' && c <= 'Z') return 10 + c - 'A'; - return -1; + return UCHAR_MAX; } void skipCommas(const char *&from, const char *to) @@ -141,7 +141,7 @@ QByteArray GdbMi::parseCString(const char *&from, const char *to) uchar prod = 0; while (true) { uchar val = fromhex(c); - if (val == uchar(-1)) + if (val == UCHAR_MAX) break; prod = prod * 16 + val; if (++chars == 3 || src == end) |