Changeset 18801 in josm for trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowser.java

Timestamp:

2023-08-09T15:30:01+02:00 (3 years ago)

Author:

taylor.smock

Message:

Fix #22832: Code cleanup and some simplification, documentation fixes (patch by gaben)

There should not be any functional changes in this patch; it is intended to do
the following:

• Simplify and cleanup code (example: Arrays.asList(item) -> Collections.singletonList(item))
• Fix typos in documentation (which also corrects the documentation to match what actually happens, in some cases)

File:

• trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowser.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowser.java

@@ -6 +6 @@
 import java.awt.BorderLayout;
 import java.awt.Dimension;
-import java.util.Arrays;
+import java.util.stream.Stream;
 
 import javax.swing.JPanel;
@@ -19 +19 @@
 
 /**
- * HistoryBrowser is an UI component which displays history information about an {@link OsmPrimitive}.
+ * HistoryBrowser is a UI component which displays history information about an {@link OsmPrimitive}.
  *
  * @since 1709
@@ -162 +162 @@
             model = null;
         }
-        Arrays.asList(tagInfoViewer, nodeListViewer, relationMemberListViewer, coordinateInfoViewer).stream()
+        Stream.of(tagInfoViewer, nodeListViewer, relationMemberListViewer, coordinateInfoViewer)
                 .filter(Destroyable.class::isInstance).forEach(Destroyable::destroy);
         tagInfoViewer = null;