[#IDEABKL-2960] Propagate parameters deletion

IDEA: Backlog

Propagate parameters deletion

Created: 23 Feb 05 13:58 Updated: 22 May 05 18:43

Component/s:

Editor. Intention Actions

Affects Version/s:

None

Fix Version/s:

None

Original Estimate:

Unknown

Remaining Estimate:

Unknown

Time Spent:

Unknown

Build:

3,229

Description

« Hide

Note: it should also work when deleting an unused parameter by intention.

Example:
---------

public void a () {
b("Forget it");
}

public b (String i_ignoredText) {
c (i_ignoredText);
}

public c (String i_ignoredText) {
System.println ("Hello world");
}

request:
--------
When deleting
c.i_ignoredText
this feature would offer to propagate and delete
b.i_ignoredText,

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Marcus Brito - 23 Feb 05 15:12

+1, this will diminish my pain after what was supposedly "fixed" after this thread:

http://www.intellij.net/forums/thread.jsp?forum=22&thread=128147

[ Permlink | « Hide ]

Andrei Oprea - 23 Feb 05 17:43

Yep, the delete should also be offered for propagation if possible. The selection screen would have to be a bit smarter, though, since you want to offer the ability to propagate it only if the caller method has no other use for the parameter (except for passing it along).

This would definitelly help with code cleaning, it gets my vote.