Removed unused "RefreshablePanel" methods
authorKonstantin Kolosovsky <konstantin.kolosovsky@jetbrains.com>
Fri, 4 Nov 2016 19:20:09 +0000 (22:20 +0300)
committerKonstantin Kolosovsky <konstantin.kolosovsky@jetbrains.com>
Wed, 16 Nov 2016 18:01:56 +0000 (21:01 +0300)
platform/lang-impl/src/com/intellij/application/options/InitialConfigurationDialog.java
platform/platform-impl/src/com/intellij/openapi/vcs/changes/RefreshablePanel.java
platform/vcs-impl/src/com/intellij/openapi/vcs/changes/AbstractRefreshablePanel.java
platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/CommitChangeListDialog.java
plugins/svn4idea/src/org/jetbrains/idea/svn/difftool/SvnTreeConflictDiffRequestProvider.java
plugins/svn4idea/src/org/jetbrains/idea/svn/treeConflict/TreeConflictRefreshablePanel.java

index 7d61d23ac2ac2c8cde1b1e5ad5ec80b01ebb24fb..a87f4608a3db8c52185a15c22787ab310fe30f3b 100644 (file)
@@ -329,19 +329,6 @@ public class InitialConfigurationDialog extends DialogWrapper {
       updateColorSchemePreview(false);
     }
 
-    @Override
-    public boolean refreshDataSynch() {
-      return false;
-    }
-
-    @Override
-    public void dataChanged() {}
-
-    @Override
-    public boolean isStillValid(Object o) {
-      return false;
-    }
-
     @Override
     public void refresh() {
       updateColorSchemePreview(false);
@@ -352,9 +339,6 @@ public class InitialConfigurationDialog extends DialogWrapper {
       return (JPanel)myPreviewEditor.getPanel();
     }
 
-    @Override
-    public void away() {}
-
     @Override
     public void dispose() {
       disposeUIResources();
index a7d7b0cb6f0257437c34019b257001814492bd29..8513a87f59ebc3e501f1a4ac9a372f8dd0d9eea2 100644 (file)
@@ -4,16 +4,7 @@ import com.intellij.openapi.Disposable;
 
 import javax.swing.*;
 
-/**
- * @author irengrig
- *         Date: 8/12/11
- *         Time: 6:47 PM
- */
-public interface RefreshablePanel<Data> extends Disposable {
-  boolean refreshDataSynch();
-  void dataChanged();
+public interface RefreshablePanel extends Disposable {
   void refresh();
   JPanel getPanel();
-  void away();
-  boolean isStillValid(Data data);
 }
index f497599fe1d4bbc111cbecdbad9bfe7aef6757e1..93192c6f897a7116def4b2cb06f89425b954088b 100644 (file)
@@ -40,7 +40,7 @@ import javax.swing.*;
  * @author Irina.Chernushina
  * @since 7.09.2011
  */
-public abstract class AbstractRefreshablePanel<T> implements RefreshablePanel<Change> {
+public abstract class AbstractRefreshablePanel<T> implements RefreshablePanel {
   private static final Logger LOG = Logger.getInstance("#com.intellij.openapi.vcs.changes.AbstractRefreshablePanel");
 
   @NotNull private final Project myProject;
@@ -62,18 +62,6 @@ public abstract class AbstractRefreshablePanel<T> implements RefreshablePanel<Ch
     myDetailsPanel.layout();
   }
 
-  @Override
-  public boolean refreshDataSynch() {
-    return false;
-  }
-
-  @CalledInAwt
-  @Override
-  public void dataChanged() {
-    ApplicationManager.getApplication().assertIsDispatchThread();
-    myTicket.increment();
-  }
-
   @CalledInAwt
   @Override
   public void refresh() {
@@ -115,11 +103,6 @@ public abstract class AbstractRefreshablePanel<T> implements RefreshablePanel<Ch
     return myDetailsPanel.getPanel();
   }
 
-  @Override
-  public boolean isStillValid(Change data) {
-    return true;
-  }
-
   private class Loader extends ModalityIgnorantBackgroundableTask {
     private final Ticket myTicketCopy;
     private T myT;
index c758ea8cff313439793610cf44b3c7a513c3a138..e4b1fe1c5ef119978e06079825489cda77b4d4e2 100644 (file)
@@ -1045,15 +1045,6 @@ public class CommitChangeListDialog extends DialogWrapper implements CheckinProj
       protected RefreshablePanel createDetails() {
         final JPanel panel = JBUI.Panels.simplePanel(myDiffDetails.getComponent());
         return new RefreshablePanel() {
-          @Override
-          public boolean refreshDataSynch() {
-            return false;
-          }
-
-          @Override
-          public void dataChanged() {
-          }
-
           @Override
           public void refresh() {
           }
@@ -1063,15 +1054,6 @@ public class CommitChangeListDialog extends DialogWrapper implements CheckinProj
             return panel;
           }
 
-          @Override
-          public void away() {
-          }
-
-          @Override
-          public boolean isStillValid(Object o) {
-            return false;
-          }
-
           @Override
           public void dispose() {
           }
index 918985fcdb3c950e54070f4a0dd684fda181f664..e981eb9375da9d412ca64efa05d01c43e1dd1a87 100644 (file)
@@ -102,7 +102,7 @@ public class SvnTreeConflictDiffRequestProvider implements ChangeDiffRequestProv
 
       myQueue = new BackgroundTaskQueue(myContext.getProject(), "Loading change details");
 
-      // We don't need to listen on File/Document, because panel always will be the same for a single change (@see myDelegate.isStillValid())
+      // We don't need to listen on File/Document, because panel always will be the same for a single change.
       // And if Change will change - we'll create new DiffRequest and DiffViewer
       myDelegate =
         new TreeConflictRefreshablePanel(myContext.getProject(), "Loading tree conflict details", myQueue, myRequest.getChange());
index 075949b9f362d929e5b7fe7948640f8cfd337f8b..4dfd0808b00e0dcbeb70d25cb507a3e8d056b4d9 100644 (file)
@@ -91,17 +91,6 @@ public class TreeConflictRefreshablePanel extends AbstractRefreshablePanel {
     myRightRevisionsList = new TLongArrayList();
   }
 
-  @Override
-  public boolean isStillValid(final Change change) {
-    return change.isTreeConflict() && change instanceof ConflictedSvnChange &&
-           descriptionsEqual(((ConflictedSvnChange)change).getBeforeDescription(), myChange.getBeforeDescription());
-  }
-
-  @Override
-  public boolean refreshDataSynch() {
-    return true;
-  }
-
   public static boolean descriptionsEqual(TreeConflictDescription d1, TreeConflictDescription d2) {
     if (d1.isPropertyConflict() != d2.isPropertyConflict()) return false;
     if (d1.isTextConflict() != d2.isTextConflict()) return false;
@@ -431,10 +420,6 @@ public class TreeConflictRefreshablePanel extends AbstractRefreshablePanel {
     Disposer.dispose(myChildDisposables);
   }
 
-  @Override
-  public void away() {
-  }
-
   private interface ConflictSidePresentation extends Disposable {
     JPanel createPanel();
     void load() throws SVNException, VcsException;