Description Markus Keller 2014-09-29 14:46:36 EDT

public class TestExtractLocalVariable {
    void foo() {
        int lonely = -1; // select whole line, try Extract Local Variable
    }
}

Ensure the problem for "Value of local variable is not used" is set to Warning or Error and then try to extract the line indicated above.

=> Error message talks about "compilation errors", which doesn't make sense.

The problem is the ASTNodes#getMessages(ASTNode, int) method, which rarely ever makes sense. We should at least use ASTNodes#getProblems(.., ERROR), or even better, a new severity FATAL_ERROR that only finds real errors and not problems that the user promoted to error (see JavaCore#getOptionForConfigurableSeverity()).