You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2005/07/27 03:02:13 UTC

svn commit: r225439 - /geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java

Author: djencks
Date: Tue Jul 26 18:02:10 2005
New Revision: 225439

URL: http://svn.apache.org/viewcvs?rev=225439&view=rev
Log:
GERONIMO-820 synchronize access to 'state' field

Modified:
    geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java

Modified: geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java?rev=225439&r1=225438&r2=225439&view=diff
==============================================================================
--- geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java (original)
+++ geronimo/trunk/modules/deploy-jsr88/src/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java Tue Jul 26 18:02:10 2005
@@ -70,7 +70,7 @@
         return (TargetModuleID[]) moduleIDs.toArray(new TargetModuleID[moduleIDs.size()]);
     }
 
-    public DeploymentStatus getDeploymentStatus() {
+    public synchronized DeploymentStatus getDeploymentStatus() {
         return new Status(command, action, state, message);
     }
 
@@ -158,7 +158,7 @@
         synchronized (this) {
             this.message = message;
             this.state = state;
-            newStatus = getDeploymentStatus();
+            newStatus = new Status(command, action, state, message);
             toNotify = (ProgressListener[]) listeners.toArray(new ProgressListener[listeners.size()]);
             event = new ProgressEvent(this, null, newStatus);
         }