Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 259122 - Code-cleanup save action throws exception
Summary: Code-cleanup save action throws exception
Status: RESOLVED DUPLICATE of bug 257057
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.4.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-17 10:54 EST by Mike M CLA
Modified: 2008-12-17 11:31 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike M CLA 2008-12-17 10:54:00 EST
Build ID: M20080911-1700

Steps To Reproduce:
1. Enable save-action for Code Cleanup
2. Enable the "Use blocks in if/while/for/do statements" -> Always option
3. Enter the code below and save
4. An error occurs

Code to test (notice the syntax error on line 5 - missing right parenthesis):
public class T
{
    public void method1()
    {
        if( method2( 0 ) {
            
        }
    }

    public boolean method2( int i )
    {
        return false;
    }
}

Relevent part of stack trace:

org.eclipse.core.runtime.AssertionFailedException: assertion failed:
at org.eclipse.core.runtime.Assert.isTrue(Assert.java:111)
at org.eclipse.core.runtime.Assert.isTrue(Assert.java:97)
at org.eclipse.text.edits.TextEdit.<init>(TextEdit.java:148)
at org.eclipse.text.edits.DeleteEdit.<init>(DeleteEdit.java:33)
at org.eclipse.jdt.internal.core.dom.rewrite.ASTRewriteAnalyzer.doTextRemove(ASTRewriteAnalyzer.java:270)
at org.eclipse.jdt.internal.core.dom.rewrite.ASTRewriteAnalyzer.doTextRemoveAndVisit(ASTRewriteAnalyzer.java:279)
at org.eclipse.jdt.internal.core.dom.rewrite.ASTRewriteAnalyzer.rewriteBodyNode(ASTRewriteAnalyzer.java:684)
at org.eclipse.jdt.internal.core.dom.rewrite.ASTRewriteAnalyzer.visit(ASTRewriteAnalyzer.java:2092)
at org.eclipse.jdt.core.dom.IfStatement.accept0(IfStatement.java:186)
at org.eclipse.jdt.core.dom.ASTNode.accept(ASTNode.java:2478)
at org.eclipse.jdt.core.dom.rewrite.ASTRewrite.internalRewriteAST(ASTRewrite.java:262)
at org.eclipse.jdt.core.dom.rewrite.ASTRewrite.rewriteAST(ASTRewrite.java:251)
at org.eclipse.jdt.internal.corext.refactoring.structure.CompilationUnitRewrite.attachChange(CompilationUnitRewrite.java:241)
at org.eclipse.jdt.internal.corext.refactoring.structure.CompilationUnitRewrite.createChange(CompilationUnitRewrite.java:206)
at org.eclipse.jdt.internal.corext.fix.CompilationUnitRewriteOperationsFix.createChange(CompilationUnitRewriteOperationsFix.java:102)
at org.eclipse.jdt.internal.corext.fix.CleanUpRefactoring.calculateChange(CleanUpRefactoring.java:784)
at org.eclipse.jdt.internal.corext.fix.CleanUpPostSaveListener.saved(CleanUpPostSaveListener.java:263)
Comment 1 Dani Megert CLA 2008-12-17 11:31:15 EST

*** This bug has been marked as a duplicate of bug 257057 ***