Changeset 15926 in josm


Ignore:
Timestamp:
2020-02-24T22:04:33+01:00 (3 months ago)
Author:
Don-vip
Message:

fix recently introduced SonarQube issues

Location:
trunk/src/org/openstreetmap/josm
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/cache/JCSCacheManager.java

    r15841 r15926  
    7979                        // Pending upstream patch: https://issues.apache.org/jira/projects/JCS/issues/JCS-200
    8080                        // Pending upstream patch: https://issues.apache.org/jira/projects/JCS/issues/JCS-201
    81                         if (msg.equals("No configuration settings found.  Using hardcoded default values for all pools.")
     81                        if ("No configuration settings found.  Using hardcoded default values for all pools.".equals(msg)
    8282                                || (msg.startsWith("Region") && msg.endsWith("Resetting cache"))) { // "Region [TMS_BLOCK_v2] Resetting cache"
    8383                            Logging.debug(msg);
  • trunk/src/org/openstreetmap/josm/gui/autofilter/AutoFilterRule.java

    r15893 r15926  
    137137            });
    138138        }
    139         return PROP_AUTO_FILTER_DEFAULTS.get() ? defaultValueSupplier.apply(osm) : IntStream.empty();
     139        return Boolean.TRUE.equals(PROP_AUTO_FILTER_DEFAULTS.get()) ? defaultValueSupplier.apply(osm) : IntStream.empty();
    140140    }
    141141
  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java

    r15856 r15926  
    352352            return tab.selectSubTab(sub);
    353353        } catch (NoSuchElementException ignore) {
     354            Logging.trace(ignore);
    354355            return false;
    355356        }
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetSelector.java

    r15909 r15926  
    9595        PresetClassification(TaggingPreset preset) {
    9696            this.preset = preset;
    97             Set<String> groups = new HashSet<>();
    98             Set<String> names = new HashSet<>();
    99             Set<String> tags = new HashSet<>();
     97            Set<String> groupSet = new HashSet<>();
     98            Set<String> nameSet = new HashSet<>();
     99            Set<String> tagSet = new HashSet<>();
    100100            TaggingPreset group = preset.group;
    101101            while (group != null) {
    102                 addLocaleNames(groups, group);
     102                addLocaleNames(groupSet, group);
    103103                group = group.group;
    104104            }
    105             addLocaleNames(names, preset);
     105            addLocaleNames(nameSet, preset);
    106106            for (TaggingPresetItem item: preset.data) {
    107107                if (item instanceof KeyedItem) {
    108                     tags.add(((KeyedItem) item).key);
     108                    tagSet.add(((KeyedItem) item).key);
    109109                    if (item instanceof ComboMultiSelect) {
    110110                        final ComboMultiSelect cms = (ComboMultiSelect) item;
    111111                        if (Boolean.parseBoolean(cms.values_searchable)) {
    112                             tags.addAll(cms.getDisplayValues());
     112                            tagSet.addAll(cms.getDisplayValues());
    113113                        }
    114114                    }
    115115                    if (item instanceof Key && ((Key) item).value != null) {
    116                         tags.add(((Key) item).value);
     116                        tagSet.add(((Key) item).value);
    117117                    }
    118118                } else if (item instanceof Roles) {
    119119                    for (Role role : ((Roles) item).roles) {
    120                         tags.add(role.key);
    121                     }
    122                 }
    123             }
    124             this.groups = Utils.toUnmodifiableList(groups);
    125             this.names = Utils.toUnmodifiableList(names);
    126             this.tags = Utils.toUnmodifiableList(tags);
     120                        tagSet.add(role.key);
     121                    }
     122                }
     123            }
     124            this.groups = Utils.toUnmodifiableList(groupSet);
     125            this.names = Utils.toUnmodifiableList(nameSet);
     126            this.tags = Utils.toUnmodifiableList(tagSet);
    127127        }
    128128
  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r15896 r15926  
    15171517
    15181518    /**
     1519     * Returns an {@link ImageIcon} for the given OSM object, at the specified size.
     1520     * This is a slow operation.
    15191521     * @param primitive Object for which an icon shall be fetched. The icon is chosen based on tags.
    15201522     * @param iconSize Target size of icon. Icon is padded if required.
     
    15271529
    15281530    /**
     1531     * Returns an {@link ImageIcon} for the given OSM object, at the specified size.
     1532     * This is a slow operation.
    15291533     * @param primitive Object for which an icon shall be fetched. The icon is chosen based on tags.
    15301534     * @param iconSize Target size of icon. Icon is padded if required.
Note: See TracChangeset for help on using the changeset viewer.