Attachment 80952 Details for Bug 207164 – Proposed patch

[patch] Proposed patch

v00.txt (text/plain), 1.93 KB, created by Frederic Fusier

on 2007-10-23 10:07:39 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Frederic Fusier

Created: 2007-10-23 10:07:39 EDT

Size: 1.93 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.test.performance
>Index: src/org/eclipse/test/internal/performance/results/BuildResults.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.test.performance/src/org/eclipse/test/internal/performance/results/BuildResults.java,v
>retrieving revision 1.1
>diff -u -r1.1 BuildResults.java
>--- src/org/eclipse/test/internal/performance/results/BuildResults.java	19 Oct 2007 14:19:07 -0000	1.1
>+++ src/org/eclipse/test/internal/performance/results/BuildResults.java	23 Oct 2007 14:03:29 -0000
>@@ -61,6 +61,7 @@
> 			PerformanceTestPlugin.log(status);
> 		}
> 		*/
>+		this.average[dim_id] = 0;
> 		for (int i=0; i<vLength; i++) {
> 			this.average[dim_id] += values[dim_id][i];
> 		}
>@@ -342,6 +343,12 @@
> 		this.stddev = new double[length];
> 		this.count = new long[length];
> 		this.dimensions[0] = dimension;
>+		for (int i=0; i<length; i++) {
>+			// init average numbers with an impossible value
>+			// to clearly identify whether it's already set or not
>+			// when several measures are made for the same build
>+			this.average[i] = -1;
>+		}
> 	} else {
> 		length = this.dimensions.length;
> 		for (int i=0; i<length; i++) {
>@@ -358,18 +365,18 @@
> 	}
> 	switch (step) {
> 		case InternalPerformanceMeter.AVERAGE:
>-			if (this.average[idx] != 0) {
>+			if (this.average[idx] != -1) {
> 				if (values == null) {
> 					values = new double[length][];
> 					values[idx] = new double[2];
> 					values[idx][0] = this.average[idx];
> 					values[idx][1] = value;
>-					this.average[idx] = 0;
>+					this.average[idx] = -1;
> 				} else if (this.values[idx] == null) {
> 					values[idx] = new double[2];
> 					values[idx][0] = this.average[idx];
> 					values[idx][1] = value;
>-					this.average[idx] = 0;
>+					this.average[idx] = -1;
> 				}
> 			} else if (this.values != null && this.values[idx] != null) {
> 				int vLength = values[idx].length;

Actions: View | Diff

Attachments on bug 207164: 80952