From 710b3603843c921110fd376be698665e66a5b413 Mon Sep 17 00:00:00 2001 From: qthjk <qthjk@ovi.com> Date: Tue, 24 May 2011 16:38:04 +0200 Subject: [PATCH] debugger: fix selection of displays of integer-ish data Change-Id: I9e9d22884a53f462b6e7a3e823cb14958c0eccd9 Reviewed-on: http://codereview.qt.nokia.com/93 Reviewed-by: hjk <qthjk@ovi.com> --- src/plugins/debugger/watchhandler.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/plugins/debugger/watchhandler.cpp b/src/plugins/debugger/watchhandler.cpp index 9cd7dbd5e49..6f5eb097a97 100644 --- a/src/plugins/debugger/watchhandler.cpp +++ b/src/plugins/debugger/watchhandler.cpp @@ -370,8 +370,10 @@ static inline QString formattedValue(const WatchData &data, int format) return reformatInteger(data.value.toLongLong(), format); } - if (data.value.toULongLong(0, 0)) - return reformatInteger(data.value.toLongLong(), format); + bool ok = false; + qulonglong integer = data.value.toULongLong(&ok, 0); + if (ok) + return reformatInteger(integer, format); QString result = data.value; result.replace(QLatin1Char('\n'), QLatin1String("\\n")); @@ -721,8 +723,9 @@ QVariant WatchModel::data(const QModelIndex &idx, int role) const return QStringList() << tr("Latin1 string") << tr("UTF8 string"); - if ((isIntType(data.type) && data.type != "bool") - || data.value.toULongLong(0, 0)) + bool ok = false; + (void)data.value.toULongLong(&ok, 0); + if ((isIntType(data.type) && data.type != "bool") || ok) return QStringList() << tr("Decimal") << tr("Hexadecimal") -- GitLab