make shorter. avoid explicit array creation
authorKonstantin Bulenkov <kb@jetbrains.com>
Mon, 12 Apr 2010 20:06:24 +0000 (00:06 +0400)
committerKonstantin Bulenkov <kb@jetbrains.com>
Mon, 12 Apr 2010 20:06:24 +0000 (00:06 +0400)
platform/platform-api/src/com/intellij/util/OpenSourceUtil.java
platform/platform-impl/src/com/intellij/ide/actions/BaseNavigateToSourceAction.java
platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/EditSourceInCommitAction.java
plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/cvsBrowser/ui/BrowserPanel.java

index a02c96569c0cf95af4bafe35f72fdb68677c2eeb..50cef3380f9e507c373e2440dc69985ddb58c1b3 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2009 JetBrains s.r.o.
+ * Copyright 2000-2010 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,19 +26,28 @@ public class OpenSourceUtil {
   }
 
   public static void openSourcesFrom(DataContext context, boolean requestFocus) {
-    navigate(PlatformDataKeys.NAVIGATABLE_ARRAY.getData(context), requestFocus);
+    navigate(requestFocus, PlatformDataKeys.NAVIGATABLE_ARRAY.getData(context));
   }
 
   public static void openSourcesFrom(DataProvider context, boolean requestFocus) {
-    navigate(PlatformDataKeys.NAVIGATABLE_ARRAY.getData(context), requestFocus);
+    navigate(requestFocus, PlatformDataKeys.NAVIGATABLE_ARRAY.getData(context));
   }
 
-  public static void navigate(final Navigatable[] navigatables, final boolean requestFocus) {
-    if (navigatables != null) {
-      for (Navigatable navigatable : navigatables) {
-        if (navigatable.canNavigate()) {
-          navigatable.navigate(requestFocus);
-        }
+  /**
+   * Equivalent to navigate(true, navigatables)
+   *
+   * @param navigatables elements navigate to
+   * 
+   * @see OpenSourceUtil#navigate(boolean, com.intellij.pom.Navigatable...)  
+   */
+  public static void navigate(final Navigatable...navigatables) {
+    navigate(true, navigatables);
+  }
+
+  public static void navigate(final boolean requestFocus, final Navigatable...navigatables) {
+    for (Navigatable navigatable : navigatables) {
+      if (navigatable.canNavigate()) {
+        navigatable.navigate(requestFocus);
       }
     }
   }
index 981083b3f3f9be733fc5f6539a77bba56b604898..e8d0a4c66eaa8f209c19645a3003e207e9b46f9c 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2009 JetBrains s.r.o.
+ * Copyright 2000-2010 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,7 +30,7 @@ public abstract class BaseNavigateToSourceAction extends AnAction implements Dum
 
   public void actionPerformed(AnActionEvent e) {
     DataContext dataContext = e.getDataContext();
-    OpenSourceUtil.navigate(getNavigatables(dataContext), myFocusEditor);
+    OpenSourceUtil.navigate(myFocusEditor, getNavigatables(dataContext));
   }
 
 
index e50dd91f87c958c356b25a2a1ccdc1d6b86a7b0c..f774fe15d591b8896a6fc9876add59ed51c6b882 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2009 JetBrains s.r.o.
+ * Copyright 2000-2010 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -41,7 +41,7 @@ public class EditSourceInCommitAction extends AnAction {
     if (navigatableArray != null && navigatableArray.length > 0) {
       SwingUtilities.invokeLater(new Runnable() {
         public void run() {
-          OpenSourceUtil.navigate(navigatableArray, true);
+          OpenSourceUtil.navigate(navigatableArray);
         }
       });
       myDialogWrapper.doCancelAction();
index 274c8c8173d2dac06cd33d4b665b329937235d1a..ba052525938205c0083a855c62c4f19e05b4bcc7 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2009 JetBrains s.r.o.
+ * Copyright 2000-2010 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -104,7 +104,7 @@ public class BrowserPanel extends JPanel implements DataProvider, CvsTabbedWindo
     public void actionPerformed(AnActionEvent e) {
       final Navigatable[] navigatableArray = e.getData(PlatformDataKeys.NAVIGATABLE_ARRAY);
       if (navigatableArray != null && navigatableArray.length > 0) {
-        OpenSourceUtil.navigate(navigatableArray, true);
+        OpenSourceUtil.navigate(navigatableArray);
       }
     }