Attachment 194284 Details for Bug 343621 – Second draft

[patch] Second draft

patch_343621_2.txt (text/plain), 7.92 KB, created by Olivier Thomann

on 2011-04-28 12:22:36 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Olivier Thomann

Created: 2011-04-28 12:22:36 EDT

Size: 7.92 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/CompilationResult.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/CompilationResult.java,v
>retrieving revision 1.65
>diff -u -r1.65 CompilationResult.java
>--- compiler/org/eclipse/jdt/internal/compiler/CompilationResult.java	11 May 2010 18:47:09 -0000	1.65
>+++ compiler/org/eclipse/jdt/internal/compiler/CompilationResult.java	28 Apr 2011 15:59:24 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -72,6 +72,7 @@
> 	public boolean hasSyntaxError = false;
> 	public char[][] packageName;
> 	public boolean checkSecondaryTypes = false; // check for secondary types which were created after the initial buildTypeBindings call
>+	public int numberOfErrors;
> 
> 	private static final int[] EMPTY_LINE_ENDS = Util.EMPTY_INT_ARRAY;
> 	private static final Comparator PROBLEM_COMPARATOR = new Comparator() {
>@@ -270,12 +271,7 @@
> }
> 
> public boolean hasErrors() {
>-	if (this.problems != null)
>-		for (int i = 0; i < this.problemCount; i++) {
>-			if (this.problems[i].isError())
>-				return true;
>-		}
>-	return false;
>+	return this.numberOfErrors != 0;
> }
> 
> public boolean hasProblems() {
>@@ -345,8 +341,14 @@
> 		if (newProblem.isError() && !referenceContext.hasErrors()) this.firstErrors.add(newProblem);
> 		this.problemsMap.put(newProblem, referenceContext);
> 	}
>-	if ((newProblem.getID() & IProblem.Syntax) != 0 && newProblem.isError())
>-		this.hasSyntaxError = true;
>+	if ((newProblem.getID() & IProblem.Syntax) != 0) {
>+		if (newProblem.isError()) {
>+			this.hasSyntaxError = true;
>+			this.numberOfErrors++;
>+		}
>+	} else if (newProblem.isError()) {
>+		this.numberOfErrors++;
>+	}
> }
> 
> /**
>Index: compiler/org/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration.java,v
>retrieving revision 1.91
>diff -u -r1.91 CompilationUnitDeclaration.java
>--- compiler/org/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration.java	25 Jun 2010 14:58:36 -0000	1.91
>+++ compiler/org/eclipse/jdt/internal/compiler/ast/CompilationUnitDeclaration.java	28 Apr 2011 15:59:24 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2000, 2010 IBM Corporation and others.
>+ * Copyright (c) 2000, 2011 IBM Corporation and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -16,6 +16,7 @@
> 
> import org.eclipse.jdt.core.compiler.CategorizedProblem;
> import org.eclipse.jdt.core.compiler.CharOperation;
>+import org.eclipse.jdt.core.compiler.IProblem;
> import org.eclipse.jdt.internal.compiler.ASTVisitor;
> import org.eclipse.jdt.internal.compiler.ClassFile;
> import org.eclipse.jdt.internal.compiler.CompilationResult;
>@@ -207,24 +208,56 @@
> }
> 
> public void finalizeProblems() {
>-	if (this.suppressWarningsCount == 0) return;
>+	if (this.suppressWarningsCount == 0) {
>+		if (this.compilationResult.hasErrors()) {
>+			// we need to check if we should discard unused locals warnings
>+			int removed = 0;
>+			CategorizedProblem[] problems = this.compilationResult.problems;
>+			int problemCount = this.compilationResult.problemCount;
>+			for (int i = 0; i < problemCount; i++) {
>+				if (problems[i].getID() == IProblem.LocalVariableIsNeverUsed) {
>+					problems[i] = null;
>+					removed++;
>+				}
>+			}
>+			// compact remaining problems
>+			if (removed > 0) {
>+				for (int i = 0, index = 0; i < problemCount; i++) {
>+					CategorizedProblem problem;
>+					if ((problem = problems[i]) != null) {
>+						if (i > index) {
>+							problems[index++] = problem;
>+						} else {
>+							index++;
>+						}
>+					}
>+				}
>+				this.compilationResult.problemCount -= removed;
>+			}
>+		}
>+		return;
>+	}
> 	int removed = 0;
> 	CategorizedProblem[] problems = this.compilationResult.problems;
> 	int problemCount = this.compilationResult.problemCount;
> 	IrritantSet[] foundIrritants = new IrritantSet[this.suppressWarningsCount];
> 	CompilerOptions options = this.scope.compilerOptions();
> 	boolean hasMandatoryErrors = false;
>+	int remainingErrors = 0;
> 	nextProblem: for (int iProblem = 0, length = problemCount; iProblem < length; iProblem++) {
> 		CategorizedProblem problem = problems[iProblem];
> 		int problemID = problem.getID();
> 		int irritant = ProblemReporter.getIrritant(problemID);
>-		if (problem.isError()) {
>+		boolean isError = problem.isError();
>+		if (isError) {
> 			if (irritant == 0) {
> 				// tolerate unused warning tokens when mandatory errors
> 				hasMandatoryErrors = true;
>+				remainingErrors++;
> 				continue;
> 			}
> 			if (!options.suppressOptionalErrors) {
>+				remainingErrors++;
> 				continue;
> 			}
> 		}
>@@ -234,13 +267,21 @@
> 			long position = this.suppressWarningScopePositions[iSuppress];
> 			int startSuppress = (int) (position >>> 32);
> 			int endSuppress = (int) position;
>-			if (start < startSuppress) continue nextSuppress;
>-			if (end > endSuppress) continue nextSuppress;
>-			if (!this.suppressWarningIrritants[iSuppress].isSet(irritant))
>+			if (start < startSuppress) {
> 				continue nextSuppress;
>+			}
>+			if (end > endSuppress) {
>+				continue nextSuppress;
>+			}
>+			if (!this.suppressWarningIrritants[iSuppress].isSet(irritant)) {
>+				continue nextSuppress;
>+			}
> 			// discard suppressed warning
> 			removed++;
> 			problems[iProblem] = null;
>+			if (isError) {
>+				this.compilationResult.numberOfErrors--;
>+			}
> 			if (this.compilationResult.problemsMap != null) this.compilationResult.problemsMap.remove(problem);
> 			if (this.compilationResult.firstErrors != null) this.compilationResult.firstErrors.remove(problem);
> 			if (foundIrritants[iSuppress] == null){
>@@ -250,6 +291,19 @@
> 			}
> 			continue nextProblem;
> 		}
>+		if (isError) {
>+			remainingErrors++;
>+		}
>+	}
>+	// we need to check if we should discard unused locals warnings that were not already filtered out
>+	if (remainingErrors > 0) {
>+		for (int i = 0; i < problemCount; i++) {
>+			CategorizedProblem problem;
>+			if ((problem = problems[i]) != null && problem.getID() == IProblem.LocalVariableIsNeverUsed) {
>+				problems[i] = null;
>+				removed++;
>+			}
>+		}
> 	}
> 	// compact remaining problems
> 	if (removed > 0) {
>Index: compiler/org/eclipse/jdt/internal/compiler/lookup/BlockScope.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/lookup/BlockScope.java,v
>retrieving revision 1.124
>diff -u -r1.124 BlockScope.java
>--- compiler/org/eclipse/jdt/internal/compiler/lookup/BlockScope.java	16 Feb 2011 08:09:18 -0000	1.124
>+++ compiler/org/eclipse/jdt/internal/compiler/lookup/BlockScope.java	28 Apr 2011 15:59:24 -0000
>@@ -200,9 +200,7 @@
> 
> 				if (!(local.declaration instanceof Argument)) // do not report unused catch arguments
> 					// https://bugs.eclipse.org/bugs/show_bug.cgi?id=336648
>-					if (!this.referenceCompilationUnit().compilationResult.hasErrors()) {
>-						problemReporter().unusedLocalVariable(local.declaration);
>-					}
>+					problemReporter().unusedLocalVariable(local.declaration);
> 			}
> 
> 			// could be optimized out, but does need to preserve unread variables ?

Actions: View | Diff

Attachments on bug 343621: 194283 | 194284 | 194745 | 194832 | 194931