Ticket #19281: Objects_hash_simplifications1.patch

File Objects_hash_simplifications1.patch, 23.6 KB (added by hiddewie, 6 years ago)

  • src/org/openstreetmap/josm/data/gpx/GpxData.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    999999
    10001000    @Override
    10011001    public synchronized int hashCode() {
    1002         final int prime = 31;
    1003         int result = prime + super.hashCode();
    1004         result = prime * result + ((namespaces == null) ? 0 : namespaces.hashCode());
    1005         result = prime * result + ((layerPrefs == null) ? 0 : layerPrefs.hashCode());
    1006         result = prime * result + ((dataSources == null) ? 0 : dataSources.hashCode());
    1007         result = prime * result + ((privateRoutes == null) ? 0 : privateRoutes.hashCode());
    1008         result = prime * result + ((privateTracks == null) ? 0 : privateTracks.hashCode());
    1009         result = prime * result + ((privateWaypoints == null) ? 0 : privateWaypoints.hashCode());
    1010         return result;
     1002        return Objects.hash(
     1003                super.hashCode(),
     1004                namespaces,
     1005                layerPrefs,
     1006                dataSources,
     1007                privateRoutes,
     1008                privateTracks,
     1009                privateWaypoints
     1010        );
    10111011    }
    10121012
    10131013    @Override

  • src/org/openstreetmap/josm/data/gpx/GpxTrackSegment.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    55import java.util.Collection;
    66import java.util.Collections;
    77import java.util.List;
     8import java.util.Objects;
    89
    910import org.openstreetmap.josm.data.Bounds;
    1011
    … …  
    7778
    7879    @Override
    7980    public int hashCode() {
    80         final int prime = 31;
    81         int result = prime + super.hashCode();
    82         result = prime * result + ((wayPoints == null) ? 0 : wayPoints.hashCode());
    83         return result;
     81        return Objects.hash(super.hashCode(), wayPoints);
    8482    }
    8583
    8684    @Override

  • src/org/openstreetmap/josm/data/gpx/WayPoint.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    287287
    288288    @Override
    289289    public int hashCode() {
    290         final int prime = 31;
    291         int result = super.hashCode();
    292         long temp = Double.doubleToLongBits(lat);
    293         result = prime * result + (int) (temp ^ (temp >>> 32));
    294         temp = Double.doubleToLongBits(lon);
    295         result = prime * result + (int) (temp ^ (temp >>> 32));
    296         temp = getTimeInMillis();
    297         result = prime * result + (int) (temp ^ (temp >>> 32));
    298         return result;
     290        long tempLat = Double.doubleToLongBits(lat);
     291        long tempLon = Double.doubleToLongBits(lon);
     292        long tempTime = getTimeInMillis();
     293        return Objects.hash(super.hashCode(), tempLat ^ (tempLat >>> 32), tempLon ^ (tempLon >>> 32), tempTime ^ (tempTime >>> 32));
    299294    }
    300295
    301296    @Override

  • src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    421421
    422422        @Override
    423423        public int hashCode() {
    424             final int prime = 31;
    425             int result = 1;
    426             result = prime * result + ((lhs == null) ? 0 : lhs.hashCode());
    427             result = prime * result + ((rhs == null) ? 0 : rhs.hashCode());
    428             return result;
     424            return Objects.hash(lhs, lhs);
    429425        }
    430426
    431427        @Override
    … …  
    529525
    530526        @Override
    531527        public int hashCode() {
    532             final int prime = 31;
    533             int result = 1;
    534             result = prime * result + (defaultValue ? 1231 : 1237);
    535             result = prime * result + ((key == null) ? 0 : key.hashCode());
    536             return result;
     528            return Objects.hash(defaultValue, key);
    537529        }
    538530
    539531        @Override
    … …  
    852844
    853845        @Override
    854846        public int hashCode() {
    855             final int prime = 31;
    856             int result = 1;
    857             result = prime * result + compareMode;
    858             result = prime * result + ((key == null) ? 0 : key.hashCode());
    859             result = prime * result + ((referenceNumber == null) ? 0 : referenceNumber.hashCode());
    860             result = prime * result + ((referenceValue == null) ? 0 : referenceValue.hashCode());
    861             return result;
     847            return Objects.hash(compareMode, key, referenceNumber, referenceValue);
    862848        }
    863849
    864850        @Override
    … …  
    1009995
    1010996        @Override
    1011997        public int hashCode() {
    1012             final int prime = 31;
    1013             int result = 1;
    1014             result = prime * result + ((key == null) ? 0 : key.hashCode());
    1015             result = prime * result + ((keyPattern == null) ? 0 : keyPattern.hashCode());
    1016             result = prime * result + ((mode == null) ? 0 : mode.hashCode());
    1017             result = prime * result + ((value == null) ? 0 : value.hashCode());
    1018             result = prime * result + ((valuePattern == null) ? 0 : valuePattern.hashCode());
    1019             return result;
     998            return Objects.hash(key, keyPattern, mode, value, valuePattern);
    1020999        }
    10211000
    10221001        @Override
    … …  
    11161095
    11171096        @Override
    11181097        public int hashCode() {
    1119             final int prime = 31;
    1120             int result = 1;
    1121             result = prime * result + (caseSensitive ? 1231 : 1237);
    1122             result = prime * result + ((search == null) ? 0 : search.hashCode());
    1123             result = prime * result + ((searchRegex == null) ? 0 : searchRegex.hashCode());
    1124             return result;
     1098            return Objects.hash(caseSensitive, search, searchRegex);
    11251099        }
    11261100
    11271101        @Override
    … …  
    13351309
    13361310        @Override
    13371311        public int hashCode() {
    1338             final int prime = 31;
    1339             int result = 1;
    1340             result = prime * result + (modulo ? 1231 : 1237);
    1341             result = prime * result + nth;
    1342             return result;
     1312            return Objects.hash(modulo, nth);
    13431313        }
    13441314
    13451315        @Override
    … …  
    13911361
    13921362        @Override
    13931363        public int hashCode() {
    1394             final int prime = 31;
    1395             int result = 1;
    1396             result = prime * result + (int) (max ^ (max >>> 32));
    1397             result = prime * result + (int) (min ^ (min >>> 32));
    1398             return result;
     1364            return Objects.hash(max ^ (max >>> 32), min ^ (min >>> 32));
    13991365        }
    14001366
    14011367        @Override

  • src/org/openstreetmap/josm/data/preferences/AbstractProperty.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    88import org.openstreetmap.josm.tools.ListenableWeakReference;
    99import org.openstreetmap.josm.tools.bugreport.BugReport;
    1010
     11import java.util.Objects;
     12
    1113/**
    1214 * Captures the common functionality of preference properties
    1315 * @param <T> The type of object accessed by this property
    … …  
    2830
    2931        @Override
    3032        public int hashCode() {
    31             final int prime = 31;
    32             int result = 1;
    33             result = prime * result + getOuterType().hashCode();
    34             result = prime * result + ((listener == null) ? 0 : listener.hashCode());
    35             return result;
     33            return Objects.hash(getOuterType(), listener);
    3634        }
    3735
    3836        @Override
    … …  
    317315
    318316    @Override
    319317    public int hashCode() {
    320         final int prime = 31;
    321         int result = 1;
    322         result = prime * result + ((key == null) ? 0 : key.hashCode());
    323         result = prime * result + ((preferences == null) ? 0 : preferences.hashCode());
    324         return result;
     318        return Objects.hash(key, preferences);
    325319    }
    326320
    327321    @Override

  • src/org/openstreetmap/josm/data/tagging/ac/AutoCompletionItem.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.data.tagging.ac;
    33
     4import java.util.Objects;
     5
    46/**
    57 * Represents an entry in the set of auto completion values.
    68 *
    … …  
    8587
    8688    @Override
    8789    public int hashCode() {
    88         final int prime = 31;
    89         int result = 1;
    90         result = prime * result
    91                 + ((priority == null) ? 0 : priority.hashCode());
    92         result = prime * result + ((value == null) ? 0 : value.hashCode());
    93         return result;
     90        return Objects.hash(priority, value);
    9491    }
    9592
    9693    @Override

  • src/org/openstreetmap/josm/gui/download/UserQueryList.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    624624
    625625        @Override
    626626        public int hashCode() {
    627             final int prime = 31;
    628             int result = 1;
    629             result = prime * result + ((itemKey == null) ? 0 : itemKey.hashCode());
    630             result = prime * result + ((query == null) ? 0 : query.hashCode());
    631             return result;
     627            return Objects.hash(itemKey, query);
    632628        }
    633629
    634630        @Override

  • src/org/openstreetmap/josm/gui/layer/geoimage/ImageEntry.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    7676
    7777    @Override
    7878    public int hashCode() {
    79         return 31 * super.hashCode() + ((thumbnail == null) ? 0 : thumbnail.hashCode());
     79        return Objects.hash(super.hashCode(), thumbnail);
    8080    }
    8181
    8282    @Override

  • src/org/openstreetmap/josm/gui/layer/imagery/TileSourceDisplaySettings.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    44import java.util.HashMap;
    55import java.util.Locale;
    66import java.util.Map;
     7import java.util.Objects;
    78import java.util.concurrent.CopyOnWriteArrayList;
    89
    910import org.openstreetmap.gui.jmapviewer.interfaces.TileSource;
    … …  
    322323
    323324    @Override
    324325    public int hashCode() {
    325         final int prime = 31;
    326         int result = 1;
    327         result = prime * result + (autoLoad ? 1231 : 1237);
    328         result = prime * result + (autoZoom ? 1231 : 1237);
    329         result = prime * result + (showErrors ? 1231 : 1237);
    330         return result;
     326        return Objects.hash(autoLoad, autoZoom, showErrors);
    331327    }
    332328
    333329    @Override

  • src/org/openstreetmap/josm/gui/mappaint/styleelement/placement/CompletelyInsideAreaStrategy.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    44import java.awt.Rectangle;
    55import java.awt.geom.Point2D;
    66import java.awt.geom.Rectangle2D;
     7import java.util.Objects;
    78
    89import org.openstreetmap.josm.gui.MapViewState;
    910import org.openstreetmap.josm.gui.draw.MapViewPath;
    … …  
    121122
    122123    @Override
    123124    public int hashCode() {
    124         final int prime = 31;
    125         int result = 1;
    126         long temp;
    127         temp = Double.doubleToLongBits(offsetX);
    128         result = prime * result + (int) (temp ^ (temp >>> 32));
    129         temp = Double.doubleToLongBits(offsetY);
    130         result = prime * result + (int) (temp ^ (temp >>> 32));
    131         return result;
     125        long tempX = Double.doubleToLongBits(offsetX);
     126        long tempY = Double.doubleToLongBits(offsetY);
     127        return Objects.hash(tempX ^ (tempX >>> 32), tempY ^ (tempY >>> 32));
    132128    }
    133129
    134130    @Override

  • src/org/openstreetmap/josm/gui/mappaint/styleelement/placement/OnLineStrategy.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    1010import java.util.Comparator;
    1111import java.util.Iterator;
    1212import java.util.List;
     13import java.util.Objects;
    1314import java.util.Optional;
    1415import java.util.stream.IntStream;
    1516
    … …  
    353354
    354355    @Override
    355356    public int hashCode() {
    356         final int prime = 31;
    357         int result = 1;
    358         long temp;
    359         temp = Double.doubleToLongBits(yOffset);
    360         result = prime * result + (int) (temp ^ (temp >>> 32));
    361         return result;
     357        long temp = Double.doubleToLongBits(yOffset);
     358        return Objects.hash(temp ^ (temp >>> 32));
    362359    }
    363360
    364361    @Override

  • src/org/openstreetmap/josm/gui/mappaint/styleelement/AreaIconElement.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    7171
    7272    @Override
    7373    public int hashCode() {
    74         final int prime = 31;
    75         int result = super.hashCode();
    76         result = prime * result + ((iconImage == null) ? 0 : iconImage.hashCode());
    77         result = prime * result + ((iconImageAngle == null) ? 0 : iconImageAngle.hashCode());
    78         result = prime * result + ((iconPosition == null) ? 0 : iconPosition.hashCode());
    79         return result;
     74        return Objects.hash(super.hashCode(), iconImage, iconImageAngle, iconPosition);
    8075    }
    8176
    8277    @Override

  • src/org/openstreetmap/josm/gui/util/WindowGeometry.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    1313import java.awt.Point;
    1414import java.awt.Rectangle;
    1515import java.awt.Window;
     16import java.util.Objects;
    1617import java.util.regex.Matcher;
    1718import java.util.regex.Pattern;
    1819
    … …  
    490491
    491492    @Override
    492493    public int hashCode() {
    493         final int prime = 31;
    494         int result = 1;
    495         result = prime * result + ((extent == null) ? 0 : extent.hashCode());
    496         result = prime * result + ((topLeft == null) ? 0 : topLeft.hashCode());
    497         return result;
     494        return Objects.hash(extent, topLeft);
    498495    }
    499496
    500497    @Override

  • src/org/openstreetmap/josm/plugins/PluginHandler.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    3232import java.util.Locale;
    3333import java.util.Map;
    3434import java.util.Map.Entry;
     35import java.util.Objects;
    3536import java.util.Set;
    3637import java.util.TreeSet;
    3738import java.util.concurrent.ExecutionException;
    … …  
    221222
    222223        @Override
    223224        public int hashCode() {
    224             final int prime = 31;
    225             int result = prime + ((name == null) ? 0 : name.hashCode());
    226             return prime * result + ((reason == null) ? 0 : reason.hashCode());
     225            return Objects.hash(name, reason);
    227226        }
    228227
    229228        @Override

  • src/org/openstreetmap/josm/tools/template_engine/ContextSwitchTemplate.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    77import java.util.Collection;
    88import java.util.Collections;
    99import java.util.List;
     10import java.util.Objects;
    1011import java.util.stream.Collectors;
    1112import java.util.stream.Stream;
    1213
    … …  
    5354
    5455        @Override
    5556        public int hashCode() {
    56             return 31 + ((condition == null) ? 0 : condition.hashCode());
     57            return Objects.hash(condition);
    5758        }
    5859
    5960        @Override
    … …  
    103104
    104105        @Override
    105106        public int hashCode() {
    106             return 31 * super.hashCode() + ((childCondition == null) ? 0 : childCondition.hashCode());
     107            return Objects.hash(super.hashCode(), childCondition);
    107108        }
    108109
    109110        @Override
    … …  
    166167
    167168        @Override
    168169        public int hashCode() {
    169             return 31 * super.hashCode() + ((parentCondition == null) ? 0 : parentCondition.hashCode());
     170            return Objects.hash(super.hashCode(), parentCondition);
    170171        }
    171172
    172173        @Override
    … …  
    209210
    210211        @Override
    211212        public int hashCode() {
    212             final int prime = 31;
    213             int result = super.hashCode();
    214             result = prime * result + ((lhs == null) ? 0 : lhs.hashCode());
    215             result = prime * result + ((rhs == null) ? 0 : rhs.hashCode());
    216             return result;
     213            return Objects.hash(super.hashCode(), lhs, rhs);
    217214        }
    218215
    219216        @Override
    … …  
    262259
    263260        @Override
    264261        public int hashCode() {
    265             final int prime = 31;
    266             int result = super.hashCode();
    267             result = prime * result + ((lhs == null) ? 0 : lhs.hashCode());
    268             result = prime * result + ((rhs == null) ? 0 : rhs.hashCode());
    269             return result;
     262            return Objects.hash(lhs, rhs);
    270263        }
    271264
    272265        @Override
    … …  
    395388
    396389    @Override
    397390    public int hashCode() {
    398         final int prime = 31;
    399         int result = 1;
    400         result = prime * result + ((context == null) ? 0 : context.hashCode());
    401         result = prime * result + ((template == null) ? 0 : template.hashCode());
    402         return result;
     391        return Objects.hash(context, template);
    403392    }
    404393
    405394    @Override

  • src/org/openstreetmap/josm/tools/template_engine/SearchExpressionCondition.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    33
    44import org.openstreetmap.josm.data.osm.search.SearchCompiler.Match;
    55
     6import java.util.Objects;
     7
    68/**
    79 * Conditional {@link TemplateEntry} that executes another template in case a search expression applies
    810 * to the given data provider.
    … …  
    3941
    4042    @Override
    4143    public int hashCode() {
    42         final int prime = 31;
    43         int result = 1;
    44         result = prime * result + ((condition == null) ? 0 : condition.hashCode());
    45         result = prime * result + ((text == null) ? 0 : text.hashCode());
    46         return result;
     44        return Objects.hash(condition, text);
    4745    }
    4846
    4947    @Override

  • src/org/openstreetmap/josm/tools/template_engine/StaticText.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.tools.template_engine;
    33
     4import java.util.Objects;
     5
    46/**
    57 * {@link TemplateEntry} representing a static string.
    68 * <p>
    … …  
    3537
    3638    @Override
    3739    public int hashCode() {
    38         return 31 + ((staticText == null) ? 0 : staticText.hashCode());
     40        return Objects.hash(staticText);
    3941    }
    4042
    4143    @Override

  • src/org/openstreetmap/josm/tools/template_engine/Variable.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    33
    44import java.util.Collection;
    55import java.util.Locale;
     6import java.util.Objects;
    67
    78/**
    89 * {@link TemplateEntry} that inserts the value of a variable.
    … …  
    8283
    8384    @Override
    8485    public int hashCode() {
    85         final int prime = 31;
    86         int result = 1;
    87         result = prime * result + (special ? 1231 : 1237);
    88         result = prime * result + ((variableName == null) ? 0 : variableName.hashCode());
    89         return result;
     86        return Objects.hash(special, variableName);
    9087    }
    9188
    9289    @Override

  • src/org/openstreetmap/josm/tools/RotationAngle.java 

    IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
     
    33
    44import java.util.List;
    55import java.util.Locale;
     6import java.util.Objects;
    67
    78import org.openstreetmap.josm.data.osm.IPrimitive;
    89import org.openstreetmap.josm.data.osm.Node;
    … …  
    8182
    8283        @Override
    8384        public int hashCode() {
    84             final int prime = 31;
    85             int result = 1;
    8685            long temp = Double.doubleToLongBits(angle);
    87             result = prime * result + (int) (temp ^ (temp >>> 32));
    88             return result;
     86            return Objects.hash(temp ^ (temp >>> 32));
    8987        }
    9088
    9189        @Override