Bug 460953 - SetCrossCommandWizardPage does not check for 'null' in
authorAlena Laskavaia <elaskavaia.cdt@gmail.com>
Tue, 21 Jul 2015 00:37:20 +0000 (20:37 -0400)
committerGerrit Code Review @ Eclipse.org <gerrit@eclipse.org>
Thu, 23 Jul 2015 18:37:43 +0000 (14:37 -0400)
result for dialog.open()

Change-Id: I8bc947e5f72891138cd41fd781d5839d09381c51

cross/org.eclipse.cdt.build.crossgcc/src/org/eclipse/cdt/internal/build/crossgcc/SetCrossCommandWizardPage.java
xlc/org.eclipse.cdt.managedbuilder.xlc.ui/src/org/eclipse/cdt/managedbuilder/xlc/ui/wizards/XLCSettingsWizardPage.java

index 1448e50..be36a18 100644 (file)
@@ -125,8 +125,9 @@ public class SetCrossCommandWizardPage extends MBSCustomPage {
                        public void widgetSelected(SelectionEvent e) {
                                DirectoryDialog dirDialog = new DirectoryDialog(composite.getShell(), SWT.APPLICATION_MODAL);
                                String browsedDirectory = dirDialog.open();
-                               pathTxt.setText(browsedDirectory);
-                               
+                               if (browsedDirectory != null) {
+                                       pathTxt.setText(browsedDirectory);
+                               }
                        }
                });
                layoutData = new GridData(SWT.LEFT, SWT.TOP, false, false, 1, 1);
index e1a4172..7e5fb78 100644 (file)
@@ -60,8 +60,9 @@ public class XLCSettingsWizardPage extends MBSCustomPage {
                        // open a browse dialog
                        DirectoryDialog dirDialog = new DirectoryDialog(composite.getShell(), SWT.APPLICATION_MODAL);
                        String browsedDirectory = dirDialog.open();
-                       fDirTextBox.setText(browsedDirectory);
-
+                       if (browsedDirectory != null) {
+                               fDirTextBox.setText(browsedDirectory);
+                       }
                }
        }