Attachment 165690 Details for Bug 306172 – Proposed patch for perf_35x

[patch] Proposed patch for perf_35x

patch35.txt (text/plain), 2.37 KB, created by Satyam Kandula

on 2010-04-22 01:31:22 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Satyam Kandula

Created: 2010-04-22 01:31:22 EDT

Size: 2.37 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core.tests.performance
>Index: src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.performance/src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java,v
>retrieving revision 1.13.6.2
>diff -u -r1.13.6.2 FullSourceWorkspaceTypeHierarchyTests.java
>--- src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java	11 Feb 2010 17:13:43 -0000	1.13.6.2
>+++ src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java	21 Apr 2010 18:31:08 -0000
>@@ -40,7 +40,7 @@
> 	}
> 
> 	static {
>-//		TESTS_NAMES = new String[] { "testPerfClassWithPotentialSubinterfaces" };
>+		TESTS_NAMES = new String[] { "testPerSuperTypes" };
> 	}
> 	public static Test suite() {
>         Test suite = buildSuite(testClass());
>@@ -154,15 +154,13 @@
> 	
> 	// Test for https://bugs.eclipse.org/bugs/show_bug.cgi?id=301438
> 	public void testPerSuperTypes() throws CoreException {
>-		tagAsSummary("Type hierarchy", true); // put in fingerprint
>-		ICompilationUnit unit = getCompilationUnit("org.eclipse.jdt.core", "org.eclipse.jdt.internal.compiler.parser", "Parser.java");
>-		assertNotNull("Parser not found!", unit);
>+		assertNotNull("Parser not found!", PARSER_WORKING_COPY);
> 
> 		// Warm up
>-		for (int i=0; i<WARMUP_COUNT; i++) {
>-			IType[] types = unit.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
>+		for (int i=0; i<10*WARMUP_COUNT; i++) { // More Warm up is required.
>+			IType[] types = PARSER_WORKING_COPY.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
> 			if (i==0) {
>-				System.out.println("  - "+INT_FORMAT.format(types.length)+" all classes found in hierarchy.");
>+				System.out.println("  - "+INT_FORMAT.format(types.length)+" classes found in hierarchy.");
> 			}
> 		}
> 
>@@ -173,7 +171,9 @@
> 		for (int i=0; i<MEASURES_COUNT; i++) {
> 			runGc();
> 			startMeasuring();
>-			unit.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
>+			for (int j =0; j < 20; j++) {
>+				PARSER_WORKING_COPY.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
>+			}
> 			stopMeasuring();
> 		}
> 
>@@ -181,5 +181,4 @@
> 		commitMeasurements();
> 		assertPerformance();
> 	}
>-
> }

Flags:

frederic_fusier: iplog+
frederic_fusier: review+

Actions: View | Diff

Attachments on bug 306172: 165527 | 165689 | 165690