mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-29 19:45:01 +02:00
Code Issues Wiki Activity

Restructure oddly and inefficiently constructed if/else section

Browse source
This commit is contained in:
John Cortell 2009-10-07 02:42:21 +00:00
parent e61498f16f
commit 1bfac4dbe8
1 changed files with 11 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/ui/viewmodel/DefaultVMModelProxyStrategy.java
View file

@ -371,20 +371,17 @@ public class DefaultVMModelProxyStrategy implements IVMModelProxy {
new DataRequestMonitor<IVMContext[]>(getVMProvider().getExecutor(), rm) { new DataRequestMonitor<IVMContext[]>(getVMProvider().getExecutor(), rm) {
@Override @Override
protected void handleCompleted() { protected void handleCompleted() {
if (isSuccess() || getStatus().getCode() == IDsfStatusConstants.NOT_SUPPORTED) { if (isSuccess()) {
assert getData() != null;
if (isSuccess()) { buildChildDeltasForEventContext(getData(), node, event, parentDelta, nodeOffset, rm);
assert getData() != null; }
buildChildDeltasForEventContext( else if (getStatus().getCode() == IDsfStatusConstants.NOT_SUPPORTED) {
getData(), node, event, parentDelta, nodeOffset, rm); // The DMC for this node was not found in the event. Call the
} else { // super-class to resort to the default behavior which will add a
// The DMC for this node was not found in the event. Call the // delta for every element in this node.
// super-class to resort to the default behavior which will add a buildChildDeltasForAllContexts(node, event, parentDelta, nodeOffset, rm);
// delta for every element in this node. }
buildChildDeltasForAllContexts( else {
node, event, parentDelta, nodeOffset, rm);
}
} else {
super.handleCompleted(); super.handleCompleted();
} }
} }