[Squirrel-sql-commits] SF.net SVN: squirrel-sql:[6009] trunk/sql12/app/src/main/java/net/ sourceforge/squirrel_sql/client/gui/ProgressCallBackDialog.java

[<man...@us...>]

Revision: 6009
          http://squirrel-sql.svn.sourceforge.net/squirrel-sql/?rev=6009&view=rev
Author:   manningr
Date:     2010-12-15 23:35:43 +0000 (Wed, 15 Dec 2010)

Log Message:
-----------
Patch-Bug 3123014: StackOverflow ProgressCallBackDialog (Stefan's patch)

Modified Paths:
--------------
    trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/gui/ProgressCallBackDialog.java

Modified: trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/gui/ProgressCallBackDialog.java
===================================================================
--- trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/gui/ProgressCallBackDialog.java	2010-12-14 20:35:57 UTC (rev 6008)
+++ trunk/sql12/app/src/main/java/net/sourceforge/squirrel_sql/client/gui/ProgressCallBackDialog.java	2010-12-15 23:35:43 UTC (rev 6009)
@@ -192,10 +192,18 @@
 			@Override
 			public void run()
 			{
-				ProgressCallBackDialog.this.dispose();
+				callDisposeFromSuperClass();
 			}
 		});
 	}
+	
+	/**
+	 * Since {@link #dispose()} uses an {@link Runnable}, we needs an
+	 * delegate to call the overridden dispose method.
+	 */
+	private void callDisposeFromSuperClass(){
+		super.dispose();
+	}
 
 	/**
 	 * @see java.awt.Dialog#setVisible(boolean)
@@ -208,10 +216,19 @@
 			@Override
 			public void run()
 			{
-				ProgressCallBackDialog.this.setVisible(b);
+				callSetVisibleFromSuperClass(b);
 			}
 		});
 	}	
+	
+	/**
+	 * Since {@link #setVisible(boolean)} uses an {@link Runnable}, we needs an
+	 * delegate to call the overridden setVisible method.
+	 */
+	private void callSetVisibleFromSuperClass(final boolean b){
+		super.setVisible(b);
+	}
+	
 
 	private void init(int totalItems)
 	{


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.