Bug 108030 - [refactoring] Move instance method inserts argument at wrong position
Summary: [refactoring] Move instance method inserts argument at wrong position
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: 3.2 RC2   Edit
Assignee: Tobias Widmer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 36032
Blocks:
  Show dependency tree
 
Reported: 2005-08-25 14:36 EDT by Markus Keller CLA
Modified: 2006-05-10 04:37 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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