Attachment 176382 Details for Bug 321115 – Revised patch

[patch] Revised patch

patch.txt (text/plain), 2.05 KB, created by Srikanth Sankaran

on 2010-08-11 13:52:00 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Srikanth Sankaran

Created: 2010-08-11 13:52:00 EDT

Size: 2.05 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: batch/org/eclipse/jdt/internal/compiler/batch/ClasspathDirectory.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/batch/org/eclipse/jdt/internal/compiler/batch/ClasspathDirectory.java,v
>retrieving revision 1.56
>diff -u -r1.56 ClasspathDirectory.java
>--- batch/org/eclipse/jdt/internal/compiler/batch/ClasspathDirectory.java	7 Jan 2010 20:18:49 -0000	1.56
>+++ batch/org/eclipse/jdt/internal/compiler/batch/ClasspathDirectory.java	11 Aug 2010 17:47:53 -0000
>@@ -115,6 +115,13 @@
> 	if (binaryExists) {
> 		try {
> 			ClassFileReader reader = ClassFileReader.read(this.path + qualifiedBinaryFileName);
>+			// https://bugs.eclipse.org/bugs/show_bug.cgi?id=321115, package names are to be treated case sensitive.
>+			String typeSearched = qualifiedPackageName.length() > 0 ? 
>+					qualifiedPackageName.replace(File.separatorChar, '/') + "/" + fileName //$NON-NLS-1$
>+					: fileName;
>+			if (!CharOperation.equals(reader.getName(), typeSearched.toCharArray())) {
>+				reader = null;
>+			}
> 			if (reader != null)
> 				return new NameEnvironmentAnswer(
> 						reader,
>#P org.eclipse.jdt.core.tests.compiler
>Index: src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java,v
>retrieving revision 1.208
>diff -u -r1.208 BatchCompilerTest.java
>--- src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java	1 Jun 2010 15:23:24 -0000	1.208
>+++ src/org/eclipse/jdt/core/tests/compiler/regression/BatchCompilerTest.java	11 Aug 2010 17:50:14 -0000
>@@ -3109,6 +3109,7 @@
> 		this.runConformTest(
> 			new String[] {
> 				"p/Y.java",
>+				"package p;\n" +
> 				"public class Y { public static final String S = \"\"; }",
> 			},
> 	        "\"" + OUTPUT_DIR +  File.separator + "p" + File.separator + "Y.java\""

Actions: View | Diff

Attachments on bug 321115: 175395 | 175400 | 176214 | 176298 | 176382 | 176430