Attachment 54423 Details for Bug 163680 – Patch for 3.2.x stream

[patch] Patch for 3.2.x stream

patch.txt (text/plain), 2.62 KB, created by Kent Johnson

on 2006-11-23 12:29:12 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Kent Johnson

Created: 2006-11-23 12:29:12 EST

Size: 2.62 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/lookup/BinaryTypeBinding.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/lookup/BinaryTypeBinding.java,v
>retrieving revision 1.95.4.2
>diff -u -r1.95.4.2 BinaryTypeBinding.java
>--- compiler/org/eclipse/jdt/internal/compiler/lookup/BinaryTypeBinding.java	6 Sep 2006 18:11:37 -0000	1.95.4.2
>+++ compiler/org/eclipse/jdt/internal/compiler/lookup/BinaryTypeBinding.java	23 Nov 2006 17:21:01 -0000
>@@ -285,7 +285,7 @@
> 		if (wrapper.signature[wrapper.start] == '<') {
> 			// ParameterPart = '<' ParameterSignature(s) '>'
> 			wrapper.start++; // skip '<'
>-			this.typeVariables = createTypeVariables(wrapper, this);
>+			this.typeVariables = createTypeVariables(wrapper, true);
> 			wrapper.start++; // skip '>'
> 			this.tagBits |=  TagBits.HasUnresolvedTypeVariables;
> 			this.modifiers |= ExtraCompilerModifiers.AccGenericSignature;
>@@ -436,7 +436,7 @@
> 			// <A::Ljava/lang/annotation/Annotation;>(Ljava/lang/Class<TA;>;)TA;
> 			// ParameterPart = '<' ParameterSignature(s) '>'
> 			wrapper.start++; // skip '<'
>-			typeVars = createTypeVariables(wrapper, this);
>+			typeVars = createTypeVariables(wrapper, false);
> 			wrapper.start++; // skip '>'
> 		}
> 
>@@ -561,7 +561,7 @@
> 		}
> 	}
> }
>-private TypeVariableBinding[] createTypeVariables(SignatureWrapper wrapper, Binding declaringElement) {
>+private TypeVariableBinding[] createTypeVariables(SignatureWrapper wrapper, boolean assignVariables) {
> 	// detect all type variables first
> 	char[] typeSignature = wrapper.signature;
> 	int depth = 0, length = typeSignature.length;
>@@ -588,7 +588,7 @@
> 						pendingVariable = false;
> 						int colon = CharOperation.indexOf(':', typeSignature, i);
> 						char[] variableName = CharOperation.subarray(typeSignature, i, colon);
>-						variables.add(new TypeVariableBinding(variableName, declaringElement, rank++));
>+						variables.add(new TypeVariableBinding(variableName, this, rank++));
> 					}
> 			}
> 		}
>@@ -596,6 +596,10 @@
> 	// initialize type variable bounds - may refer to forward variables
> 	TypeVariableBinding[] result;
> 	variables.toArray(result = new TypeVariableBinding[rank]);
>+	// when creating the type variables for a type, the type must remember them before initializing each variable
>+	// see https://bugs.eclipse.org/bugs/show_bug.cgi?id=163680
>+	if (assignVariables)
>+		this.typeVariables = result;
> 	for (int i = 0; i < rank; i++) {
> 		initializeTypeVariable(result[i], result, wrapper);
> 	}

Actions: View | Diff

Attachments on bug 163680: 54422 | 54423