From 1d1820c83b3dca098728b22afbceb07ade6ee457 Mon Sep 17 00:00:00 2001
From: Alena Laskavaia <elaskavaia.cdt@gmail.com>
Date: Wed, 20 May 2009 16:33:50 +0000
Subject: [PATCH] [277057] - check for dialog ok, patch

---
 .../eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java  | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java b/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java
index e5a10ae9705..3be698dd72f 100644
--- a/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java
+++ b/jtag/org.eclipse.cdt.debug.gdbjtag.ui/src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java
@@ -116,9 +116,11 @@ public class GDBJtagDebuggerTab extends AbstractLaunchConfigurationTab {
 	}
 	
 	private void variablesButtonSelected(Text text) {
-		StringVariableSelectionDialog dialog = new StringVariableSelectionDialog(getShell());
-		dialog.open();
-		text.append(dialog.getVariableExpression());
+		StringVariableSelectionDialog dialog = new StringVariableSelectionDialog(
+				getShell());
+		if (dialog.open() == StringVariableSelectionDialog.OK) {
+			text.append(dialog.getVariableExpression());
+		}
 	}
 	
 	private void createCommandControl(Composite parent) {