Attachment 165527 Details for Bug 306172 – Proposed patch

[patch] Proposed patch

patch.txt (text/plain), 2.09 KB, created by Satyam Kandula

on 2010-04-21 02:01:08 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Satyam Kandula

Created: 2010-04-21 02:01:08 EDT

Size: 2.09 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.15
>diff -u -r1.15 FullSourceWorkspaceTypeHierarchyTests.java
>--- src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java	11 Feb 2010 17:16:16 -0000	1.15
>+++ src/org/eclipse/jdt/core/tests/performance/FullSourceWorkspaceTypeHierarchyTests.java	21 Apr 2010 05:45:16 -0000
>@@ -154,15 +154,14 @@
> 	
> 	// 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);
>+		tagAsSummary("Super type hierarchy", true); // put in fingerprint
>+		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 +172,9 @@
> 		for (int i=0; i<MEASURES_COUNT; i++) {
> 			runGc();
> 			startMeasuring();
>-			unit.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
>+			for (int j =0; j < 10; j++) {
>+				PARSER_WORKING_COPY.getType("Parser").newSupertypeHierarchy(null).getAllClasses();
>+			}
> 			stopMeasuring();
> 		}
>

Flags:

frederic_fusier: review-

Actions: View | Diff

Attachments on bug 306172: 165527 | 165689 | 165690