Fix UnsupportedOperationException in GUI. #1718

checkstyle · Aug 17, 2015 · f0c7433 · f0c7433
1 parent 7e1d5d0
commit f0c7433
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/gui/ParseTreeInfoPanel.java b/src/main/java/com/puppycrawl/tools/checkstyle/gui/ParseTreeInfoPanel.java
@@ -27,7 +27,6 @@
 import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import java.util.TooManyListenersException;
 
@@ -112,9 +111,9 @@ public void openAst(DetailAST parseTree, final Component parent) {
         reloadAction.setEnabled(true);
 
         // clear for each new file
-        getLinesToPosition().clear();
+        clearLinesToPosition();
         // starts line counting at 1
-        getLinesToPosition().add(0);
+        addLineToPosition(0);
         // insert the contents of the file to the text area
 
         // clean the text area before inserting the lines of the new file
@@ -141,12 +140,12 @@ public void openFile(File file, final Component parent) {
                 final String[] sourceLines = text.toLinesArray();
 
                 // clear for each new file
-                getLinesToPosition().clear();
+                clearLinesToPosition();
                 // starts line counting at 1
-                getLinesToPosition().add(0);
+                addLineToPosition(0);
                 // insert the contents of the file to the text area
                 for (String element : sourceLines) {
-                    getLinesToPosition().add(textArea.getText().length());
+                    addLineToPosition(textArea.getText().length());
                     textArea.append(element + "\n");
                 }
 
@@ -199,8 +198,12 @@ private static void showErrorDialog(final Component parent, final String msg) {
         SwingUtilities.invokeLater(showError);
     }
 
-    public List<Integer> getLinesToPosition() {
-        return Collections.unmodifiableList(linesToPosition);
+    void addLineToPosition(int value) {
+        linesToPosition.add(value);
+    }
+
+    void clearLinesToPosition() {
+        linesToPosition.clear();
     }
 
     /**