Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 108030

Summary: [refactoring] Move instance method inserts argument at wrong position
Product: [Eclipse Project] JDT Reporter: Markus Keller <markus.kell.r>
Component: UIAssignee: Tobias Widmer <tobias_widmer>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P2 CC: bmiller, martinae
Version: 3.2   
Target Milestone: 3.2 RC2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on: 36032    
Bug Blocks:    

Description Markus Keller CLA 2005-08-25 14:36:54 EDT
I20050823-0800

class A {
    B fB;
    
    public void doit(String doitArg) {
        subroutine(1.2f);
    }
    
    public void subroutine(float subArg) {
        subsub();
    }

    public void subsub() {
        
    }
    
}

class B {
    
}

- move doit(..) to fB
  -> compiles
- move subroutine(..) to fB
  -> does not compile since subroutine's new parameter  A a is at first position
but the argument at the call site is at the second position

Please fix and add a second test case where methods have 2 or more parameters.
Comment 1 Tobias Widmer CLA 2005-08-26 05:02:59 EDT
Filed related bug 108088
Comment 2 Tobias Widmer CLA 2006-04-26 09:13:44 EDT
Fixed by fix for 36032

Added regression test MoveInstanceMethodTests#test36
Comment 3 Johannes Rieken CLA 2006-05-10 03:49:11 EDT
verifying for RC3
Comment 4 Johannes Rieken CLA 2006-05-10 03:54:30 EDT
verified