You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jd...@apache.org on 2007/10/14 10:03:16 UTC

svn commit: r584505 - /wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java

Author: jdonnerstag
Date: Sun Oct 14 01:03:15 2007
New Revision: 584505

URL: http://svn.apache.org/viewvc?rev=584505&view=rev
Log:
wicket-1065: FileCleaner problem fixed

Modified:
    wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java?rev=584505&r1=584504&r2=584505&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/util/file/FileCleaner.java Sun Oct 14 01:03:15 2007
@@ -34,7 +34,7 @@
 	/**
 	 * Queue of <code>Tracker</code> instances being watched.
 	 */
-	private static ReferenceQueue /* Tracker */q = new ReferenceQueue();
+	private static final ReferenceQueue /* Tracker */q = new ReferenceQueue();
 
 	/**
 	 * Collection of <code>Tracker</code> instances in existence.
@@ -52,7 +52,9 @@
 		 */
 		public void run()
 		{
-			for (;;)
+			// Though q is final, it happens while hot deploying that Wicket runs into an infinite
+			// loop because q == null (NullPointerException). To prevent that happening ...
+			while (q != null)
 			{
 				Tracker tracker = null;
 				try