Description Markus Schorn 2011-04-29 04:48:03 EDT

When using the 'Extract Function' refactoring a dialog is shown. I suggest the following improvements:

(1) There is a group named 'Return value:', however it primarily deals with the
    parameters.

(2) The option 'No return value (void)' is confusing: When I do not check this
    option and do not select any of the radio buttons in the 'return' column,
    I will still get no return value.
    It looks like the only purpose of the option is to allow for deselecting
    the radio button in the return column, once it has been selected.
    --> I suggest to remove this option and let the user deselect a radio button
        by clicking on it again. 

(3) For c++ it be nice to have the option to use a pointer type rather than a
    reference type for arguments passed by-reference.

(4) It'd be nice to show a preview of the function declaration.