From a4c0712cb221e706e0ea634cd83093d8cd6b0818 Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Tue, 21 Feb 2017 11:20:12 +0100
Subject: [PATCH] Debugger: Add a message when trying to evaluate a bad
 expression

Change-Id: Id10f11876e9b6c852a04add6bd75aec7e8337891
Reviewed-by: Leena Miettinen <riitta-leena.miettinen@qt.io>
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
---
 src/plugins/debugger/debuggerplugin.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/plugins/debugger/debuggerplugin.cpp b/src/plugins/debugger/debuggerplugin.cpp
index d9af7e9fe8e..a91b719ced1 100644
--- a/src/plugins/debugger/debuggerplugin.cpp
+++ b/src/plugins/debugger/debuggerplugin.cpp
@@ -920,8 +920,12 @@ public:
         else
             exp = fixCppExpression(exp);
         exp = exp.trimmed();
-        if (exp.isEmpty())
+        if (exp.isEmpty()) {
+            // Happens e.g. when trying to evaluate 'char' or 'return'.
+            AsynchronousMessageBox::warning(tr("Warning"),
+                tr("Select a valid expression to evaluate."));
             return;
+        }
         currentEngine()->watchHandler()->watchVariable(exp);
     }