Attachment 136665 Details for Bug 277237 – patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] patch

dup-locals.patch (text/plain), 1.57 KB, created by Darin Wright

on 2009-05-21 11:50:23 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Darin Wright

Created: 2009-05-21 11:50:23 EDT

Size: 1.57 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.debug
>Index: eval/org/eclipse/jdt/internal/debug/eval/ast/engine/ASTEvaluationEngine.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.debug/eval/org/eclipse/jdt/internal/debug/eval/ast/engine/ASTEvaluationEngine.java,v
>retrieving revision 1.69
>diff -u -r1.69 ASTEvaluationEngine.java
>--- eval/org/eclipse/jdt/internal/debug/eval/ast/engine/ASTEvaluationEngine.java	20 Feb 2009 20:35:41 -0000	1.69
>+++ eval/org/eclipse/jdt/internal/debug/eval/ast/engine/ASTEvaluationEngine.java	21 May 2009 15:50:17 -0000
>@@ -11,7 +11,9 @@
> package org.eclipse.jdt.internal.debug.eval.ast.engine;
> 
> 
>+import java.util.HashSet;
> import java.util.Map;
>+import java.util.Set;
> import java.util.StringTokenizer;
> import java.util.regex.Matcher;
> import java.util.regex.Pattern;
>@@ -188,13 +190,16 @@
> 		try {
> 			IJavaVariable[] localsVar = context.getLocals();
> 			int numLocalsVar= localsVar.length;
>+			Set names = new HashSet();
> 			// ******
> 			// to hide problems with local variable declare as instance of Local Types
>+			// and to remove locals with duplicate names
> 			IJavaVariable[] locals= new IJavaVariable[numLocalsVar];
> 			int numLocals= 0;
> 			for (int i = 0; i < numLocalsVar; i++) {
>-				if (!isLocalType(localsVar[i].getSignature())) {
>+				if (!isLocalType(localsVar[i].getSignature()) && !names.contains(localsVar[i].getName())) {
> 					locals[numLocals++]= localsVar[i];
>+					names.add(localsVar[i].getName());
> 				}
> 			}
> 			// to solve and remove

Actions: View | Diff

Attachments on bug 277237: 136665