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);
}