Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPierre-Charles David2015-04-17 12:22:05 +0000
committerPierre-Charles David2015-04-21 06:14:43 +0000
commitb4e6e96b74f2685cc8755daa472131c543603c93 (patch)
tree275a332d50a5f286b41d43cca81aaf2e09f2fc2b
parentfc45c0224943725e31d8b5ce611999df763b8ecf (diff)
downloadorg.eclipse.sirius-b4e6e96b74f2685cc8755daa472131c543603c93.tar.gz
org.eclipse.sirius-b4e6e96b74f2685cc8755daa472131c543603c93.tar.xz
org.eclipse.sirius-b4e6e96b74f2685cc8755daa472131c543603c93.zip
[427799] Remove redundant test in DASI.open()
SessionManagerImpl.add() already checks if the session is registered and is a no-op in this case; no need for the caller-side check. Bug: 427799 Change-Id: I3610e690255bc6a194c9d6c0b5aff37427e48398 Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
-rw-r--r--plugins/org.eclipse.sirius/src/org/eclipse/sirius/business/internal/session/danalysis/DAnalysisSessionImpl.java4
1 files changed, 1 insertions, 3 deletions
diff --git a/plugins/org.eclipse.sirius/src/org/eclipse/sirius/business/internal/session/danalysis/DAnalysisSessionImpl.java b/plugins/org.eclipse.sirius/src/org/eclipse/sirius/business/internal/session/danalysis/DAnalysisSessionImpl.java
index 055de66dcd..106afdb4e4 100644
--- a/plugins/org.eclipse.sirius/src/org/eclipse/sirius/business/internal/session/danalysis/DAnalysisSessionImpl.java
+++ b/plugins/org.eclipse.sirius/src/org/eclipse/sirius/business/internal/session/danalysis/DAnalysisSessionImpl.java
@@ -1206,9 +1206,7 @@ public class DAnalysisSessionImpl extends DAnalysisSessionEObjectImpl implements
public void open(IProgressMonitor monitor) {
try {
monitor.beginTask("Open session", 33);
- if (!SessionManager.INSTANCE.getSessions().contains(this)) {
- SessionManager.INSTANCE.add(this);
- }
+ SessionManager.INSTANCE.add(this);
monitor.worked(1);
notifyListeners(SessionListener.OPENING);
monitor.worked(1);

Back to the top