Author: sebb
Date: Wed Mar 7 10:20:40 2007
New Revision: 515678
URL: http://svn.apache.org/viewvc?view=rev&rev=515678
Log:
Fix up some messy comments
Modified:
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/reporters/Summariser.java
Modified:
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/reporters/Summariser.java
URL:
http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/reporters/Summariser.java?view=diff&rev=515678&r1=515677&r2=515678
==============================================================================
---
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/reporters/Summariser.java
(original)
+++
jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/reporters/Summariser.java
Wed Mar 7 10:20:40 2007
@@ -65,10 +65,11 @@
private static Hashtable accumulators = new Hashtable();
/*
- * Constructor is initially called once for each occurrence in the test
plan
- * For GUI, several more instances are created Then clear is called at
start
- * of test Called several times during test startup The name will not
- * necessarily have been set at this point.
+ * Constructor is initially called once for each occurrence in the test
plan.
+ * For GUI, several more instances are created.
+ * Then clear is called at start of test.
+ * Called several times during test startup.
+ * The name will not necessarily have been set at this point.
*/
public Summariser() {
super();
@@ -77,9 +78,10 @@
// "+Thread.currentThread().getName());
}
- /*
- * Constructor for use during startup (intended for non-GUI use) @param
name
- * of summariser
+ /**
+ * Constructor for use during startup (intended for non-GUI use)
+ *
+ * @param name of summariser
*/
public Summariser(String name) {
this();
@@ -97,8 +99,7 @@
myName = this.getName();
- // Hashtable is synchronised, but there could be more than one
- // Summariser
+ // Hashtable is synchronised, but there could be more than one
Summariser
// with the same name, so we need to synch.
synchronized (accumulators) {
Totals tots = (Totals) accumulators.get(myName);
@@ -114,10 +115,9 @@
super.clear();
}
- /**
+ /*
* Contains the items needed to collect stats for a summariser
*
- * @version $revision$ Last updated: $date$
*/
private static class Totals {
@@ -161,7 +161,7 @@
/**
* Accumulates the sample in two SampleResult objects - one for running
- * totals, and the other for deltas
+ * totals, and the other for deltas.
*
* @see
org.apache.jmeter.samplers.SampleListener#sampleOccurred(org.apache.jmeter.samplers.SampleEvent)
*/
@@ -189,24 +189,19 @@
boolean reportNow = false;
/*
- * Have we reached the reporting boundary? Need to allow for a
margin of
- * error, otherwise can miss the slot Also need to check we've
not hit
- * the window already
+ * Have we reached the reporting boundary?
+ * Need to allow for a margin of error, otherwise can miss the
slot.
+ * Also need to check we've not hit the window already
*/
synchronized (myTotals) {
if ((now > myTotals.last + INTERVAL_WINDOW) && (now %
INTERVAL <= INTERVAL_WINDOW)) {
reportNow = true;
- myDelta = new RunningSample(myTotals.delta);//
copy
-
// the
-
// data
-
// to
-
// minimise
-
// ...
+
+ // copy the data to minimise the synch time
+ myDelta = new RunningSample(myTotals.delta);
myTotals.moveDelta();
- myTotal = new RunningSample(myTotals.total);//
...
-
// the
-
// synch
-
// time
+ myTotal = new RunningSample(myTotals.total);
+
myTotals.last = now;
}
}
@@ -369,4 +364,4 @@
// not used
}
-}
+}
\ No newline at end of file
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
