Bug 129876 [quick fix] Light bulb on unneccessary else statement suggests "surround with runnable"
When a statement is unnessarily nested within an else clause, quick fix suggests to "surround with runnable". Which is of course incorrect. I'm using 3.2m5, check the quick fix on the else. (Java->Compiler->Errors/Warnings->Unnessary else statement = Warning) package demo; public class TestElse { public void main(String args[]) { if (true) { return; } else { return; } } }
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/text/correction/QuickAssistProcessor.java
return false;
		final IBinding binding= name.resolveBinding();
		if (binding == null || !binding.isDeprecated())
			return false;
		String fileName= null;
		if (binding instanceof IVariableBinding)
			fileName= DelegateFieldCreator.getRefactoringScriptName((IVariableBinding) binding);
		else if (binding instanceof IMethodBinding) {
			fileName= DelegateMethodCreator.getRefactoringScriptName((IMethodBinding) binding);
		}
		if (fileName == null)
			return false;
		final ICompilationUnit unit= context.getCompilationUnit();
		if (unit == null || !unit.exists())
			return false;
		final IProject project= unit.getJavaProject().getProject();
		final IFile file= project.getFolder(DelegateCreator.SCRIPT_FOLDER).getFile(fileName);
		if (!file.exists())
			return false;
		if (resultingCollections == null)
			return true;
		final ChangeCorrectionProposal proposal= new ChangeCorrectionProposal(CorrectionMessages.QuickAssistProcessor_fix_deprecation_name, null, 2, JavaPluginImages.get(JavaPluginImages.IMG_CORRECTION_CHANGE)) {
			private static final int SIZING_WIZARD_HEIGHT= 470;
			private static final int SIZING_WIZARD_WIDTH= 490;
			public final void apply(final IDocument document) {
				RefactoringHistory history= null;
				InputStream stream= null;
				try {
					stream= new BufferedInputStream(file.getContents(true));
					history= RefactoringCore.getHistoryService().readRefactoringHistory(stream, JavaRefactoringDescriptor.DEPRECATION_RESOLVING);