From d878466cef1e64766144d368ef86a176136b998e Mon Sep 17 00:00:00 2001 From: Kevin Doyle Date: Wed, 31 Oct 2007 04:47:34 +0000 Subject: [PATCH] [150492] Tutorial FolderInfoPropertyPage doesn't work reliably --- .../propertypages/FolderInfoPropertyPage.java | 27 +++++++++---------- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/rse/examples/org.eclipse.rse.examples.tutorial/src/samples/ui/propertypages/FolderInfoPropertyPage.java b/rse/examples/org.eclipse.rse.examples.tutorial/src/samples/ui/propertypages/FolderInfoPropertyPage.java index 45e00e98642..7f54344bb04 100644 --- a/rse/examples/org.eclipse.rse.examples.tutorial/src/samples/ui/propertypages/FolderInfoPropertyPage.java +++ b/rse/examples/org.eclipse.rse.examples.tutorial/src/samples/ui/propertypages/FolderInfoPropertyPage.java @@ -13,6 +13,7 @@ * * Contributors: * Martin Oberhuber (Wind River) - Adapted original tutorial code to Open RSE. + * Kevin Doyle (IBM) - [150492] FolderInfoPropertyPage doesn't work reliably *******************************************************************************/ package samples.ui.propertypages; @@ -181,15 +182,14 @@ public class FolderInfoPropertyPage public void run() { - if (stopped) + if (stopped) { return; - walkFolder(inputFolder); - updateGUI(); - if (!stopped) - { - stopped = true; - updateGUI(); } + walkFolder(inputFolder); + if (!stopped) { + stopped = true; + } + updateGUI(); } /** @@ -239,18 +239,15 @@ public class FolderInfoPropertyPage { if (stopButton.isDisposed()) return; - if (!stopped) - { - sizeLabel.setText(Integer.toString(totalSize)); - filesLabel.setText(Integer.toString(totalFiles)); - foldersLabel.setText(Integer.toString(totalFolders)); - } - else if (stopped) + if (stopped) { setValid(true); // re-enable OK button stopButton.setEnabled(false); // disable Stop button clearMessage(); // clear "Processing..." message - } + } + sizeLabel.setText(Integer.toString(totalSize)); + filesLabel.setText(Integer.toString(totalFiles)); + foldersLabel.setText(Integer.toString(totalFolders)); } }