Changeset 18801 in josm for trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

Timestamp:

2023-08-09T15:30:01+02:00 (9 months 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/preferences/shortcut/PrefJPanel.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

@@ -97 +97 @@
                     int i = field.getInt(null);
                     String s = KeyEvent.getKeyText(i);
-                    if (s != null && s.length() > 0 && !s.contains(unknown)) {
+                    if (s != null && !s.isEmpty() && !s.contains(unknown)) {
                         list.put(Integer.valueOf(i), s);
                     }