Inspect code
authorValentina Kiryushkina <valentina.kiryushkina@jetbrains.com>
Thu, 20 Oct 2016 15:09:04 +0000 (18:09 +0300)
committerValentina Kiryushkina <valentina.kiryushkina@jetbrains.com>
Mon, 24 Oct 2016 17:09:51 +0000 (20:09 +0300)
13 files changed:
python/ipnb/src/org/jetbrains/plugins/ipnb/configuration/IpnbConnectionManager.java
python/ipnb/src/org/jetbrains/plugins/ipnb/configuration/IpnbSettings.java
python/ipnb/src/org/jetbrains/plugins/ipnb/editor/IpnbEditorUtil.java
python/ipnb/src/org/jetbrains/plugins/ipnb/editor/actions/IpnbHideOutputAction.java
python/ipnb/src/org/jetbrains/plugins/ipnb/editor/panels/IpnbFilePanel.java
python/ipnb/src/org/jetbrains/plugins/ipnb/editor/panels/IpnbPanel.java
python/ipnb/src/org/jetbrains/plugins/ipnb/format/IpnbParser.java
python/ipnb/src/org/jetbrains/plugins/ipnb/format/cells/IpnbEditableCell.java
python/ipnb/src/org/jetbrains/plugins/ipnb/format/cells/IpnbHeadingCell.java
python/ipnb/src/org/jetbrains/plugins/ipnb/format/cells/IpnbMarkdownCell.java
python/ipnb/src/org/jetbrains/plugins/ipnb/format/cells/IpnbRawCell.java
python/ipnb/src/org/jetbrains/plugins/ipnb/protocol/IpnbConnection.java
python/ipnb/src/org/jetbrains/plugins/ipnb/psi/IpnbPyFileType.java

index 733db0cec8b81abf98ae7612eabfcbd7c13ed17b..b6bb566986c688821b1c6cc387002be2390ff087 100644 (file)
@@ -60,8 +60,8 @@ import java.util.Map;
 public final class IpnbConnectionManager implements ProjectComponent {
   private static final Logger LOG = Logger.getInstance(IpnbConnectionManager.class);
   private final Project myProject;
-  private Map<String, IpnbConnection> myKernels = new HashMap<>();
-  private Map<String, IpnbCodePanel> myUpdateMap = new HashMap<>();
+  private final Map<String, IpnbConnection> myKernels = new HashMap<>();
+  private final Map<String, IpnbCodePanel> myUpdateMap = new HashMap<>();
   private static final int MAX_ATTEMPTS = 10;
 
   public IpnbConnectionManager(final Project project) {
index 5dd54c6954a747620969629a45a9bea6b5d3ce4d..3806ef7deaec03c95c507b38b87f45ed0252dff7 100644 (file)
@@ -11,7 +11,7 @@ import org.jetbrains.annotations.Nullable;
 
 @State(name = "IpnbSettings")
 public class IpnbSettings implements PersistentStateComponent<IpnbSettings> {
-  public static String DEFAULT_URL = "http://127.0.0.1:8888";
+  public static final String DEFAULT_URL = "http://127.0.0.1:8888";
   public String URL = DEFAULT_URL;
   private String myWorkingDirectory;
 
index ee0874168f6ca89ce0f5ea4a5a40b7ce9947e3e8..336a41c767f10e81eae2ca7ea6da5adeaa802c41 100644 (file)
@@ -47,7 +47,7 @@ import java.util.List;
 public class IpnbEditorUtil {
   public enum PromptType {In, Out, None}
 
-  public static Dimension PROMPT_SIZE = new Dimension(80, 30);
+  public static final Dimension PROMPT_SIZE = new Dimension(80, 30);
 
   public static Editor createPythonCodeEditor(@NotNull final Project project, @NotNull final IpnbCodeSourcePanel codeSourcePanel) {
     final EditorFactory editorFactory = EditorFactory.getInstance();
index 3875c5e13b1e5a899c5bdf92e4b9363261e8500c..5f00581d109350b1f20863cca0ccedc9955135d9 100644 (file)
@@ -6,7 +6,7 @@ import org.jetbrains.annotations.NotNull;
 import org.jetbrains.plugins.ipnb.editor.panels.code.IpnbCodePanel;
 
 public class IpnbHideOutputAction extends AnAction {
-  private IpnbCodePanel myParent;
+  private final IpnbCodePanel myParent;
 
   public IpnbHideOutputAction(@NotNull IpnbCodePanel parent) {
     super("Toggle Output Button (Double-Click)");
index d1a14f91e5a14ebe3c28fb001c62b99db047c2f3..d4653a5f8d051ce0038013c0e9e002513055a423 100644 (file)
@@ -49,10 +49,10 @@ import java.util.stream.Collectors;
 public class IpnbFilePanel extends JPanel implements Scrollable, DataProvider, Disposable {
   private static final Logger LOG = Logger.getInstance(IpnbFilePanel.class);
   private final DocumentAdapter myDocumentListener;
-  private Document myDocument;
+  private final Document myDocument;
   private IpnbFile myIpnbFile;
-  private Project myProject;
-  @NotNull private IpnbFileEditor myParent;
+  private final Project myProject;
+  @NotNull private final IpnbFileEditor myParent;
   @NotNull private final VirtualFile myVirtualFile;
   @NotNull private final IpnbFileEditor.CellSelectionListener myListener;
 
@@ -255,7 +255,7 @@ public class IpnbFilePanel extends JPanel implements Scrollable, DataProvider, D
     }
     saveToFile(false);
   }
-  
+
   public void mergeCell(boolean below) {
     final IpnbEditablePanel currentCellPanel = getSelectedCellPanel();
     if (currentCellPanel == null) return;
@@ -373,9 +373,9 @@ public class IpnbFilePanel extends JPanel implements Scrollable, DataProvider, D
     }
   }
 
-  private boolean deleteCell(@NotNull final IpnbEditablePanel cell) {
+  private void deleteCell(@NotNull final IpnbEditablePanel cell) {
     final int index = myIpnbPanels.indexOf(cell);
-    if (index < 0) return true;
+    if (index < 0) return;
     myIpnbPanels.remove(index);
     myIpnbFile.removeCell(index);
     remove(index);
@@ -383,7 +383,6 @@ public class IpnbFilePanel extends JPanel implements Scrollable, DataProvider, D
     int indexToSelect = index < myIpnbPanels.size() ? index : index-1;
     final IpnbEditablePanel panel = myIpnbPanels.get(indexToSelect);
     setSelectedCell(panel);
-    return false;
   }
 
   public void saveToFile(boolean synchronize) {
index e6c569bf3a4f3de4f29f9f20963627c06187aa09..c97638893c2f2429b0e10f19468a1681a3adc5e8 100644 (file)
@@ -13,7 +13,7 @@ import java.awt.*;
 
 public abstract class IpnbPanel<T extends JComponent, K extends IpnbCell> extends JPanel {
   protected T myViewPanel;
-  protected K myCell;
+  protected final K myCell;
 
   public IpnbPanel(@NotNull final K cell) {
     super(new CardLayout());
index a83fd4347d0d96fb898bfed5b6bc8fb75ef886f1..f2acf5f5450a2c43317375425932064e554d6eea 100644 (file)
@@ -50,7 +50,7 @@ public class IpnbParser {
   }
 
   @NotNull
-  public static IpnbFile parseIpnbFile(@NotNull final CharSequence fileText, @NotNull final VirtualFile virtualFile) throws IOException {
+  public static IpnbFile parseIpnbFile(@NotNull final CharSequence fileText, @NotNull final VirtualFile virtualFile) {
     myErrors.clear();
 
     final String path = virtualFile.getPath();
@@ -193,7 +193,7 @@ public class IpnbParser {
   }
 
   private static class IpnbWorksheet {
-    List<IpnbCellRaw> cells = new ArrayList<>();
+    final List<IpnbCellRaw> cells = new ArrayList<>();
   }
 
   @SuppressWarnings("unused")
@@ -267,7 +267,6 @@ public class IpnbParser {
         cell = new IpnbCodeCell(language == null ? "python" : language,
                                 input == null ? (isValidSource ? source : Collections.emptyList()) : input,
                                 prompt, outputCells, metadata);
-        ;
       }
       else if (cell_type.equals("raw")) {
         cell = new IpnbRawCell(isValidSource ? source : Collections.emptyList());
index 9709befe6e5c6ef56f1ddd76a15e64c9ec0bda37..31ba128d3822f7a1598aeefe2ee5162352af3295 100644 (file)
@@ -8,7 +8,7 @@ import java.util.Map;
 
 public abstract class IpnbEditableCell implements IpnbCell {
   @NotNull private List<String> mySource;
-  @NotNull Map<String, Object> myMetadata;
+  @NotNull final Map<String, Object> myMetadata;
 
   IpnbEditableCell(@NotNull final List<String> source, @NotNull Map<String, Object> metadata) {
     mySource = source;
@@ -42,7 +42,7 @@ public abstract class IpnbEditableCell implements IpnbCell {
     IpnbEditableCell cell = (IpnbEditableCell)o;
 
     if (!mySource.equals(cell.mySource)) return false;
-    if (myMetadata != null ? !myMetadata.equals(cell.myMetadata) : cell.myMetadata != null) return false;
+    if (!myMetadata.equals(cell.myMetadata)) return false;
 
     return true;
   }
@@ -50,7 +50,7 @@ public abstract class IpnbEditableCell implements IpnbCell {
   @Override
   public int hashCode() {
     int result = mySource.hashCode();
-    result = 31 * result + (myMetadata != null ? myMetadata.hashCode() : 0);
+    result = 31 * result + myMetadata.hashCode();
     return result;
   }
 }
index de692ca18f151b4d39fb007af32b5b134eaa8f13..5bec9a7fe5bb56f6b342b43afc39a2ad54e1e3b0 100644 (file)
@@ -1,7 +1,6 @@
 package org.jetbrains.plugins.ipnb.format.cells;
 
 import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 
 import java.util.List;
 import java.util.Map;
@@ -9,7 +8,7 @@ import java.util.Map;
 public class IpnbHeadingCell extends IpnbEditableCell {
   private int myLevel;   // from 1 to 6
 
-  public IpnbHeadingCell(@NotNull final List<String> source, int level, @Nullable Map<String, Object> metadata) {
+  public IpnbHeadingCell(@NotNull final List<String> source, int level, @NotNull Map<String, Object> metadata) {
     super(source, metadata);
     myLevel = level;
   }
index bcd55ed3884cecc3294aa709e7343cfbbd0845b3..a5e3ec893ee6a8967ba9fdb7658817ffcce8ecbb 100644 (file)
@@ -1,13 +1,12 @@
 package org.jetbrains.plugins.ipnb.format.cells;
 
 import org.jetbrains.annotations.NotNull;
-import org.jetbrains.annotations.Nullable;
 
 import java.util.List;
 import java.util.Map;
 
 public class IpnbMarkdownCell extends IpnbEditableCell {
-  public IpnbMarkdownCell(@NotNull final List<String> source, @Nullable Map<String, Object> metadata) {
+  public IpnbMarkdownCell(@NotNull final List<String> source, @NotNull Map<String, Object> metadata) {
     super(source, metadata);
   }
 
index 5b9986fb37b8ec99a2422b7df3280b5c684fca5f..e48272371976f9226e4c8b579172cbb9b5035f0c 100644 (file)
@@ -5,7 +5,7 @@ import org.jetbrains.annotations.NotNull;
 import java.util.List;
 
 public class IpnbRawCell implements IpnbCell {
-  @NotNull private List<String> mySource;
+  @NotNull private final List<String> mySource;
 
   public IpnbRawCell(@NotNull final List<String> source) {
     mySource = source;
index 46842478806806a30da65f0d31953c9bf33c263d..7e503a74214215c64abadc3be325bfc260512e3b 100644 (file)
@@ -45,7 +45,7 @@ public class IpnbConnection {
   private volatile boolean myIsIOPubOpen = false;
   protected volatile boolean myIsOpened = false;
 
-  private ArrayList<IpnbOutputCell> myOutput = new ArrayList<>();
+  private final ArrayList<IpnbOutputCell> myOutput = new ArrayList<>();
   private int myExecCount;
 
 
index 35a8386b038b8deff5e5e6f9f44ce1635bf7c17f..cbf8cc56f2976cacdce2e33ae4bf062e52f912cf 100644 (file)
@@ -4,7 +4,7 @@ import com.jetbrains.python.PythonFileType;
 import org.jetbrains.annotations.NotNull;
 
 public class IpnbPyFileType extends PythonFileType {
-  public static PythonFileType INSTANCE = new IpnbPyFileType();
+  public static final PythonFileType INSTANCE = new IpnbPyFileType();
 
   protected IpnbPyFileType() {
     super(new IpnbPyLanguageDialect());