cleanup
[idea/community.git] / platform / vcs-impl / src / com / intellij / openapi / vcs / changes / committed / CommittedChangesTreeBrowser.java
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);