svn commit: r365953 - /geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java

[sp...@apache.org]

Author: sppatel
Date: Wed Jan  4 09:36:23 2006
New Revision: 365953

URL: http://svn.apache.org/viewcvs?rev=365953&view=rev
Log:
fixme comment

Modified:
    geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java

Modified: geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java
URL: http://svn.apache.org/viewcvs/geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java?rev=365953&r1=365952&r2=365953&view=diff
==============================================================================
--- geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java (original)
+++ geronimo/devtools/trunk/modules/eclipse-plugin/plugins/org.apache.geronimo.devtools.eclipse.core/src/org/apache/geronimo/core/internal/GeronimoServerBehaviour.java Wed Jan  4 09:36:23 2006
@@ -323,6 +323,12 @@
 	/**
 	 * This timer task runs at scheduled intervals to sync the server state if
 	 * the users controls the server instance outside of the eclipse workbench.
+	 * 
+	 * WTP manages the server process, and if the process is dead, the state is updated.
+	 * So the only scenario that needs to be considered is if the server is restarted.
+	 * 
+	 * FIXME When the server is stop the GeronimoServerBehavior instance is destroyed so
+	 * the task never runs to handle this scenario.
 	 */
 	private class UpdateServerStateTask extends TimerTask {