JAL-3141 Made some more suggested changes from code review

diff --git a/src/jalview/io/BackupFiles.java b/src/jalview/io/BackupFiles.java
index 28eefd6..c58a714 100644 (file)
--- a/src/jalview/io/BackupFiles.java
+++ b/src/jalview/io/BackupFiles.java
@@ -8,6 +8,7 @@ import jalview.util.MessageManager;
 import java.io.File;
 import java.io.IOException;
 import java.util.Arrays;
+import java.util.Map;
 import java.util.HashMap;
 import java.util.TreeMap;
 
@@ -263,7 +264,7 @@ public class BackupFiles
 
       // sort the backup files (based on integer found in the suffix) using a
       // precomputed Hashmap for speed
-      HashMap<Integer, File> bfHashMap = new HashMap<>();
+      Map<Integer, File> bfHashMap = new HashMap<>();
       for (int i = 0; i < backupFiles.length; i++)
       {
         File f = backupFiles[i];
@@ -465,7 +466,7 @@ public class BackupFiles
     return rename;
   }
 
-  public static TreeMap<Integer, File> lsBackupFilesAsTreeMap(
+  public static TreeMap<Integer, File> getBackupFilesAsTreeMap(
           String fileName,
           String suffix, int digits)
   {
@@ -504,7 +505,7 @@ public class BackupFiles
     
     // sort the backup files (based on integer found in the suffix) using a
     // precomputed Hashmap for speed
-    HashMap<Integer, File> bfHashMap = new HashMap<>();
+    Map<Integer, File> bfHashMap = new HashMap<>();
     for (int i = 0; i < backupFiles.length; i++)
     {
       File f = backupFiles[i];
@@ -518,10 +519,10 @@ public class BackupFiles
     return bfTreeMap;
   }
 
-  public static File[] lsBackupFiles(String fileName, String suffix,
+  public static File[] getBackupFiles(String fileName, String suffix,
           int digits)
   {
-    TreeMap<Integer, File> bfTreeMap = lsBackupFilesAsTreeMap(fileName,
+    TreeMap<Integer, File> bfTreeMap = getBackupFilesAsTreeMap(fileName,
             suffix, digits);
     File[] backupFiles = new File[bfTreeMap.size()];
     bfTreeMap.values().toArray(backupFiles);