Attachment 31816 Details for Bug 119175 – Proposed fix

[patch] Proposed fix

patch_119175.txt (text/plain), 1.42 KB, created by Olivier Thomann

on 2005-12-15 10:19:15 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Olivier Thomann

Created: 2005-12-15 10:19:15 EST

Size: 1.42 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/ast/ForeachStatement.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/ast/ForeachStatement.java,v
>retrieving revision 1.25
>diff -u -r1.25 ForeachStatement.java
>--- compiler/org/eclipse/jdt/internal/compiler/ast/ForeachStatement.java	18 Nov 2005 16:46:22 -0000	1.25
>+++ compiler/org/eclipse/jdt/internal/compiler/ast/ForeachStatement.java	15 Dec 2005 15:20:10 -0000
>@@ -261,7 +261,6 @@
> 		this.action.generateCode(scope, codeStream);
> 
> 		// continuation point
>-		int continuationPC = codeStream.position;
> 		if (this.continueLabel != null) {
> 			this.continueLabel.place();
> 			// generate the increments for next iteration
>@@ -276,6 +275,8 @@
> 		}
> 		// generate the condition
> 		conditionLabel.place();
>+		int conditionPC = codeStream.position;
>+		
> 		if (this.postCollectionInitStateIndex != -1) {
> 			codeStream.removeNotDefinitelyAssignedVariables(currentScope, postCollectionInitStateIndex);
> 		}
>@@ -292,7 +293,7 @@
> 				codeStream.ifne(actionLabel);
> 				break;
> 		}
>-		codeStream.recordPositionsFrom(continuationPC, this.elementVariable.sourceStart);
>+		codeStream.recordPositionsFrom(conditionPC, this.elementVariable.sourceStart);
> 
> 		breakLabel.place();
> 		codeStream.exitUserScope(scope);

Actions: View | Diff

Attachments on bug 119175: 31206 | 31816