tmf.ui: merge nested if statement

Change-Id: I42d9ef509d0be765118eca25e90668c99e5a07f5
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/44507
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

diff --git a/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/project/handlers/OpenAction.java b/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/project/handlers/OpenAction.java
index 4708b23d370e3878ab835f1ad7ec320cd9ae23b1..f7bec56714c922fd45c2fa599d076c6d446b3836 100644 (file)
--- a/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/project/handlers/OpenAction.java
+++ b/org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/internal/tmf/ui/project/handlers/OpenAction.java
@@ -54,13 +54,12 @@ public class OpenAction extends Action {
         ISelection selection = selectionProvider.getSelection();
         if (!selection.isEmpty()) {
             IStructuredSelection sSelection = (IStructuredSelection) selection;
-            if (sSelection.size() == 1) {
-                if (sSelection.getFirstElement() instanceof TmfTraceElement ||
-                        sSelection.getFirstElement() instanceof TmfExperimentElement ||
-                        sSelection.getFirstElement() instanceof TmfAnalysisOutputElement) {
-                    element = (TmfProjectModelElement) sSelection.getFirstElement();
-                    return true;
-                }
+            Object firstElement = sSelection.getFirstElement();
+            if ((sSelection.size() == 1) && (firstElement instanceof TmfTraceElement ||
+                    firstElement instanceof TmfExperimentElement ||
+                    firstElement instanceof TmfAnalysisOutputElement)) {
+                element = (TmfProjectModelElement) firstElement;
+                return true;
             }
         }
         return false;