From 25aa86d80e5bd20d313ff63e63c2f50d06bdb57d Mon Sep 17 00:00:00 2001 From: David Dykstal Date: Wed, 14 Jun 2006 17:53:20 +0000 Subject: [PATCH] Documenting future work. --- .../eclipse/rse/internal/model/SystemHostPool.java | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/rse/plugins/org.eclipse.rse.ui/model/org/eclipse/rse/internal/model/SystemHostPool.java b/rse/plugins/org.eclipse.rse.ui/model/org/eclipse/rse/internal/model/SystemHostPool.java index 537c8471bf4..d9c8e53fc99 100644 --- a/rse/plugins/org.eclipse.rse.ui/model/org/eclipse/rse/internal/model/SystemHostPool.java +++ b/rse/plugins/org.eclipse.rse.ui/model/org/eclipse/rse/internal/model/SystemHostPool.java @@ -32,7 +32,12 @@ import org.eclipse.rse.ui.SystemResources; /** * A pool of host objects. * There is one pool per profile. - * + * It is named the same as its owning profile. + */ +/* + * DWD this may be a candidate for elimination. It is not persisted but derived + * when Host objects come into existance. Not sure it provides much value. Code + * could be implemented directly in the profile. */ public class SystemHostPool extends RSEModelObject implements ISystemHostPool { @@ -452,7 +457,12 @@ public class SystemHostPool extends RSEModelObject implements ISystemHostPool */ private void moveConnection(IHost conn, int newPos) { - java.util.List connList = getHostList(); + /* + * DWD revisit, make sure that connections can be "moved", whatever that means. + * It appears that connections can be moved up and down in the list which + * probably provides for some rational for keeping this around. + */ +// java.util.List connList = getHostList(); DWD temporarily(?) removed. //FIXME connList.move(newPos, conn); invalidateCache(); }