Changeset 6334 in josm


Ignore:
Timestamp:
2013-10-27T04:06:10+01:00 (8 years ago)
Author:
Don-vip
Message:

Sonar/Findbugs - Format string should use %n rather than \n

Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/InspectPrimitiveDialog.java

    r6267 r6334  
    231231                for (String key : o.keySet()) {
    232232                    s.append(INDENT).append(INDENT);
    233                     s.append(String.format("\"%s\"=\"%s\"\n", key, o.get(key)));
     233                    s.append(String.format("\"%s\"=\"%s\"%n", key, o.get(key)));
    234234                }
    235235            }
  • trunk/src/org/openstreetmap/josm/io/GpxWriter.java

    r6203 r6334  
    6666        out.println("<?xml version='1.0' encoding='UTF-8'?>");
    6767        out.println("<gpx version=\"1.1\" creator=\"JOSM GPX export\" xmlns=\"http://www.topografix.com/GPX/1/1\"\n" +
    68                 (hasExtensions ? String.format("    xmlns:josm=\"%s\"\n", JOSM_EXTENSIONS_NAMESPACE_URI) : "") +
     68                (hasExtensions ? String.format("    xmlns:josm=\"%s\"%n", JOSM_EXTENSIONS_NAMESPACE_URI) : "") +
    6969                "    xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" \n" +
    7070                "    xsi:schemaLocation=\"http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd\">");
  • trunk/test/unit/org/openstreetmap/josm/data/osm/FilterTest.java

    r5556 r6334  
    169169                    foundAtLeastOne = true;
    170170                    if (!osm.get(key).equals(filterCode(osm))) {
    171                         failedPrimitives.append(String.format("Object %s. Expected [%s] but was [%s]\n", osm.toString(), osm.get(key), filterCode(osm)));
     171                        failedPrimitives.append(String.format("Object %s. Expected [%s] but was [%s]%n", osm.toString(), osm.get(key), filterCode(osm)));
    172172                    }
    173173                }
     
    175175            assertTrue(foundAtLeastOne);
    176176            if (failedPrimitives.length() != 0)
    177                 throw new AssertionError(String.format("Run #%d\n%s", i, failedPrimitives.toString()));
     177                throw new AssertionError(String.format("Run #%d%n%s", i, failedPrimitives.toString()));
    178178        }
    179179    }
  • trunk/test/unit/org/openstreetmap/josm/data/projection/EllipsoidTest.java

    r6286 r6334  
    66import org.junit.Assert;
    77import org.junit.Test;
    8 
    98import org.openstreetmap.josm.data.coor.LatLon;
    109
  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionRefTest.java

    r5554 r6334  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.data.projection;
    3 
    4 import static org.openstreetmap.josm.tools.I18n.tr;
    53
    64import java.io.BufferedReader;
     
    108import java.util.Collection;
    119import java.util.HashMap;
    12 import java.util.LinkedHashMap;
    13 import java.util.List;
    1410import java.util.Map;
    1511import java.util.Map.Entry;
     
    6864            {
    6965                EastNorth en = p.latlon2eastNorth(new LatLon(lat, lon));
    70                 String error = String.format("%s (%s): Projecting latlon(%s,%s):\n" +
    71                         "        expected: eastnorth(%s,%s),\n" +
    72                         "        but got:  eastnorth(%s,%s)!\n",
     66                String error = String.format("%s (%s): Projecting latlon(%s,%s):%n" +
     67                        "        expected: eastnorth(%s,%s),%n" +
     68                        "        but got:  eastnorth(%s,%s)!%n",
    7369                        p.toString(), code, lat, lon, east, north, en.east(), en.north());
    7470                double EPSILON = 1e-3; // 1 mm accuracy
     
    7975            {
    8076                LatLon ll = p.eastNorth2latlon(new EastNorth(east, north));
    81                 String error = String.format("%s (%s): Inverse projecting eastnorth(%s,%s):\n" +
    82                         "        expected: latlon(%s,%s),\n" +
    83                         "        but got:  latlon(%s,%s)!\n",
     77                String error = String.format("%s (%s): Inverse projecting eastnorth(%s,%s):%n" +
     78                        "        expected: latlon(%s,%s),%n" +
     79                        "        but got:  latlon(%s,%s)!%n",
    8480                        p.toString(), code, east, north, lat, lon, ll.lat(), ll.lon());
    8581                double EPSILON = Math.toDegrees(1e-3 / 6378137); // 1 mm accuracy (or better)
  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionRegressionTest.java

    r5554 r6334  
    108108            EastNorth en = proj.latlon2eastNorth(new LatLon(lat, lon));
    109109            LatLon ll2 = proj.eastNorth2latlon(en);
    110             out.write(String.format("%s\n  ll  %s %s\n  en  %s %s\n  ll2 %s %s\n", proj.toCode(), lat, lon, en.east(), en.north(), ll2.lat(), ll2.lon()));
     110            out.write(String.format("%s%n  ll  %s %s%n  en  %s %s%n  ll2 %s %s%n", proj.toCode(), lat, lon, en.east(), en.north(), ll2.lat(), ll2.lon()));
    111111        }
    112112        out.close();
     
    178178            EastNorth en = proj.latlon2eastNorth(data.ll);
    179179            if (!en.equals(data.en)) {
    180                 String error = String.format("%s (%s): Projecting latlon(%s,%s):\n" +
    181                         "        expected: eastnorth(%s,%s),\n" +
    182                         "        but got:  eastnorth(%s,%s)!\n",
     180                String error = String.format("%s (%s): Projecting latlon(%s,%s):%n" +
     181                        "        expected: eastnorth(%s,%s),%n" +
     182                        "        but got:  eastnorth(%s,%s)!%n",
    183183                        proj.toString(), data.code, data.ll.lat(), data.ll.lon(), data.en.east(), data.en.north(), en.east(), en.north());
    184184                fail.append(error);
     
    186186            LatLon ll2 = proj.eastNorth2latlon(data.en);
    187187            if (!ll2.equals(data.ll2)) {
    188                 String error = String.format("%s (%s): Inverse projecting eastnorth(%s,%s):\n" +
    189                         "        expected: latlon(%s,%s),\n" +
    190                         "        but got:  latlon(%s,%s)!\n",
     188                String error = String.format("%s (%s): Inverse projecting eastnorth(%s,%s):%n" +
     189                        "        expected: latlon(%s,%s),%n" +
     190                        "        but got:  latlon(%s,%s)!%n",
    191191                        proj.toString(), data.code, data.en.east(), data.en.north(), data.ll2.lat(), data.ll2.lon(), ll2.lat(), ll2.lon());
    192192                fail.append(error);
  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionTest.java

    r5556 r6334  
    6565            Bounds b = p.getWorldBoundsLatLon();
    6666   
    67             text += String.format("*** %s %s\n", p.toString(), p.toCode());
     67            text += String.format("*** %s %s%n", p.toString(), p.toCode());
    6868            for (int num=0; num < 1000; ++num) {
    6969   
     
    8686                error = true;
    8787            }
    88             text += String.format("%s errorLat: %s errorLon: %s\n", mark, maxErrLat, maxErrLon);
     88            text += String.format("%s errorLat: %s errorLon: %s%n", mark, maxErrLat, maxErrLon);
    8989        }
    9090    }
  • trunk/test/unit/org/openstreetmap/josm/data/projection/SwissGridTest.java

    r6264 r6334  
    5252            EastNorth en2 = swiss.latlon2eastNorth(pd.ll);
    5353            if (Math.abs(pd.en.east() - en2.east()) > EPSILON || Math.abs(pd.en.north() - en2.north()) > EPSILON) {
    54                 errs.append(String.format("%s should be: %s but is: %s\n", pd.name, pd.en, en2));
     54                errs.append(String.format("%s should be: %s but is: %s%n", pd.name, pd.en, en2));
    5555            }
    5656        }
Note: See TracChangeset for help on using the changeset viewer.