Changeset 11366 in josm for trunk/src/org/openstreetmap/josm/gui/io/RecentlyOpenedFilesMenu.java

Timestamp:

2016-12-08T00:54:43+01:00 (7 years ago)

Author:

Don-vip

Message:

findbugs - SIC_INNER_SHOULD_BE_STATIC_ANON

File:

• trunk/src/org/openstreetmap/josm/gui/io/RecentlyOpenedFilesMenu.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/io/RecentlyOpenedFilesMenu.java

@@ -60 +60 @@
 
         for (final String file : fileHistory) {
-            add(new AbstractAction() {
-                {
-                    putValue(NAME, file);
-                    putValue("help", ht("/Action/OpenRecent"));
-                    putValue("toolbar", Boolean.FALSE);
-                }
-                @Override
-                public void actionPerformed(ActionEvent e) {
-                    File f = new File(file);
-                    OpenFileTask task = new OpenFileTask(Collections.singletonList(f), null);
-                    task.setRecordHistory(true);
-                    Main.worker.submit(task);
-                }
-            });
+            add(new OpenRecentAction(file));
         }
         add(new JSeparator());
@@ -82 +69 @@
         clearItem.setEnabled(!fileHistory.isEmpty());
         add(clearItem);
+    }
+
+    static final class OpenRecentAction extends AbstractAction {
+        private final String file;
+
+        OpenRecentAction(String file) {
+            this.file = file;
+            putValue(NAME, file);
+            putValue("help", ht("/Action/OpenRecent"));
+            putValue("toolbar", Boolean.FALSE);
+        }
+
+        @Override
+        public void actionPerformed(ActionEvent e) {
+            OpenFileTask task = new OpenFileTask(Collections.singletonList(new File(file)), null);
+            task.setRecordHistory(true);
+            Main.worker.submit(task);
+        }
     }