Attachment 136699 Details for Bug 277372 – Proposed fix

[patch] Proposed fix

patch_277372.txt (text/plain), 1.79 KB, created by Olivier Thomann

on 2009-05-21 14:48:30 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Olivier Thomann

Created: 2009-05-21 14:48:30 EDT

Size: 1.79 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/ClassFile.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/ClassFile.java,v
>retrieving revision 1.188
>diff -u -r1.188 ClassFile.java
>--- compiler/org/eclipse/jdt/internal/compiler/ClassFile.java	12 May 2009 20:49:56 -0000	1.188
>+++ compiler/org/eclipse/jdt/internal/compiler/ClassFile.java	21 May 2009 18:48:16 -0000
>@@ -346,7 +346,7 @@
> 		}
> 		// add signature attribute
> 		char[] genericSignature = this.referenceBinding.genericSignature();
>-		if (genericSignature != null) {
>+		if (genericSignature != null && this.targetJDK >= ClassFileConstants.JDK1_5) {
> 			// check that there is enough space to write all the bytes for the field info corresponding
> 			// to the @fieldBinding
> 			if (this.contentsOffset + 8 >= this.contents.length) {
>@@ -652,7 +652,7 @@
> 		}
> 		// add signature attribute
> 		char[] genericSignature = fieldBinding.genericSignature();
>-		if (genericSignature != null) {
>+		if (genericSignature != null && this.targetJDK >= ClassFileConstants.JDK1_5) {
> 			// check that there is enough space to write all the bytes for the field info corresponding
> 			// to the @fieldBinding
> 			if (this.contentsOffset + 8 >= this.contents.length) {
>@@ -6269,7 +6269,7 @@
> 		}
> 		// add signature attribute
> 		char[] genericSignature = methodBinding.genericSignature();
>-		if (genericSignature != null) {
>+		if (genericSignature != null && this.targetJDK >= ClassFileConstants.JDK1_5) {
> 			// check that there is enough space to write all the bytes for the field info corresponding
> 			// to the @fieldBinding
> 			if (this.contentsOffset + 8 >= this.contents.length) {

Actions: View | Diff

Attachments on bug 277372: 136699