Changeset 8376 in josm for trunk/src


Ignore:
Timestamp:
2015-05-17T03:24:57+02:00 (10 years ago)
Author:
Don-vip
Message:

code style - remove useless calls to toString()

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

Legend:

Unmodified
Added
Removed

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

    r8357 r8376  
    9999                List<String> arguments = ManagementFactory.getRuntimeMXBean().getInputArguments();
    100100                if (Main.isDebugEnabled()) {
    101                     Main.debug("VM arguments: "+arguments.toString());
     101                    Main.debug("VM arguments: "+arguments);
    102102                }
    103103                for (String arg : arguments) {

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

    r8356 r8376  
    179179            }
    180180            for (Entry<String, Setting<?>> entry : settings.entrySet()) {
    181                 text.append(entry.getKey()).append("=").append(entry.getValue().getValue().toString()).append("\n");
     181                text.append(entry.getKey()).append("=").append(entry.getValue().getValue()).append("\n");
    182182            }
    183183        } catch (Exception x) {

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r8291 r8376  
    123123        for (OsmPrimitive osm: toDelete) {
    124124            if (osm.isDeleted())
    125                 throw new IllegalArgumentException(osm.toString() + " is already deleted");
     125                throw new IllegalArgumentException(osm + " is already deleted");
    126126            clonedPrimitives.put(osm, osm.save());
    127127

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

    r7518 r8376  
    9595    @Override
    9696    public String toString() {
    97         return "WayPoint (" + (attr.containsKey(GPX_NAME) ? get(GPX_NAME) + ", " :"") + getCoor().toString() + ", " + attr + ")";
     97        return "WayPoint (" + (attr.containsKey(GPX_NAME) ? get(GPX_NAME) + ", " :"") + getCoor() + ", " + attr + ")";
    9898    }
    9999

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

    r7005 r8376  
    5151    @Override
    5252    public String toString() {
    53         return super.toString() + " WAY" + nodes.toString();
     53        return super.toString() + " WAY" + nodes;
    5454    }
    5555

  • trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java

    r8365 r8376  
    231231     */
    232232    public String getLocalName() {
    233         String key = "name:" + Locale.getDefault().toString();
     233        String key = "name:" + Locale.getDefault();
    234234        if (get(key) != null)
    235235            return get(key);

  • trunk/src/org/openstreetmap/josm/gui/GettingStarted.java

    r7434 r8376  
    178178            URL u = getClass().getResource(im);
    179179            if (u != null) {
    180                 m.appendReplacement(sb, Matcher.quoteReplacement("src=\"" + u.toString() + "\""));
     180                m.appendReplacement(sb, Matcher.quoteReplacement("src=\"" + u + "\""));
    181181            }
    182182        }

  • trunk/src/org/openstreetmap/josm/gui/PleaseWaitDialog.java

    r8308 r8376  
    3232    private final JProgressBar progressBar = new JProgressBar();
    3333
    34     public final JLabel currentAction = new JLabel("");
     34    private final JLabel currentAction = new JLabel("");
    3535    private final JLabel customText = new JLabel("");
    3636    public final transient BoundedRangeModel progress = progressBar.getModel();

  • trunk/src/org/openstreetmap/josm/gui/dialogs/MapPaintDialog.java

    r8323 r8376  
    643643            p.add(new JScrollPane(txtErrors), GBC.std().fill());
    644644            for (Throwable t : s.getErrors()) {
    645                 txtErrors.append(t.toString() + "\n");
     645                txtErrors.append(t + "\n");
    646646            }
    647647        }

  • trunk/src/org/openstreetmap/josm/gui/io/UploadLayerTask.java

    r8291 r8376  
    140140        } catch (Exception sxe) {
    141141            if (isCanceled()) {
    142                 Main.info("Ignoring exception caught because upload is canceled. Exception is: " + sxe.toString());
     142                Main.info("Ignoring exception caught because upload is canceled. Exception is: " + sxe);
    143143                return;
    144144            }

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r8373 r8376  
    804804
    805805                if (toDelete.getFile().delete()) {
    806                     Main.info("File "+toDelete.getFile().toString()+" deleted. ");
     806                    Main.info("File "+toDelete.getFile()+" deleted. ");
    807807                } else {
    808808                    JOptionPane.showMessageDialog(

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ThumbsLoader.java

    r8285 r8376  
    5959
    6060    private BufferedImage loadThumb(ImageEntry entry) {
    61         final String cacheIdent = entry.getFile().toString()+":"+maxSize;
     61        final String cacheIdent = entry.getFile()+":"+maxSize;
    6262
    6363        if (!cacheOff) {

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/WebMarker.java

    r8056 r8376  
    2424 *
    2525 * @author Frederik Ramm
    26  *
     26 * @since 200
    2727 */
    2828public class WebMarker extends ButtonMarker {
    … …  
    3636    }
    3737
    38     @Override public void actionPerformed(ActionEvent ev) {
     38    @Override
     39    public void actionPerformed(ActionEvent ev) {
    3940        String error = OpenBrowser.displayUrl(webUrl.toString());
    4041        if (error != null) {
    … …  
    4243            new Notification(
    4344                    "<b>" + tr("There was an error while trying to display the URL for this marker") + "</b><br>" +
    44                                   tr("(URL was: ") + webUrl.toString() + ")" + "<br>" + error)
     45                                  tr("(URL was: ") + webUrl + ")" + "<br>" + error)
    4546                    .setIcon(JOptionPane.ERROR_MESSAGE)
    4647                    .setDuration(Notification.TIME_LONG)

  • trunk/src/org/openstreetmap/josm/gui/mappaint/Cascade.java

    r8338 r8376  
    213213                res.append(Utils.toString((Color)val));
    214214            } else if (val != null) {
    215                 res.append(val.toString());
     215                res.append(val);
    216216            }
    217217            res.append("; ");

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/LiteralExpression.java

    r8285 r8376  
    2828            return Arrays.toString((float[]) literal);
    2929        }
    30         return "<" + literal.toString() + ">";
     30        return "<" + literal + ">";
    3131    }
    3232}

  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/ListListEditor.java

    r8308 r8376  
    118118        @Override
    119119        public String getElementAt(int index) {
    120             return (index+1) + ": " + data.get(index).toString();
     120            return (index+1) + ": " + data.get(index);
    121121        }
    122122

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetSelector.java

    r8338 r8376  
    206206        @Override
    207207        public String toString() {
    208             return classification + " " + preset.toString();
     208            return classification + " " + preset;
    209209        }
    210210    }

  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionListItem.java

    r8291 r8376  
    8585    }
    8686
    87     @Override public String toString() {
     87    @Override
     88    public String toString() {
    8889        StringBuilder sb = new StringBuilder();
    8990        sb.append("<val='");
    9091        sb.append(value);
    9192        sb.append("',");
    92         sb.append(priority.toString());
     93        sb.append(priority);
    9394        sb.append(">");
    9495        return sb.toString();
    9596    }
    9697
    97     @Override public int hashCode() {
     98    @Override
     99    public int hashCode() {
    98100        final int prime = 31;
    99101        int result = 1;
    … …  
    104106    }
    105107
    106     @Override public boolean equals(Object obj) {
     108    @Override
     109    public boolean equals(Object obj) {
    107110        if (this == obj)
    108111            return true;

  • trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitLayout.java

    r8345 r8376  
    11131113        @Override
    11141114        public String toString() {
    1115             return "MultiSplitLayout.Divider " + getBounds().toString();
     1115            return "MultiSplitLayout.Divider " + getBounds();
    11161116        }
    11171117    }

  • trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java

    r8338 r8376  
    139139        }
    140140
    141         Main.info("GET " + getCapabilitiesUrl.toString());
     141        Main.info("GET " + getCapabilitiesUrl);
    142142        URLConnection openConnection = Utils.openHttpConnection(getCapabilitiesUrl);
    143143        StringBuilder ba = new StringBuilder();

  • trunk/src/org/openstreetmap/josm/tools/MultikeyActionsHandler.java

    r8308 r8376  
    8585        @Override
    8686        public String toString() {
    87             return "MultikeyAction" + action.toString();
     87            return "MultikeyAction" + action;
    8888        }
    8989    }

  • trunk/src/org/openstreetmap/josm/tools/Utils.java

    r8347 r8376  
    246246            }
    247247            if (s != null) {
    248                 s.append(sep).append(a.toString());
     248                s.append(sep).append(a);
    249249            } else {
    250250                s = new StringBuilder(a.toString());

  • trunk/src/org/openstreetmap/josm/tools/template_engine/CompoundTemplateEntry.java

    r7937 r8376  
    3636    }
    3737
    38 
    3938    @Override
    4039    public String toString() {
    4140        StringBuilder result = new StringBuilder();
    4241        for (TemplateEntry te: entries) {
    43             result.append(te.toString());
     42            result.append(te);
    4443        }
    4544        return result.toString();
    4645    }
    47 
    48 
    4946}

  • trunk/src/org/openstreetmap/josm/tools/template_engine/Condition.java

    r7937 r8376  
    4545        for (TemplateEntry entry: entries) {
    4646            if (entry instanceof SearchExpressionCondition) {
    47                 sb.append(entry.toString());
     47                sb.append(entry);
    4848            } else {
    4949                sb.append("'");
    50                 sb.append(entry.toString());
     50                sb.append(entry);
    5151                sb.append("'");
    5252            }

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

    r7937 r8376  
    2626    @Override
    2727    public String toString() {
    28         return condition.toString() + " '" + text.toString() + "'";
     28        return condition.toString() + " '" + text + "'";
    2929    }
    30 
    3130}
Note: See TracChangeset for help on using the changeset viewer.