- removed progress dialog for loading kernel choices (it is a short process and 
it only causes the Process Dialog to appear and disapear quickly)

Signed-off-by: Ioana Grigoropol <[email protected]>
---
 .../remotetools/wizards/bsp/PropertiesPage.java    |   36 +++++++++++---------
 1 file changed, 20 insertions(+), 16 deletions(-)

diff --git 
a/plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/wizards/bsp/PropertiesPage.java
 
b/plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/wizards/bsp/PropertiesPage.java
index 14de18c..4a79d64 100644
--- 
a/plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/wizards/bsp/PropertiesPage.java
+++ 
b/plugins/org.yocto.sdk.remotetools/src/org/yocto/sdk/remotetools/wizards/bsp/PropertiesPage.java
@@ -135,7 +135,7 @@ public class PropertiesPage extends WizardPage {
                        newButton.setSelection(true);
                        existingButton.setSelection(false);
 
-                       updateKernelValues(KERNEL_CHOICES, KERNEL_CHOICE);
+                       updateKernelValues(KERNEL_CHOICES, KERNEL_CHOICE, 
false);
                }
 
                try {
@@ -383,7 +383,7 @@ public class PropertiesPage extends WizardPage {
                         existingButton.setSelection(false);
                         kbCombo.removeAll();
                         
-                        updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + NEW_KBRANCH_NAME);
+                        updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + NEW_KBRANCH_NAME, true);
                 } else if (widget == kbCombo) {
                         setErrorMessage(null);
                 } else if (widget == newButton) {
@@ -392,34 +392,38 @@ public class PropertiesPage extends WizardPage {
                         
                         if (newBranch) {
                                 kbCombo.removeAll();
-                                updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + NEW_KBRANCH_NAME);
+                                updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + NEW_KBRANCH_NAME, true);
                         }
                 } else if (widget == existingButton) {
                         boolean existingBranch = existingButton.getSelection();
 
                         if (existingBranch) {
                                 kbCombo.removeAll();
-                                updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + EXISTING_KBRANCH_NAME);
+                                updateKernelValues(KERNEL_BRANCHES, "\\\"" + 
kernel_choice + "\\\"." + EXISTING_KBRANCH_NAME, true);
                         }
                 }
                 canFlipToNextPage();
                 getWizard().getContainer().updateButtons();
        }
 
-       private void updateKernelValues(final String value, String property) {
+       private void updateKernelValues(final String value, String property, 
boolean showProgress) {
                final ValuesGetter runnable  = new ValuesGetter(property);
 
-               ProgressMonitorDialog dialog = new 
ProgressMonitorDialog(getShell());
-               try {
-                       dialog.run(true, true, new IRunnableWithProgress(){
-                            public void run(IProgressMonitor monitor) {
-                                monitor.beginTask("Loading Kernel " + value + 
" ...", 100);
-                                runnable.run();
-                                monitor.done();
-                            }
-                        });
-               } catch (Exception e) {
-                       runnable.getBspAction().setMessage(e.getMessage());
+               if (showProgress) {
+                       ProgressMonitorDialog dialog = new 
ProgressMonitorDialog(getShell());
+                       try {
+                               dialog.run(true, true, new 
IRunnableWithProgress(){
+                                    public void run(IProgressMonitor monitor) {
+                                        monitor.beginTask("Loading Kernel " + 
value + " ...", 100);
+                                        runnable.run();
+                                        monitor.done();
+                                    }
+                                });
+                       } catch (Exception e) {
+                               
runnable.getBspAction().setMessage(e.getMessage());
+                       }
+               } else {
+                       runnable.run();
                }
 
                BSPAction action = runnable.getBspAction();
-- 
1.7.9.5

_______________________________________________
yocto mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to