The behavior of this refactoring seems broken.  Given code such as:

void method1( int numB ) {
     int numA = getValue( numB );
     handle( numA );
}

if I select numA and "Refactor -> Replace Temp with Query I should get:


void method1( int numB ) {
     handle( getValue( numB ) );
}

What I do get is a dialog - and if I accept the defaults the code becomes

void method1( int numB ) {
     handle( numA( numB ) );
}

int numA( int NumB ) {
     return getValue( numB );
}

If the variable is set from an expression, it makes sense to extract that 
expression into a new method - but if it is set from a method call, that 
method call should simply replace all uses of the variable.


_______________________________________________
Eap-features mailing list
[EMAIL PROTECTED]
http://www.intellij.com/mailman/listinfo/eap-features

Reply via email to