cleanup
authorAlexey Kudravtsev <cdr@intellij.com>
Tue, 6 Jul 2010 12:14:27 +0000 (16:14 +0400)
committerAlexey Kudravtsev <cdr@intellij.com>
Tue, 6 Jul 2010 12:53:00 +0000 (16:53 +0400)
platform/platform-api/src/com/intellij/ui/content/Content.java
platform/platform-impl/src/com/intellij/ui/content/impl/ContentImpl.java
platform/vcs-impl/src/com/intellij/openapi/vcs/changes/committed/CommittedChangesTreeBrowser.java
platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/ChangesViewContentManager.java

index 9ef8e14bd132bdfd079cf3b7b6e2d040489ef575..56cb366cabcf827db4e6ffb344d4df1db170f34d 100644 (file)
@@ -63,7 +63,7 @@ public interface Content extends UserDataHolder, ComponentContainer {
   Disposable getDisposer();
 
   /**
-   * @param disposer a Disposable object whoes dispose() method will be invoken upon this content release.
+   * @param disposer a Disposable object whoes dispose() method will be invoked upon this content release.
    */
   void setDisposer(Disposable disposer);
 
index 34570a2b290c1ff75372e6b7ae5e277116d51d49..743e7bddef856c488aae123de3b32393b961920c 100644 (file)
@@ -53,7 +53,6 @@ public class ContentImpl extends UserDataHolderBase implements Content {
 
   private AlertIcon myAlertIcon;
 
-  private final boolean myAlerting = false;
   private JComponent myActionsContextComponent;
   private JComponent mySearchComponent;
 
@@ -238,7 +237,7 @@ public class ContentImpl extends UserDataHolderBase implements Content {
 
   public void dispose() {
     if (myComponent instanceof Disposable) {
-      Disposer.dispose(((Disposable)myComponent));
+      Disposer.dispose((Disposable)myComponent);
     }
 
     myComponent = null;
index 232e66e78a91f9ca066420815b4c2d7a0b5d1b98..bca3f6f311c7d208e1c6051479265c5f8f8d018c 100644 (file)
@@ -172,10 +172,10 @@ public class CommittedChangesTreeBrowser extends JPanel implements TypeSafeDataP
   private TreeModel buildTreeModel(final List<CommittedChangeList> filteredChangeLists) {
     DefaultMutableTreeNode root = new DefaultMutableTreeNode();
     DefaultTreeModel model = new DefaultTreeModel(root);
-    DefaultMutableTreeNode lastGroupNode = null;
-    String lastGroupName = null;
     Collections.sort(filteredChangeLists, myGroupingStrategy.getComparator());
     myGroupingStrategy.beforeStart();
+    DefaultMutableTreeNode lastGroupNode = null;
+    String lastGroupName = null;
     for(CommittedChangeList list: filteredChangeLists) {
       String groupName = myGroupingStrategy.getGroupName(list);
       if (!Comparing.equal(groupName, lastGroupName)) {
@@ -374,7 +374,7 @@ public class CommittedChangesTreeBrowser extends JPanel implements TypeSafeDataP
     }
     else if (key.equals(VcsDataKeys.CHANGE_LISTS)) {
       final List<CommittedChangeList> lists = getSelectedChangeLists();
-      if (lists.size() > 0) {
+      if (!lists.isEmpty()) {
         sink.put(VcsDataKeys.CHANGE_LISTS, lists.toArray(new CommittedChangeList[lists.size()]));
       }
     }
@@ -432,7 +432,7 @@ public class CommittedChangesTreeBrowser extends JPanel implements TypeSafeDataP
   }
 
   public void append(final List<CommittedChangeList> list) {
-    final TreeState state = (myChangeLists.isEmpty() && myState != null) ? myState :
+    final TreeState state = myChangeLists.isEmpty() && myState != null ? myState :
       TreeState.createOn(myChangesTree, (DefaultMutableTreeNode)myChangesTree.getModel().getRoot());
     state.setScrollToSelection(false);
     myChangeLists.addAll(list);
index b4bd66f967dabae2a21387d77cc372e77d3383ca..a6e07245b5f1a0619477416207efe0f9efe0dc4f 100644 (file)
@@ -191,11 +191,6 @@ public class ChangesViewContentManager extends AbstractProjectComponent {
     myContentManager.setSelectedContent(content);
   }
 
-  @Nullable
-  public Content getSelectedContent() {
-    return myContentManager.getSelectedContent();
-  }
-
   @Nullable
   public <T> T getActiveComponent(final Class<T> aClass) {
     final Content content = myContentManager.getSelectedContent();
@@ -242,20 +237,21 @@ public class ChangesViewContentManager extends AbstractProjectComponent {
 
   private class MyContentManagerListener extends ContentManagerAdapter {
     public void selectionChanged(final ContentManagerEvent event) {
-      if (event.getContent().getComponent() instanceof ContentStub) {
-        ChangesViewContentEP ep = ((ContentStub) event.getContent().getComponent()).getEP();
+      Content content = event.getContent();
+      if (content.getComponent() instanceof ContentStub) {
+        ChangesViewContentEP ep = ((ContentStub) content.getComponent()).getEP();
         ChangesViewContentProvider provider = ep.getInstance(myProject);
         final JComponent contentComponent = provider.initContent();
-        event.getContent().setComponent(contentComponent);
+        content.setComponent(contentComponent);
         if (contentComponent instanceof Disposable) {
-          event.getContent().setDisposer((Disposable) contentComponent);          
+          content.setDisposer((Disposable) contentComponent);
         }
       }
     }
   }
 
-  private final static String[] ourPresetOrder = {"Local", "Repository", "Incoming", "Shelf"};
-  private List<Content> doPresetOrdering(final List<Content> contents) {
+  private static final String[] ourPresetOrder = {"Local", "Repository", "Incoming", "Shelf"};
+  private static List<Content> doPresetOrdering(final List<Content> contents) {
     final List<Content> result = new ArrayList<Content>(contents.size());
     for (final String preset : ourPresetOrder) {
       for (Iterator<Content> iterator = contents.iterator(); iterator.hasNext();) {