Bug 6605 - refactoring: reverse conditional [refactoring]
Summary: refactoring: reverse conditional [refactoring]
Status: RESOLVED WONTFIX
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P4 enhancement with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 23087 47790 (view as bug list)
Depends on:
Blocks:
 
Reported: 2001-12-05 13:32 EST by Adam Kiezun CLA
Modified: 2009-08-30 02:40 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Kiezun CLA 2001-12-05 13:32:05 EST
given the selected code:
if (condition){
 m();
} else {
 n();
}

it could produce the following:

if (!condition){
 n();
} else {
 m();
}

with possible optional simplyfication of !condition
i mean, it could apply deMorgan's law (sometimes and on demand)
Comment 1 Adam Kiezun CLA 2002-03-09 18:40:23 EST
to be reconsidered after 2.0
Comment 2 Dirk Baeumer CLA 2003-02-17 14:12:07 EST
*** Bug 23087 has been marked as a duplicate of this bug. ***
Comment 3 Jim Kingdon CLA 2003-03-12 11:11:40 EST
See also bug 23261 which might be a generalization of this (depending on whether
"boolean" in that bug is an expression or a variable and other details).
Comment 4 Dirk Baeumer CLA 2003-04-24 11:32:19 EDT
This should be more a quick assist then a refactoring.
Comment 5 Dirk Baeumer CLA 2003-04-28 06:03:01 EDT
Chaning state from assigned later to resolved later. Assigned later got 
introduced by the last bug conversion and is not a supported Eclipse bug state.
Comment 6 Adam Kiezun CLA 2003-10-03 15:26:46 EDT
can we revive this for quick assist? seems easy to implement 
and i miss it a lot sometimes (a pain to do by hand)

refactoring is *way* too heavyweight here , but it would make a perfect quick assist
Comment 7 Dirk Baeumer CLA 2003-12-01 12:48:02 EST
*** Bug 47790 has been marked as a duplicate of this bug. ***
Comment 8 Ilja Preuss CLA 2004-08-03 04:40:41 EDT
The Patch of Bug 71244 seems to implement (part of) this request.
Comment 9 Eclipse Webmaster CLA 2009-08-30 02:40:36 EDT
As of now 'LATER' and 'REMIND' resolutions are no longer supported.
Please reopen this bug if it is still valid for you.