Attachment 49214 Details for Bug 155887 – Proposed fix

[patch] Proposed fix

patch_for_155887.txt (text/plain), 1.41 KB, created by Olivier Thomann

on 2006-08-31 16:34:13 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Olivier Thomann

Created: 2006-08-31 16:34:13 EDT

Size: 1.41 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/ast/TryStatement.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/ast/TryStatement.java,v
>retrieving revision 1.100
>diff -u -r1.100 TryStatement.java
>--- compiler/org/eclipse/jdt/internal/compiler/ast/TryStatement.java	23 Aug 2006 16:32:56 -0000	1.100
>+++ compiler/org/eclipse/jdt/internal/compiler/ast/TryStatement.java	31 Aug 2006 20:23:18 -0000
>@@ -538,7 +538,7 @@
> 						naturalExitExceptionHandler.placeEnd();
> 						codeStream.recordPositionsFrom(
> 							position,
>-							this.finallyBlock.sourceEnd);	
>+							this.finallyBlock.sourceEnd);
> 						break;
> 					case FINALLY_INLINE :
> 						// inlined finally here can see all merged variables
>@@ -550,7 +550,12 @@
> 						// entire sequence for finally is associated to finally block
> 						this.finallyBlock.generateCode(this.scope, codeStream);
> 						if (postCatchesFinallyLabel != null && requiresCatchesExit) {
>+							position = codeStream.position;
>+							// entire sequence for finally is associated to finally block
> 							codeStream.goto_(postCatchesFinallyLabel);
>+							codeStream.recordPositionsFrom(
>+									position,
>+									this.finallyBlock.sourceEnd);
> 						}
> 						break;
> 					case FINALLY_DOES_NOT_COMPLETE :

Actions: View | Diff

Attachments on bug 155887: 49214 | 49227