svn commit: r628393 - /jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

[sebb]

Author: sebb
Date: Sat Feb 16 16:09:50 2008
New Revision: 628393

URL: http://svn.apache.org/viewvc?rev=628393&view=rev
Log:
Access to hosts is from different threads so must be synch

Modified:
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java?rev=628393&r1=628392&r2=628393&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
(original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
Sat Feb 16 16:09:50 2008
@@ -98,9 +98,9 @@
 
        private boolean inTest = false;
 
-       private static Map files = new HashMap();
+       private static final Map files = new HashMap();
 
-       private Set hosts = new HashSet();
+       private final Set hosts = new HashSet();
 
        protected boolean isStats = false;
 
@@ -180,7 +180,7 @@
                setFilenameProperty(f);
        }
 
-       public void testEnded(String host) {
+       public synchronized void testEnded(String host) {
                hosts.remove(host);
                if (hosts.size() == 0) {
                        finalizeFileOutput();
@@ -188,7 +188,7 @@
                }
        }
 
-       public void testStarted(String host) {
+       public synchronized void testStarted(String host) {
                hosts.add(host);
                try {
                        initializeFileOutput();



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]