Changeset 10593 in josm for trunk/src

Timestamp:

2016-07-23T00:26:24+02:00 (8 years ago)

Author:

Don-vip

Message:

see #11390 - sonar - squid:S2293 - improved use of diamond operator thanks to Java 8 (JEP 101: ​http://openjdk.java.net/jeps/101)

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/CustomConfigurator.java (modified) (1 diff)
• gui/conflict/tags/PasteTagsConflictResolverDialog.java (modified) (1 diff)
• gui/conflict/tags/RelationMemberConflictResolverModel.java (modified) (2 diffs)
• gui/conflict/tags/TagConflictResolverModel.java (modified) (1 diff)
• gui/dialogs/ValidatorDialog.java (modified) (2 diffs)
• gui/preferences/DefaultTabPreferenceSetting.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/CustomConfigurator.java

@@ -940 +940 @@
         }
         if (existing != null)
-            return new ArrayList<Collection<String>>(existing.getValue());
+            return new ArrayList<>(existing.getValue());
         else
-            return defaults.getValue() == null ? null : new ArrayList<Collection<String>>(defaults.getValue());
+            return defaults.getValue() == null ? null : new ArrayList<>(defaults.getValue());
     }
 

trunk/src/org/openstreetmap/josm/gui/conflict/tags/PasteTagsConflictResolverDialog.java

@@ -156 +156 @@
         mode = Mode.RESOLVING_ONE_TAGCOLLECTION_ONLY;
         tagsForAllPrimitives = tagsForAllPrimitives == null ? new TagCollection() : tagsForAllPrimitives;
-        sourceStatistics = sourceStatistics == null ? new HashMap<OsmPrimitiveType, Integer>() : sourceStatistics;
-        targetStatistics = targetStatistics == null ? new HashMap<OsmPrimitiveType, Integer>() : targetStatistics;
+        sourceStatistics = sourceStatistics == null ? new HashMap<>() : sourceStatistics;
+        targetStatistics = targetStatistics == null ? new HashMap<>() : targetStatistics;
 
         // init the resolver

trunk/src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverModel.java

@@ -162 +162 @@
         decisions.clear();
         relations = relations == null ? Collections.<Relation>emptyList() : relations;
-        memberPrimitives = memberPrimitives == null ? new LinkedList<OsmPrimitive>() : memberPrimitives;
+        memberPrimitives = memberPrimitives == null ? new LinkedList<>() : memberPrimitives;
         for (Relation r : relations) {
             for (OsmPrimitive p: memberPrimitives) {
@@ -180 +180 @@
      */
     public void populate(Collection<RelationToChildReference> references) {
-        references = references == null ? new LinkedList<RelationToChildReference>() : references;
+        references = references == null ? new LinkedList<>() : references;
         decisions.clear();
         this.relations = new HashSet<>(references.size());

trunk/src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolverModel.java

@@ -133 +133 @@
         this.tags = tags;
         displayedKeys = new ArrayList<>();
-        this.keysWithConflicts = keysWithConflicts == null ? new HashSet<String>() : keysWithConflicts;
+        this.keysWithConflicts = keysWithConflicts == null ? new HashSet<>() : keysWithConflicts;
         decisions = new HashMap<>();
         rebuild();

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

@@ -473 +473 @@
             boolean isDblClick = isDoubleClick(e);
 
-            Collection<OsmPrimitive> sel = isDblClick ? new HashSet<OsmPrimitive>(40) : null;
+            Collection<OsmPrimitive> sel = isDblClick ? new HashSet<>(40) : null;
 
             boolean hasFixes = setSelection(sel, isDblClick);
@@ -576 +576 @@
         FixTask(Collection<TestError> testErrors) {
             super(tr("Fixing errors ..."), false /* don't ignore exceptions */);
-            this.testErrors = testErrors == null ? new ArrayList<TestError>() : testErrors;
+            this.testErrors = testErrors == null ? new ArrayList<>() : testErrors;
         }
 

trunk/src/org/openstreetmap/josm/gui/preferences/DefaultTabPreferenceSetting.java

@@ -42 +42 @@
         this.title = title;
         this.tabpane = tabpane;
-        this.subSettingMap = tabpane != null ? new HashMap<SubPreferenceSetting, Component>() : null;
+        this.subSettingMap = tabpane != null ? new HashMap<>() : null;
     }