Changeset 8839 in josm


Ignore:
Timestamp:
2015-10-09T01:06:30+02:00 (4 years ago)
Author:
Don-vip
Message:

sonar - squid:S2325 - "private" methods that don't access instance data should be "static"

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r8836 r8839  
    11101110     * Returns true if from1-to1 and from2-to2 vertors directions are opposite
    11111111     */
    1112     private boolean isOppositeDirection(EastNorth from1, EastNorth to1, EastNorth from2, EastNorth to2) {
     1112    private static boolean isOppositeDirection(EastNorth from1, EastNorth to1, EastNorth from2, EastNorth to2) {
    11131113        return (from1.getX()-to1.getX())*(from2.getX()-to2.getX())
    11141114              +(from1.getY()-to1.getY())*(from2.getY()-to2.getY()) < 0;
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r8836 r8839  
    14591459    }
    14601460
    1461     private Path2D.Double getPath(Way w) {
     1461    private static Path2D.Double getPath(Way w) {
    14621462        Path2D.Double path = new Path2D.Double();
    14631463        boolean initial = true;
  • trunk/src/org/openstreetmap/josm/io/GeoJSONWriter.java

    r8836 r8839  
    118118    }
    119119
    120     private JsonArrayBuilder getCoorArray(JsonArrayBuilder builder, EastNorth c) {
     120    private static JsonArrayBuilder getCoorArray(JsonArrayBuilder builder, EastNorth c) {
    121121        return builder.add(c.getX()).add(c.getY());
    122122    }
Note: See TracChangeset for help on using the changeset viewer.