Changeset 8285 in josm for trunk/src/org/openstreetmap/josm/gui/history

Timestamp:

2015-04-28T00:49:49+02:00 (9 years ago)

Author:

Don-vip

Message:

fix sonar squid:S2039 - Member variable visibility should be specified

Location:

trunk/src/org/openstreetmap/josm/gui/history

Files:

• HistoryBrowserDialogManager.java (modified) (2 diffs)
• NodeListViewer.java (modified) (1 diff)
• TwoColumnDiff.java (modified) (1 diff)

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

@@ -87 +87 @@
     }
 
-    final String WINDOW_GEOMETRY_PREF = getClass().getName() + ".geometry";
+    private final String WINDOW_GEOMETRY_PREF = getClass().getName() + ".geometry";
 
     protected void placeOnScreen(HistoryBrowserDialog dialog) {
@@ -217 +217 @@
     private final Predicate<PrimitiveId> unloadedHistoryPredicate = new Predicate<PrimitiveId>() {
 
-        HistoryDataSet hds = HistoryDataSet.getInstance();
+        private HistoryDataSet hds = HistoryDataSet.getInstance();
 
         @Override

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

@@ -90 +90 @@
         return new TableModelListener() {
             private Boolean reversed = null;
-            final String nonReversedText = tr("Nodes") + (table.getFont().canDisplay('\u25bc') ? " \u25bc" : " (1-n)");
-            final String reversedText = tr("Nodes") + (table.getFont().canDisplay('\u25b2') ? " \u25b2" : " (n-1)");
+            private final String nonReversedText = tr("Nodes") + (table.getFont().canDisplay('\u25bc') ? " \u25bc" : " (1-n)");
+            private final String reversedText = tr("Nodes") + (table.getFont().canDisplay('\u25b2') ? " \u25b2" : " (n-1)");
 
             @Override

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

@@ -55 +55 @@
     public List<Item> referenceDiff;
     public List<Item> currentDiff;
-    Object[] reference;
-    Object[] current;
+    private Object[] reference;
+    private Object[] current;
     boolean referenceReversed = false;