Changeset 16824 in josm for trunk/src


Ignore:
Timestamp:
2020-08-02T20:30:02+02:00 (4 years ago)
Author:
simon04
Message:

Remove Collection.contains check for Collection.remove

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/ToggleAction.java

    r16553 r16824  
    101101     */
    102102    public final void removeButtonModel(ButtonModel model) {
    103         if (model != null && buttonModels.contains(model)) {
     103        if (model != null) {
    104104            buttonModels.remove(model);
    105105        }

  • trunk/src/org/openstreetmap/josm/actions/UploadAction.java

    r16509 r16824  
    129129    public static void unregisterUploadHook(UploadHook hook) {
    130130        if (hook == null) return;
    131         if (UPLOAD_HOOKS.contains(hook)) {
    132             UPLOAD_HOOKS.remove(hook);
    133         }
    134         if (LATE_UPLOAD_HOOKS.contains(hook)) {
    135             LATE_UPLOAD_HOOKS.remove(hook);
    136         }
     131        UPLOAD_HOOKS.remove(hook);
     132        LATE_UPLOAD_HOOKS.remove(hook);
    137133    }
    138134

  • trunk/src/org/openstreetmap/josm/data/osm/DefaultNameFormatter.java

    r16657 r16824  
    8181    public static void unregisterFormatHook(NameFormatterHook hook) {
    8282        if (hook == null) return;
    83         if (formatHooks.contains(hook)) {
    84             formatHooks.remove(hook);
    85         }
     83        formatHooks.remove(hook);
    8684    }
    8785

  • trunk/src/org/openstreetmap/josm/data/validation/tests/Addresses.java

    r16643 r16824  
    161161                            if (!ignoredAddresses.contains(simplifiedAddress)) {
    162162                                ignoredAddresses.add(simplifiedAddress);
    163                             } else if (knownAddresses.containsKey(simplifiedAddress)) {
     163                            } else {
    164164                                knownAddresses.remove(simplifiedAddress);
    165165                            }

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/AbstractListMergeModel.java

    r16601 r16824  
    223223    public void addPropertyChangeListener(PropertyChangeListener listener) {
    224224        synchronized (listeners) {
    225             if (listener != null && !listeners.contains(listener)) {
     225            if (listener != null) {
    226226                listeners.add(listener);
    227227            }
    … …  
    231231    public void removePropertyChangeListener(PropertyChangeListener listener) {
    232232        synchronized (listeners) {
    233             if (listener != null && listeners.contains(listener)) {
     233            if (listener != null) {
    234234                listeners.remove(listener);
    235235            }

  • trunk/src/org/openstreetmap/josm/gui/util/CellEditorSupport.java

    r8390 r16824  
    4141    public final void addCellEditorListener(CellEditorListener l) {
    4242        synchronized (this) {
    43             if (l != null && !listeners.contains(l)) {
     43            if (l != null) {
    4444                listeners.add(l);
    4545            }
    … …  
    5353    public final void removeCellEditorListener(CellEditorListener l) {
    5454        synchronized (this) {
    55             if (l != null && listeners.contains(l)) {
     55            if (l != null) {
    5656                listeners.remove(l);
    5757            }

  • trunk/src/org/openstreetmap/josm/plugins/ReadLocalPluginInformationTask.java

    r14384 r16824  
    184184        for (PluginHandler.DeprecatedPlugin p : PluginHandler.DEPRECATED_PLUGINS) {
    185185            if (canceled) return;
    186             if (availablePlugins.containsKey(p.name)) {
    187                 availablePlugins.remove(p.name);
    188             }
     186            availablePlugins.remove(p.name);
    189187        }
    190188    }
Note: See TracChangeset for help on using the changeset viewer.