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.tests/ui/org/eclipse/jdt/ui/tests/quickfix/AssistQuickFixTest.java
buf.append("\n");
		ICompilationUnit cu= pack1.createCompilationUnit("E.java", buf.toString(), false, null);
		
		StringBuffer selection= new StringBuffer();
		selection.append("        int i = 10;\n");
		
		AssistContext context= getCorrectionContext(cu, buf.toString().indexOf(selection.toString()), selection.toString().length());
		List proposals= collectAssists(context, false);
		
		assertNumberOfProposals(proposals, 9);
		assertCorrectLabels(proposals);
		StringBuffer expected1= new StringBuffer();
		expected1.append("package test1;\n");
		expected1.append("public class E {\n");
		expected1.append("    public void foo() {\n");
		expected1.append("        int i;\n");
		expected1.append("        Runnable runnable = new Runnable() {\n");
		expected1.append("            public void run() {\n");
		expected1.append("                i = 10;\n");
		expected1.append("            }\n");
		expected1.append("        };\n");
		expected1.append("        System.out.println(i);\n");
		expected1.append("    }\n");
		expected1.append("}\n");
		expected1.append("\n");
		assertExpectedExistInProposals(proposals, new String[] {expected1.toString()});
	}
	
	public void testSurroundWithRunnable6() throws Exception {