svn commit: r13797 - trunk/src_new/org/argouml/ui/SettingsDialog.java

Author: euluis
Date: 2007-11-19 22:52:24-0800
New Revision: 13797

Modified:
   trunk/src_new/org/argouml/ui/SettingsDialog.java

Log:
Issue #4917: replaced show() by setVisible(boolean)

Modified: trunk/src_new/org/argouml/ui/SettingsDialog.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/ui/SettingsDialog.java?view=diff&rev=13797&p1=trunk/src_new/org/argouml/ui/SettingsDialog.java&p2=trunk/src_new/org/argouml/ui/SettingsDialog.java&r1=13796&r2=13797
==============================================================================
--- trunk/src_new/org/argouml/ui/SettingsDialog.java	(original)
+++ trunk/src_new/org/argouml/ui/SettingsDialog.java	2007-11-19 22:52:24-0800
@@ -53,8 +53,6 @@
 
     private JTabbedPane tabs;
 
-    private boolean doingShow;
-
     private boolean windowOpen;
 
     /**
@@ -99,20 +97,13 @@
         addWindowListener(this);
     }
 
-    /*
-     * @see java.awt.Dialog#show()
-     */
-    public void show() {
-        // If a recursive call from setVisible(), just return
-        if (doingShow) {
-            return;
-        }
-        doingShow = true;
-        handleRefresh();
-        super.show();
-        toFront();
-        setVisible(true);
-        doingShow = false;
+    @Override
+    public void setVisible(boolean show) {
+        if (show) {
+            handleRefresh();
+            toFront();
+        }
+        super.setVisible(show);
         // windowOpen state will be changed when window is activated
     }