Changeset 16630 in josm


Ignore:
Timestamp:
2020-06-14T17:17:08+02:00 (8 weeks ago)
Author:
simon04
Message:

see #19334 - https://errorprone.info/bugpattern/UnnecessaryParentheses

Location:
trunk
Files:
34 edited

Legend:

Unmodified
Added
Removed
  • trunk/build.xml

    r16629 r16630  
    302302            <!-- Undocumented argument to ignore "Sun internal proprietary API" warning, see http://stackoverflow.com/a/13862308/2257172 -->
    303303            <compilerarg value="-XDignore.symbol.file"/>
    304             <compilerarg value="-Xplugin:ErrorProne -XepExcludedPaths:.*/parsergen/.* -Xep:ReferenceEquality:OFF -Xep:FutureReturnValueIgnored:OFF -Xep:ShortCircuitBoolean:OFF -Xep:StringSplitter:OFF -Xep:JdkObsolete:OFF -Xep:UnnecessaryParentheses:OFF -Xep:EqualsGetClass:OFF -Xep:UndefinedEquals:OFF -Xep:BadImport:OFF -Xep:AnnotateFormatMethod:OFF -Xep:MutablePublicArray:OFF"/>
     304            <compilerarg value="-Xplugin:ErrorProne -XepExcludedPaths:.*/parsergen/.* -Xep:ReferenceEquality:OFF -Xep:FutureReturnValueIgnored:OFF -Xep:ShortCircuitBoolean:OFF -Xep:StringSplitter:OFF -Xep:JdkObsolete:OFF -Xep:EqualsGetClass:OFF -Xep:UndefinedEquals:OFF -Xep:BadImport:OFF -Xep:AnnotateFormatMethod:OFF -Xep:MutablePublicArray:OFF"/>
    305305            <compilerarg line="-Xmaxwarns 1000"/>
    306306            <classpath>
  • trunk/src/org/openstreetmap/josm/actions/ReverseWayAction.java

    r16119 r16630  
    157157        Collection<Command> corrCmds = Collections.<Command>emptyList();
    158158        if (Config.getPref().getBoolean("tag-correction.reverse-way", true)) {
    159             corrCmds = (new ReverseWayTagCorrector()).execute(w, wnew);
     159            corrCmds = new ReverseWayTagCorrector().execute(w, wnew);
    160160        }
    161161        return new ReverseWayResult(wnew, corrCmds, new ChangeCommand(w, wnew));
  • trunk/src/org/openstreetmap/josm/actions/SaveAction.java

    r16509 r16630  
    101101        Layer activeLayer = getLayerManager().getActiveLayer();
    102102        boolean en = activeLayer != null
    103                 && activeLayer.isSavable() && (!(activeLayer.getAssociatedFile() != null
    104                 && activeLayer instanceof SaveToFile && !((SaveToFile) activeLayer).requiresSaveToFile()));
     103                && activeLayer.isSavable() && !(activeLayer.getAssociatedFile() != null
     104                && activeLayer instanceof SaveToFile && !((SaveToFile) activeLayer).requiresSaveToFile());
    105105        GuiHelper.runInEDT(() -> setEnabled(en));
    106106    }
  • trunk/src/org/openstreetmap/josm/actions/SimplifyWayAction.java

    r16566 r16630  
    173173                auto ? tr("Proceed without simplifying") : tr("Cancel"))
    174174                .setContent(p)
    175                 .configureContextsensitiveHelp(("Action/SimplifyWay"), true);
     175                .configureContextsensitiveHelp("Action/SimplifyWay", true);
    176176        if (auto) {
    177177            ed.setButtonIcons("simplify", "ok");
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadSessionTask.java

    r13927 r16630  
    4444    @Override
    4545    public Future<?> loadUrl(DownloadParams settings, String url, ProgressMonitor progressMonitor) {
    46         if (url != null && (url.matches(PATTERN_SESSION))) {
     46        if (url != null && url.matches(PATTERN_SESSION)) {
    4747            try {
    4848                URL u = new URL(url);
  • trunk/src/org/openstreetmap/josm/command/PurgeCommand.java

    r16436 r16630  
    375375                for (OsmPrimitive parent : osm.getReferrers()) {
    376376                    if (parent instanceof Relation
    377                             && !(toPurgeChecked.contains(parent))
     377                            && !toPurgeChecked.contains(parent)
    378378                            && hasOnlyIncompleteMembers((Relation) parent, toPurgeChecked, relSet)) {
    379379                        relSet.add((Relation) parent);
     
    386386            for (int i = 0; i < relLst.size(); ++i) { // foreach loop not applicable since list gets extended while looping over it
    387387                for (OsmPrimitive parent : relLst.get(i).getReferrers()) {
    388                     if (!(toPurgeChecked.contains(parent))
     388                    if (!toPurgeChecked.contains(parent)
    389389                            && hasOnlyIncompleteMembers((Relation) parent, toPurgeChecked, relLst)) {
    390390                        relLst.add((Relation) parent);
  • trunk/src/org/openstreetmap/josm/data/osm/Storage.java

    r16621 r16630  
    174174    public Storage(Hash<? super T, ? super T> ha, int capacity, boolean safeIterator) {
    175175        this.hash = ha;
    176         int cap = 1 << (int) (Math.ceil(Math.log(capacity/LOAD_FACTOR) / Math.log(2)));
     176        int cap = 1 << (int) Math.ceil(Math.log(capacity/LOAD_FACTOR) / Math.log(2));
    177177        @SuppressWarnings("unchecked")
    178178        T[] newData = (T[]) new Object[cap];
  • trunk/src/org/openstreetmap/josm/data/projection/Ellipsoid.java

    r13626 r16630  
    304304        final double n3n4 = n3 - n4;
    305305        final double n4n5 = n4 - n5;
    306         final double ap = a * (1.0 - n + (5.0 / 4.0) * (n2n3) + (81.0 / 64.0) * (n4n5));
    307         final double bp = (3.0 / 2.0) * a * (n1n2 + (7.0 / 8.0) * (n3n4) + (55.0 / 64.0) * n5);
    308         final double cp = (15.0 / 16.0) * a * (n2n3 + (3.0 / 4.0) * (n4n5));
     306        final double ap = a * (1.0 - n + (5.0 / 4.0) * n2n3 + (81.0 / 64.0) * n4n5);
     307        final double bp = (3.0 / 2.0) * a * (n1n2 + (7.0 / 8.0) * n3n4 + (55.0 / 64.0) * n5);
     308        final double cp = (15.0 / 16.0) * a * (n2n3 + (3.0 / 4.0) * n4n5);
    309309        final double dp = (35.0 / 48.0) * a * (n3n4 + (11.0 / 16.0) * n5);
    310         final double ep = (315.0 / 512.0) * a * (n4n5);
     310        final double ep = (315.0 / 512.0) * a * n4n5;
    311311        return ap * phi - bp * sin2Phi + cp * sin4Phi - dp * sin6Phi + ep * sin8Phi;
    312312    }
  • trunk/src/org/openstreetmap/josm/data/projection/datum/SevenParameterDatum.java

    r13627 r16630  
    5959    public LatLon fromWGS84(LatLon ll) {
    6060        double[] xyz = Ellipsoid.WGS84.latLon2Cart(ll);
    61         double x = (1-s)*(-dx + xyz[0] + ((-dz+xyz[2])*(-ry) - (-dy+xyz[1])*(-rz)));
    62         double y = (1-s)*(-dy + xyz[1] + ((-dx+xyz[0])*(-rz) - (-dz+xyz[2])*(-rx)));
    63         double z = (1-s)*(-dz + xyz[2] + ((-dy+xyz[1])*(-rx) - (-dx+xyz[0])*(-ry)));
     61        double x = (1-s)*(-dx + xyz[0] + ((-dz+xyz[2])* -ry - (-dy+xyz[1])* -rz));
     62        double y = (1-s)*(-dy + xyz[1] + ((-dx+xyz[0])* -rz - (-dz+xyz[2])* -rx));
     63        double z = (1-s)*(-dz + xyz[2] + ((-dy+xyz[1])* -rx - (-dx+xyz[0])* -ry));
    6464        return this.ellps.cart2LatLon(x, y, z);
    6565    }
  • trunk/src/org/openstreetmap/josm/data/projection/proj/AbstractProj.java

    r16627 r16630  
    126126              (en2 + sphi *
    127127              (en3 + sphi *
    128               (en4))));
     128              en4)));
    129129    }
    130130
  • trunk/src/org/openstreetmap/josm/data/projection/proj/CassiniSoldner.java

    r16627 r16630  
    7676            double cosphi = Math.cos(phi);
    7777
    78             double n = 1.0 / (Math.sqrt(1.0 - e2 * sinphi * sinphi));
     78            double n = 1.0 / Math.sqrt(1.0 - e2 * sinphi * sinphi);
    7979            double tn = Math.tan(phi);
    8080            double t = tn * tn;
  • trunk/src/org/openstreetmap/josm/data/projection/proj/Mercator.java

    r12013 r16630  
    8484            final double lat0 = Utils.toRadians(params.lat0);
    8585            final double sinPhi = Math.sin(lat0);
    86             scaleFactor *= (Math.cos(lat0) / (Math.sqrt(1 - e2 * sinPhi * sinPhi)));
     86            scaleFactor *= Math.cos(lat0) / Math.sqrt(1 - e2 * sinPhi * sinPhi);
    8787        }
    8888    }
  • trunk/src/org/openstreetmap/josm/data/validation/tests/ConnectivityRelations.java

    r16628 r16630  
    308308    private static boolean checkMemberTagsForImpliedConnectivity(Relation relation, Map<String, Integer> roleLanes) {
    309309        // The members have different lane counts
    310         if (roleLanes.containsKey(TO) && roleLanes.containsKey(FROM) && (!roleLanes.get(TO).equals(roleLanes.get(FROM)))) {
     310        if (roleLanes.containsKey(TO) && roleLanes.containsKey(FROM) && !roleLanes.get(TO).equals(roleLanes.get(FROM))) {
    311311            return false;
    312312        }
  • trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java

    r16553 r16630  
    233233        public boolean isPrimitiveUsable(OsmPrimitive p) {
    234234            return super.isPrimitiveUsable(p) && p.hasKey("boundary") && !p.hasTag("boundary", "protected_area")
    235                     && (!(p instanceof Relation) || (((Relation) p).isMultipolygon()));
     235                    && (!(p instanceof Relation) || p.isMultipolygon());
    236236        }
    237237
  • trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java

    r16445 r16630  
    630630        for (RelationMember rm : r.getMembers()) {
    631631            if (rm.isWay()) {
    632                 if (rm.hasRole() && !(rm.hasRole("inner", "outer")))
     632                if (rm.hasRole() && !rm.hasRole("inner", "outer"))
    633633                    hasUnexpectedWayRole = true;
    634                 if (!(rm.hasRole("inner", "outer")) || !rm.hasRole()) {
     634                if (!rm.hasRole("inner", "outer") || !rm.hasRole()) {
    635635                    tmpErrors.add(TestError.builder(this, Severity.ERROR, WRONG_MEMBER_ROLE)
    636636                            .message(tr("Role for multipolygon way member should be inner or outer"))
  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r16445 r16630  
    737737        if (!checkValues || value == null)
    738738            return;
    739         if ((containsUnwantedNonPrintingControlCharacter(value)) && !withErrors.contains(p, "ICV")) {
     739        if (containsUnwantedNonPrintingControlCharacter(value) && !withErrors.contains(p, "ICV")) {
    740740            errors.add(TestError.builder(this, Severity.WARNING, LOW_CHAR_VALUE)
    741741                    .message(tr("Tag value contains non-printing (usually invisible) character"), s, key)
     
    761761            withErrors.put(p, "LV");
    762762        }
    763         if ((value.trim().isEmpty()) && !withErrors.contains(p, "EV")) {
     763        if (value.trim().isEmpty() && !withErrors.contains(p, "EV")) {
    764764            errors.add(TestError.builder(this, Severity.WARNING, EMPTY_VALUES)
    765765                    .message(tr("Tags with empty values"), s, key)
     
    795795        if (!checkKeys || key == null)
    796796            return;
    797         if ((containsUnwantedNonPrintingControlCharacter(key)) && !withErrors.contains(p, "ICK")) {
     797        if (containsUnwantedNonPrintingControlCharacter(key) && !withErrors.contains(p, "ICK")) {
    798798            errors.add(TestError.builder(this, Severity.WARNING, LOW_CHAR_KEY)
    799799                    .message(tr("Tag key contains non-printing character"), s, key)
  • trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

    r16553 r16630  
    10421042                    }
    10431043                } else {
    1044                     if (distSq-minDistSq < (4)*(4)) {
     1044                    if (distSq-minDistSq < 16) {
    10451045                        nearestList.addAll(nlist);
    10461046                    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java

    r16553 r16630  
    856856     */
    857857    public boolean isDialogInDefaultView() {
    858         return isShowing && isDocked && (!isCollapsed);
     858        return isShowing && isDocked && !isCollapsed;
    859859    }
    860860
  • trunk/src/org/openstreetmap/josm/gui/dialogs/layer/MergeGpxLayerDialog.java

    r14341 r16630  
    133133
    134134            if ((row == 0 || newRow == 0)
    135                     && (!ConditionalOptionPaneUtil.showConfirmationDialog(
     135                    && !ConditionalOptionPaneUtil.showConfirmationDialog(
    136136                            "gpx_target_change",
    137137                            getParent(),
     
    142142                            JOptionPane.OK_CANCEL_OPTION,
    143143                            JOptionPane.INFORMATION_MESSAGE,
    144                             JOptionPane.OK_OPTION))) {
     144                            JOptionPane.OK_OPTION)) {
    145145                return;
    146146            }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationTreeModel.java

    r16438 r16630  
    6262        int count = 0;
    6363        for (RelationMember member : parent.getMembers()) {
    64             if (!(member.isRelation())) {
     64            if (!member.isRelation()) {
    6565                continue;
    6666            }
     
    8787        int idx = 0;
    8888        for (RelationMember member : parent.getMembers()) {
    89             if (!(member.isRelation())) {
     89            if (!member.isRelation()) {
    9090                continue;
    9191            }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationNodeMap.java

    r14030 r16630  
    199199        if (onewayReverseMap.ways.containsKey(way)) {
    200200            for (Node n : onewayReverseMap.ways.get(way)) {
    201                 if ((map.nodes.containsKey(n))
     201                if (map.nodes.containsKey(n)
    202202                        || (onewayMap.nodes.containsKey(n) && onewayMap.nodes.get(n).size() > 1))
    203203                    return n;
  • trunk/src/org/openstreetmap/josm/gui/layer/MainLayerManager.java

    r14802 r16630  
    292292    @Override
    293293    protected Collection<Layer> realRemoveSingleLayer(Layer layer) {
    294         if ((layer instanceof OsmDataLayer) && (((OsmDataLayer) layer).isUploadInProgress())) {
     294        if (layer instanceof OsmDataLayer && ((OsmDataLayer) layer).isUploadInProgress()) {
    295295            GuiHelper.runInEDT(() -> JOptionPane.showMessageDialog(MainApplication.getMainFrame(),
    296296                    tr("Trying to delete the layer with background upload. Please wait until the upload is finished.")));
  • trunk/src/org/openstreetmap/josm/gui/mappaint/StyleCache.java

    r13354 r16630  
    2121     * An empty style cache entry
    2222     */
    23     public static final StyleCache EMPTY_STYLECACHE = (new StyleCache()).intern();
     23    public static final StyleCache EMPTY_STYLECACHE = new StyleCache().intern();
    2424
    2525    private static final int PLAIN = 0;
  • trunk/src/org/openstreetmap/josm/gui/mappaint/styleelement/RepeatImageElement.java

    r13919 r16630  
    155155        return "RepeatImageStyle{" + super.toString() + "pattern=[" + pattern +
    156156                "], offset=" + offset + ", spacing=" + spacing +
    157                 ", phase=" + (-phase) + ", align=" + align + '}';
     157                ", phase=" + -phase + ", align=" + align + '}';
    158158    }
    159159}
  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AdvancedPreference.java

    r16321 r16630  
    146146        if (o2.isChanged() && !o1.isChanged())
    147147            return 1;
    148         if (!(o1.isDefault()) && o2.isDefault())
     148        if (!o1.isDefault() && o2.isDefault())
    149149            return -1;
    150         if (!(o2.isDefault()) && o1.isDefault())
     150        if (!o2.isDefault() && o1.isDefault())
    151151            return 1;
    152152        return o1.compareTo(o2);
     
    280280                JFileChooser.FILES_ONLY, "customsettings.lastDirectory");
    281281        if (fc != null) {
    282             File[] sel = fc.isMultiSelectionEnabled() ? fc.getSelectedFiles() : (new File[]{fc.getSelectedFile()});
     282            File[] sel = fc.isMultiSelectionEnabled() ? fc.getSelectedFiles() : new File[]{fc.getSelectedFile()};
    283283            if (sel.length == 1 && !sel[0].getName().contains("."))
    284284                sel[0] = new File(sel[0].getAbsolutePath()+".xml");
  • trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java

    r16398 r16630  
    488488        // iterate over the buttons, add change listener to any change event
    489489        for (Enumeration<AbstractButton> button = colorGroup.getElements(); button.hasMoreElements();) {
    490             (button.nextElement()).addChangeListener(e -> {
     490            button.nextElement().addChangeListener(e -> {
    491491                colorTypeVelocityTune.setEnabled(colorTypeVelocity.isSelected());
    492492                colorTypeHeatMapTune.setEnabled(colorTypeHeatMap.isSelected());
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java

    r16276 r16630  
    135135                try {
    136136                    // TODO there must be a better way to parse a number like "+3" than this.
    137                     final int buttonvalue = (NumberFormat.getIntegerInstance().parse(ai.replace("+", ""))).intValue();
     137                    final int buttonvalue = NumberFormat.getIntegerInstance().parse(ai.replace("+", "")).intValue();
    138138                    if (auto_increment_selected == buttonvalue) aibutton.setSelected(true);
    139139                    aibutton.addActionListener(e -> auto_increment_selected = buttonvalue);
  • trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java

    r16628 r16630  
    344344    private static Rectangle boundsWithYandHeight(Rectangle bounds, double y, double height) {
    345345        Rectangle r = new Rectangle();
    346         r.setBounds((int) (bounds.getX()), (int) y, (int) (bounds.getWidth()), (int) height);
     346        r.setBounds((int) bounds.getX(), (int) y, (int) bounds.getWidth(), (int) height);
    347347        return r;
    348348    }
     
    350350    private static Rectangle boundsWithXandWidth(Rectangle bounds, double x, double width) {
    351351        Rectangle r = new Rectangle();
    352         r.setBounds((int) x, (int) (bounds.getY()), (int) width, (int) (bounds.getHeight()));
     352        r.setBounds((int) x, (int) bounds.getY(), (int) width, (int) bounds.getHeight());
    353353        return r;
    354354    }
  • trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java

    r16553 r16630  
    131131        this.data = updateData();
    132132        saveToDisk();
    133         Config.getPref().putInt("cache." + ident, (int) (TimeUnit.MILLISECONDS.toSeconds(System.currentTimeMillis())));
     133        Config.getPref().putInt("cache." + ident, (int) TimeUnit.MILLISECONDS.toSeconds(System.currentTimeMillis()));
    134134        return data;
    135135    }
  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r16427 r16630  
    188188        String host = null;
    189189        try {
    190             host = (new URL(serverUrl)).getHost();
     190            host = new URL(serverUrl).getHost();
    191191        } catch (MalformedURLException e) {
    192192            Logging.warn(e);
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetParser.java

    r14231 r16630  
    205205                    throwException(tr("Missing mandatory attribute ''{0}''.", "version"));
    206206                }
    207                 if (!("0.6".equals(v))) {
     207                if (!"0.6".equals(v)) {
    208208                    throwException(tr("Unsupported version: {0}", v));
    209209                }
  • trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java

    r16553 r16630  
    485485                        if ("GET".equalsIgnoreCase(mode) && getMapUrl != null && !"".equals(getMapUrl)) {
    486486                            try {
    487                                 String query = (new URL(getMapUrl)).getQuery();
     487                                String query = new URL(getMapUrl).getQuery();
    488488                                if (query == null) {
    489489                                    this.getMapUrl = getMapUrl + "?";
  • trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java

    r16407 r16630  
    649649    public static <T> T getNestedException(Exception e, Class<T> nestedClass) {
    650650        Throwable t = e;
    651         while (t != null && !(nestedClass.isInstance(t))) {
     651        while (t != null && !nestedClass.isInstance(t)) {
    652652            t = t.getCause();
    653653        }
  • trunk/src/org/openstreetmap/josm/tools/TextTagParser.java

    r16472 r16630  
    3838        }
    3939        String text = k.substring(1, k.length()-1);
    40         return (new TextAnalyzer(text)).parseString("\r\t\n");
     40        return new TextAnalyzer(text).parseString("\r\t\n");
    4141    }
    4242
Note: See TracChangeset for help on using the changeset viewer.