Changeset 8514 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2015-06-21T03:19:37+02:00 (11 years ago)

Author:

Don-vip

Message:

checkstyle: various checks

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• dialogs/ChangesetDialog.java (modified) (1 diff)
• mappaint/mapcss/Condition.java (modified) (2 diffs)
• mappaint/mapcss/ExpressionFactory.java (modified) (1 diff)
• mappaint/mapcss/Selector.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/dialogs/ChangesetDialog.java

@@ -479 +479 @@
      * A utility class to fetch changesets and display the changeset dialog.
      */
-    public static class LaunchChangesetManager {
+    public static final class LaunchChangesetManager {
 
         private LaunchChangesetManager() {

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Condition.java

@@ -486 +486 @@
         public final boolean not;
 
-        private PseudoClassCondition(Method method, boolean not) {
+        protected PseudoClassCondition(Method method, boolean not) {
             this.method = method;
             this.not = not;
@@ -501 +501 @@
             }
             throw new IllegalArgumentException("Invalid pseudo class specified: " + id);
-
         }
 

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java

@@ -108 +108 @@
      */
     @SuppressWarnings("UnusedDeclaration")
-    public static class Functions {
+    public static final class Functions {
 
         private Functions() {

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Selector.java

@@ -266 +266 @@
 
         private class ContainsFinder extends AbstractFinder {
-            private ContainsFinder(Environment e) {
+            protected ContainsFinder(Environment e) {
                 super(e);
                 CheckParameterUtil.ensureThat(!(e.osm instanceof Node), "Nodes not supported");