Changeset 2789 in josm for trunk


Ignore:
Timestamp:
2010-01-09T15:49:04+01:00 (15 years ago)
Author:
jttt
Message:

Fixed some FindBugs warnings

Location:
trunk
Files:
13 edited
1 moved

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/projection/LambertEST.java

    r2516 r2789  
    5858    }
    5959
    60     public  static double IterateAngle(double e, double t)
     60    public static double iterateAngle(double e, double t)
    6161    {
    6262        double a1 = 0.0;
    … …  
    8888        double theta = Math.atan((p.getX() - ef) / (rf - p.getY() + nf));
    8989        double y = (theta / n + lonf) ;
    90         double x = (IterateAngle(ee, T)) ;
     90        double x = iterateAngle(ee, T);
    9191        return new LatLon(Math.toDegrees(x),Math.toDegrees(y));
    9292    }

  • trunk/src/org/openstreetmap/josm/data/projection/Puwg.java

    r2676 r2789  
    4646        double scale = z.getPuwgScaleFactor();
    4747        double center = z.getPuwgCentralMeridian(); /* in radians */
    48         EastNorth a = MapLatLonToXY(Math.toRadians(p.lat()), Math.toRadians(p.lon()), center);
     48        EastNorth a = mapLatLonToXY(Math.toRadians(p.lat()), Math.toRadians(p.lon()), center);
    4949        return new EastNorth(a.east() * scale + easting, a.north() * scale + northing);
    5050    }
    … …  
    5757        double scale = z.getPuwgScaleFactor();
    5858        double center = z.getPuwgCentralMeridian(); /* in radians */
    59         return MapXYToLatLon((p.east() - easting)/scale, (p.north() - northing)/scale, center);
     59        return mapXYToLatLon((p.east() - easting)/scale, (p.north() - northing)/scale, center);
    6060    }
    6161

  • trunk/src/org/openstreetmap/josm/data/projection/UTM.java

    r2516 r2789  
    1212import javax.swing.JPanel;
    1313
    14 import org.openstreetmap.josm.Main;
    1514import org.openstreetmap.josm.data.Bounds;
    1615import org.openstreetmap.josm.data.coor.EastNorth;
    … …  
    2928    private int zone = DEFAULT_ZONE;
    3029
    31     final private double UTMScaleFactor = 0.9996;
     30    final private static double UTMScaleFactor = 0.9996;
    3231
    3332    /* Ellipsoid model constants (WGS84) - TODO Use Elliposid class here too */
    34     final private double sm_EccSquared = 6.69437999013e-03;
     33    //final private double sm_EccSquared = 6.69437999013e-03;
    3534
    3635    /*
    37     * ArcLengthOfMeridian
    38     *
    39     * Computes the ellipsoidal distance from the equator to a point at a
    40     * given latitude.
    41     *
    42     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
    43     * GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
    44     *
    45     * Inputs:
    46     *     phi - Latitude of the point, in radians.
    47     *
    48     * Globals:
    49     *     Ellipsoid.GRS80.a - Ellipsoid model major axis.
    50     *     Ellipsoid.GRS80.b - Ellipsoid model minor axis.
    51     *
    52     * Returns:
    53     *     The ellipsoidal distance of the point from the equator, in meters.
    54     *
    55     */
     36     * ArcLengthOfMeridian
     37     *
     38     * Computes the ellipsoidal distance from the equator to a point at a
     39     * given latitude.
     40     *
     41     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
     42     * GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
     43     *
     44     * Inputs:
     45     *     phi - Latitude of the point, in radians.
     46     *
     47     * Globals:
     48     *     Ellipsoid.GRS80.a - Ellipsoid model major axis.
     49     *     Ellipsoid.GRS80.b - Ellipsoid model minor axis.
     50     *
     51     * Returns:
     52     *     The ellipsoidal distance of the point from the equator, in meters.
     53     *
     54     */
    5655    private double ArcLengthOfMeridian(double phi)
    5756    {
    … …  
    6160        /* Precalculate alpha */
    6261        double alpha = ((Ellipsoid.GRS80.a + Ellipsoid.GRS80.b) / 2.0)
    63            * (1.0 + (Math.pow (n, 2.0) / 4.0) + (Math.pow (n, 4.0) / 64.0));
     62        * (1.0 + (Math.pow (n, 2.0) / 4.0) + (Math.pow (n, 4.0) / 64.0));
    6463
    6564        /* Precalculate beta */
    6665        double beta = (-3.0 * n / 2.0) + (9.0 * Math.pow (n, 3.0) / 16.0)
    67            + (-3.0 * Math.pow (n, 5.0) / 32.0);
     66        + (-3.0 * Math.pow (n, 5.0) / 32.0);
    6867
    6968        /* Precalculate gamma */
    7069        double gamma = (15.0 * Math.pow (n, 2.0) / 16.0)
    71             + (-15.0 * Math.pow (n, 4.0) / 32.0);
     70        + (-15.0 * Math.pow (n, 4.0) / 32.0);
    7271
    7372        /* Precalculate delta */
    7473        double delta = (-35.0 * Math.pow (n, 3.0) / 48.0)
    75             + (105.0 * Math.pow (n, 5.0) / 256.0);
     74        + (105.0 * Math.pow (n, 5.0) / 256.0);
    7675
    7776        /* Precalculate epsilon */
    … …  
    8180        return alpha
    8281        * (phi + (beta * Math.sin (2.0 * phi))
    83             + (gamma * Math.sin (4.0 * phi))
    84             + (delta * Math.sin (6.0 * phi))
    85             + (epsilon * Math.sin (8.0 * phi)));
     82                + (gamma * Math.sin (4.0 * phi))
     83                + (delta * Math.sin (6.0 * phi))
     84                + (epsilon * Math.sin (8.0 * phi)));
    8685    }
    8786
    8887    /*
    89     * UTMCentralMeridian
    90     *
    91     * Determines the central meridian for the given UTM zone.
    92     *
    93     * Inputs:
    94     *     zone - An integer value designating the UTM zone, range [1,60].
    95     *
    96     * Returns:
    97     *   The central meridian for the given UTM zone, in radians, or zero
    98     *   if the UTM zone parameter is outside the range [1,60].
    99     *   Range of the central meridian is the radian equivalent of [-177,+177].
    100     *
    101     */
     88     * UTMCentralMeridian
     89     *
     90     * Determines the central meridian for the given UTM zone.
     91     *
     92     * Inputs:
     93     *     zone - An integer value designating the UTM zone, range [1,60].
     94     *
     95     * Returns:
     96     *   The central meridian for the given UTM zone, in radians, or zero
     97     *   if the UTM zone parameter is outside the range [1,60].
     98     *   Range of the central meridian is the radian equivalent of [-177,+177].
     99     *
     100     */
    102101    private double UTMCentralMeridian(int zone)
    103102    {
    … …  
    110109
    111110    /*
    112     * FootpointLatitude
    113     *
    114     * Computes the footpoint latitude for use in converting transverse
    115     * Mercator coordinates to ellipsoidal coordinates.
    116     *
    117     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
    118     *   GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
    119     *
    120     * Inputs:
    121     *   y - The UTM northing coordinate, in meters.
    122     *
    123     * Returns:
    124     *   The footpoint latitude, in radians.
    125     *
    126     */
     111     * FootpointLatitude
     112     *
     113     * Computes the footpoint latitude for use in converting transverse
     114     * Mercator coordinates to ellipsoidal coordinates.
     115     *
     116     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
     117     *   GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
     118     *
     119     * Inputs:
     120     *   y - The UTM northing coordinate, in meters.
     121     *
     122     * Returns:
     123     *   The footpoint latitude, in radians.
     124     *
     125     */
    127126    private double FootpointLatitude(double y)
    128127    {
    … …  
    133132        /* (Same as alpha in Eq. 10.17) */
    134133        double alpha_ = ((Ellipsoid.GRS80.a + Ellipsoid.GRS80.b) / 2.0)
    135             * (1 + (Math.pow (n, 2.0) / 4) + (Math.pow (n, 4.0) / 64));
     134        * (1 + (Math.pow (n, 2.0) / 4) + (Math.pow (n, 4.0) / 64));
    136135
    137136        /* Precalculate y_ (Eq. 10.23) */
    … …  
    140139        /* Precalculate beta_ (Eq. 10.22) */
    141140        double beta_ = (3.0 * n / 2.0) + (-27.0 * Math.pow (n, 3.0) / 32.0)
    142             + (269.0 * Math.pow (n, 5.0) / 512.0);
     141        + (269.0 * Math.pow (n, 5.0) / 512.0);
    143142
    144143        /* Precalculate gamma_ (Eq. 10.22) */
    145144        double gamma_ = (21.0 * Math.pow (n, 2.0) / 16.0)
    146             + (-55.0 * Math.pow (n, 4.0) / 32.0);
     145        + (-55.0 * Math.pow (n, 4.0) / 32.0);
    147146
    148147        /* Precalculate delta_ (Eq. 10.22) */
    149148        double delta_ = (151.0 * Math.pow (n, 3.0) / 96.0)
    150             + (-417.0 * Math.pow (n, 5.0) / 128.0);
     149        + (-417.0 * Math.pow (n, 5.0) / 128.0);
    151150
    152151        /* Precalculate epsilon_ (Eq. 10.22) */
    … …  
    155154        /* Now calculate the sum of the series (Eq. 10.21) */
    156155        return y_ + (beta_ * Math.sin (2.0 * y_))
    157             + (gamma_ * Math.sin (4.0 * y_))
    158             + (delta_ * Math.sin (6.0 * y_))
    159             + (epsilon_ * Math.sin (8.0 * y_));
     156        + (gamma_ * Math.sin (4.0 * y_))
     157        + (delta_ * Math.sin (6.0 * y_))
     158        + (epsilon_ * Math.sin (8.0 * y_));
    160159    }
    161160
    162161    /*
    163     * MapLatLonToXY
    164     *
    165     * Converts a latitude/longitude pair to x and y coordinates in the
    166     * Transverse Mercator projection.  Note that Transverse Mercator is not
    167     * the same as UTM; a scale factor is required to convert between them.
    168     *
    169     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
    170     * GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
    171     *
    172     * Inputs:
    173     *    phi - Latitude of the point, in radians.
    174     *    lambda - Longitude of the point, in radians.
    175     *    lambda0 - Longitude of the central meridian to be used, in radians.
    176     *
    177     * Outputs:
    178     *    xy - A 2-element array containing the x and y coordinates
    179     *         of the computed point.
    180     *
    181     * Returns:
    182     *    The function does not return a value.
    183     *
    184     */
    185     public EastNorth MapLatLonToXY(double phi, double lambda, double lambda0)
     162     * MapLatLonToXY
     163     *
     164     * Converts a latitude/longitude pair to x and y coordinates in the
     165     * Transverse Mercator projection.  Note that Transverse Mercator is not
     166     * the same as UTM; a scale factor is required to convert between them.
     167     *
     168     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
     169     * GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
     170     *
     171     * Inputs:
     172     *    phi - Latitude of the point, in radians.
     173     *    lambda - Longitude of the point, in radians.
     174     *    lambda0 - Longitude of the central meridian to be used, in radians.
     175     *
     176     * Outputs:
     177     *    xy - A 2-element array containing the x and y coordinates
     178     *         of the computed point.
     179     *
     180     * Returns:
     181     *    The function does not return a value.
     182     *
     183     */
     184    public EastNorth mapLatLonToXY(double phi, double lambda, double lambda0)
    186185    {
    187186        /* Precalculate ep2 */
    … …  
    197196        double t = Math.tan (phi);
    198197        double t2 = t * t;
    199         double tmp = (t2 * t2 * t2) - Math.pow (t, 6.0);
    200198
    201199        /* Precalculate l */
    … …  
    211209
    212210        double l5coef = 5.0 - 18.0 * t2 + (t2 * t2) + 14.0 * nu2
    213             - 58.0 * t2 * nu2;
     211        - 58.0 * t2 * nu2;
    214212
    215213        double l6coef = 61.0 - 58.0 * t2 + (t2 * t2) + 270.0 * nu2
    216             - 330.0 * t2 * nu2;
     214        - 330.0 * t2 * nu2;
    217215
    218216        double l7coef = 61.0 - 479.0 * t2 + 179.0 * (t2 * t2) - (t2 * t2 * t2);
    … …  
    221219
    222220        return new EastNorth(
    223         /* Calculate easting (x) */
    224         N * Math.cos (phi) * l
    225             + (N / 6.0 * Math.pow (Math.cos (phi), 3.0) * l3coef * Math.pow (l, 3.0))
    226             + (N / 120.0 * Math.pow (Math.cos (phi), 5.0) * l5coef * Math.pow (l, 5.0))
    227             + (N / 5040.0 * Math.pow (Math.cos (phi), 7.0) * l7coef * Math.pow (l, 7.0)),
    228         /* Calculate northing (y) */
    229         ArcLengthOfMeridian (phi)
    230             + (t / 2.0 * N * Math.pow (Math.cos (phi), 2.0) * Math.pow (l, 2.0))
    231             + (t / 24.0 * N * Math.pow (Math.cos (phi), 4.0) * l4coef * Math.pow (l, 4.0))
    232             + (t / 720.0 * N * Math.pow (Math.cos (phi), 6.0) * l6coef * Math.pow (l, 6.0))
    233             + (t / 40320.0 * N * Math.pow (Math.cos (phi), 8.0) * l8coef * Math.pow (l, 8.0)));
     221                /* Calculate easting (x) */
     222                N * Math.cos (phi) * l
     223                + (N / 6.0 * Math.pow (Math.cos (phi), 3.0) * l3coef * Math.pow (l, 3.0))
     224                + (N / 120.0 * Math.pow (Math.cos (phi), 5.0) * l5coef * Math.pow (l, 5.0))
     225                + (N / 5040.0 * Math.pow (Math.cos (phi), 7.0) * l7coef * Math.pow (l, 7.0)),
     226                /* Calculate northing (y) */
     227                ArcLengthOfMeridian (phi)
     228                + (t / 2.0 * N * Math.pow (Math.cos (phi), 2.0) * Math.pow (l, 2.0))
     229                + (t / 24.0 * N * Math.pow (Math.cos (phi), 4.0) * l4coef * Math.pow (l, 4.0))
     230                + (t / 720.0 * N * Math.pow (Math.cos (phi), 6.0) * l6coef * Math.pow (l, 6.0))
     231                + (t / 40320.0 * N * Math.pow (Math.cos (phi), 8.0) * l8coef * Math.pow (l, 8.0)));
    234232    }
    235233
    236234    /*
    237     * MapXYToLatLon
    238     *
    239     * Converts x and y coordinates in the Transverse Mercator projection to
    240     * a latitude/longitude pair.  Note that Transverse Mercator is not
    241     * the same as UTM; a scale factor is required to convert between them.
    242     *
    243     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
    244     *   GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
    245     *
    246     * Inputs:
    247     *   x - The easting of the point, in meters.
    248     *   y - The northing of the point, in meters.
    249     *   lambda0 - Longitude of the central meridian to be used, in radians.
    250     *
    251     * Outputs:
    252     *   philambda - A 2-element containing the latitude and longitude
    253     *               in radians.
    254     *
    255     * Returns:
    256     *   The function does not return a value.
    257     *
    258     * Remarks:
    259     *   The local variables Nf, nuf2, tf, and tf2 serve the same purpose as
    260     *   N, nu2, t, and t2 in MapLatLonToXY, but they are computed with respect
    261     *   to the footpoint latitude phif.
    262     *
    263     *   x1frac, x2frac, x2poly, x3poly, etc. are to enhance readability and
    264     *   to optimize computations.
    265     *
    266     */
    267     public LatLon MapXYToLatLon(double x, double y, double lambda0)
     235     * MapXYToLatLon
     236     *
     237     * Converts x and y coordinates in the Transverse Mercator projection to
     238     * a latitude/longitude pair.  Note that Transverse Mercator is not
     239     * the same as UTM; a scale factor is required to convert between them.
     240     *
     241     * Reference: Hoffmann-Wellenhof, B., Lichtenegger, H., and Collins, J.,
     242     *   GPS: Theory and Practice, 3rd ed.  New York: Springer-Verlag Wien, 1994.
     243     *
     244     * Inputs:
     245     *   x - The easting of the point, in meters.
     246     *   y - The northing of the point, in meters.
     247     *   lambda0 - Longitude of the central meridian to be used, in radians.
     248     *
     249     * Outputs:
     250     *   philambda - A 2-element containing the latitude and longitude
     251     *               in radians.
     252     *
     253     * Returns:
     254     *   The function does not return a value.
     255     *
     256     * Remarks:
     257     *   The local variables Nf, nuf2, tf, and tf2 serve the same purpose as
     258     *   N, nu2, t, and t2 in MapLatLonToXY, but they are computed with respect
     259     *   to the footpoint latitude phif.
     260     *
     261     *   x1frac, x2frac, x2poly, x3poly, etc. are to enhance readability and
     262     *   to optimize computations.
     263     *
     264     */
     265    public LatLon mapXYToLatLon(double x, double y, double lambda0)
    268266    {
    269267        /* Get the value of phif, the footpoint latitude. */
    … …  
    272270        /* Precalculate ep2 */
    273271        double ep2 = (Math.pow (Ellipsoid.GRS80.a, 2.0) - Math.pow (Ellipsoid.GRS80.b, 2.0))
    274               / Math.pow (Ellipsoid.GRS80.b, 2.0);
     272        / Math.pow (Ellipsoid.GRS80.b, 2.0);
    275273
    276274        /* Precalculate cos (phif) */
    … …  
    325323
    326324        return new LatLon(
    327         /* Calculate latitude */
    328         Math.toDegrees(
    329         phif + x2frac * x2poly * (x * x)
    330         + x4frac * x4poly * Math.pow (x, 4.0)
    331         + x6frac * x6poly * Math.pow (x, 6.0)
    332         + x8frac * x8poly * Math.pow (x, 8.0)),
    333         Math.toDegrees(
    334         /* Calculate longitude */
    335         lambda0 + x1frac * x
    336         + x3frac * x3poly * Math.pow (x, 3.0)
    337         + x5frac * x5poly * Math.pow (x, 5.0)
    338         + x7frac * x7poly * Math.pow (x, 7.0)));
     325                /* Calculate latitude */
     326                Math.toDegrees(
     327                        phif + x2frac * x2poly * (x * x)
     328                        + x4frac * x4poly * Math.pow (x, 4.0)
     329                        + x6frac * x6poly * Math.pow (x, 6.0)
     330                        + x8frac * x8poly * Math.pow (x, 8.0)),
     331                        Math.toDegrees(
     332                                /* Calculate longitude */
     333                                lambda0 + x1frac * x
     334                                + x3frac * x3poly * Math.pow (x, 3.0)
     335                                + x5frac * x5poly * Math.pow (x, 5.0)
     336                                + x7frac * x7poly * Math.pow (x, 7.0)));
    339337    }
    340338
    341339    public EastNorth latlon2eastNorth(LatLon p) {
    342         EastNorth a = MapLatLonToXY(Math.toRadians(p.lat()), Math.toRadians(p.lon()), UTMCentralMeridian(getzone()));
     340        EastNorth a = mapLatLonToXY(Math.toRadians(p.lat()), Math.toRadians(p.lon()), UTMCentralMeridian(getzone()));
    343341        return new EastNorth(a.east() * UTMScaleFactor + 3500000.0, a.north() * UTMScaleFactor);
    344342    }
    345343
    346344    public LatLon eastNorth2latlon(EastNorth p) {
    347         return MapXYToLatLon((p.east()-3500000.0)/UTMScaleFactor, p.north()/UTMScaleFactor, UTMCentralMeridian(getzone()));
     345        return mapXYToLatLon((p.east()-3500000.0)/UTMScaleFactor, p.north()/UTMScaleFactor, UTMCentralMeridian(getzone()));
    348346    }
    349347
    … …  
    414412                {
    415413                    zone = Integer.parseInt(s);
    416                     if(zone <= 0 || zone > 60)
     414                    if(zone <= 0 || zone > 60) {
    417415                        zone = DEFAULT_ZONE;
     416                    }
    418417                    break;
    419418                }
    … …  
    430429                int zoneval = Integer.parseInt(zonestring);
    431430                if(zoneval > 0 && zoneval <= 60)
    432                 {
    433431                    return Collections.singleton(zonestring);
    434                 }
    435432            } catch(NumberFormatException e) {}
    436433        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r2711 r2789  
    333333     * the cancel / close action
    334334     */
    335     class CancelAction extends AbstractAction {
     335    static class CancelAction extends AbstractAction {
    336336        public CancelAction() {
    337337            putValue(NAME, tr("Close"));

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/UrlBasedQueryPanel.java

    r2728 r2789  
    6767        HtmlPanel pnl = new HtmlPanel();
    6868        pnl.setText(
    69                 "<html><body>" 
    70                  + tr("Please enter or paste an URL to retrieve changesets from the OSM API.")
    71                  + "<p><strong>" + tr("Examples") + "</strong></p>"
    72                  + "<ul>"
    73                  + "<li><a href=\"http://www.openstreetmap.org/browse/changesets?open=true\">http://www.openstreetmap.org/browse/changesets?open=true</a></li>"
    74                  + "<li><a href=\"http://api.openstreetmap.org/api/0.6/changesets?open=true\">http://api.openstreetmap.org/api/0.6/changesets?open=true</a></li>"
    75                  + "</ul>"
    76                  + tr("Note that changeset queries are currently always submitted to ''{0}'', regardless of the "
    77                     + "host, port and path of the URL entered below.", OsmApi.getOsmApi().getBaseUrl())
    78                  + "</body></html>"
     69                "<html><body>"
     70                + tr("Please enter or paste an URL to retrieve changesets from the OSM API.")
     71                + "<p><strong>" + tr("Examples") + "</strong></p>"
     72                + "<ul>"
     73                + "<li><a href=\"http://www.openstreetmap.org/browse/changesets?open=true\">http://www.openstreetmap.org/browse/changesets?open=true</a></li>"
     74                + "<li><a href=\"http://api.openstreetmap.org/api/0.6/changesets?open=true\">http://api.openstreetmap.org/api/0.6/changesets?open=true</a></li>"
     75                + "</ul>"
     76                + tr("Note that changeset queries are currently always submitted to ''{0}'', regardless of the "
     77                        + "host, port and path of the URL entered below.", OsmApi.getOsmApi().getBaseUrl())
     78                        + "</body></html>"
    7979        );
    8080        pnl.getEditorPane().addHyperlinkListener(
    … …  
    9191    }
    9292
    93     protected ChangesetQuery buildChangesetQueryFromUrlQuery(String query) {
    94         if (query == null)
    95             return new ChangesetQuery();
    96         query = query.trim();
    97         return null;
    98     }
    99 
    10093    protected void build() {
    10194        setLayout(new GridBagLayout());

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/DownloadRelationMemberTask.java

    r2711 r2789  
    3535    private Collection<OsmPrimitive> children;
    3636    private OsmDataLayer curLayer;
    37     private MemberTableModel memberTableModel;
    3837    private MultiFetchServerObjectReader objectReader;
    3938
    … …  
    4342        this.children = children;
    4443        this.curLayer = curLayer;
    45         this.memberTableModel = memberTableModel;
    4644    }
    4745
    … …  
    5149        this.children = children;
    5250        this.curLayer = curLayer;
    53         this.memberTableModel = memberTableModel;
    5451    }
    5552

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

    r2781 r2789  
    103103        JosmUserIdentityManager im = null;
    104104        try {
     105            im = JosmUserIdentityManager.getInstance();
    105106            OsmServerUserInfoReader reader = new OsmServerUserInfoReader();
    106107            UserInfo info = reader.fetchUserInfo(getProgressMonitor().createSubTaskMonitor(1, false));
    107             im = JosmUserIdentityManager.getInstance();
    108108            im.setFullyIdentified(info.getDisplayName(), info);
    109109        } catch(OsmTransferException e) {

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

    r2512 r2789  
    1919            Mode mode = (Mode)evt.getNewValue();
    2020            switch(mode) {
    21                 case EDITING_DATA: setEnabled(true);
    22                 case UPLOADING_AND_SAVING: setEnabled(false);
     21            case EDITING_DATA: setEnabled(true);
     22            break;
     23            case UPLOADING_AND_SAVING: setEnabled(false);
     24            break;
    2325            }
    2426        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/PrefJPanel.java

    r2787 r2789  
    2727 * If someone wants to merge it with ShortcutPreference.java, feel free.
    2828 */
    29 public class prefJPanel extends javax.swing.JPanel {
     29public class PrefJPanel extends javax.swing.JPanel {
    3030
    3131        // table of shortcuts
    … …  
    9696    /** Creates new form prefJPanel */
    9797    // Ain't those auto-generated comments helpful or what? <g>
    98     public prefJPanel(TableModel model) {
     98    public PrefJPanel(TableModel model) {
    9999        this.model = model;
    100100        initComponents();
    … …  
    372372    // more expirience with GUI coding than I have.
    373373    private class cbAction extends javax.swing.AbstractAction implements ListSelectionListener {
    374         private prefJPanel panel;
    375             public cbAction (prefJPanel panel) {
     374        private PrefJPanel panel;
     375            public cbAction (PrefJPanel panel) {
    376376                this.panel = panel;
    377377        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/ShortcutPreference.java

    r2745 r2789  
    3030                tr("Changing keyboard shortcuts manually."), false);
    3131
    32         prefJPanel prefpanel = new prefJPanel(new scListModel());
     32        PrefJPanel prefpanel = new PrefJPanel(new scListModel());
    3333        p.add(prefpanel, GBC.eol().fill(GBC.BOTH));
    3434

  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r2711 r2789  
    334334                value = value.substring(0,value.length() - 1) + "GMT+00:00";
    335335            }
    336             DateFormat formatter = new SimpleDateFormat("MM/dd/yy");
    337             formatter = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ssz");
     336            DateFormat formatter = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ssz");
    338337            try {
    339338                return formatter.parse(value);

  • trunk/src/org/openstreetmap/josm/io/GpxReader.java

    r2512 r2789  
    4141     */
    4242    public GpxData data;
    43     public enum state { init, metadata, wpt, rte, trk, ext, author, link, trkseg, copyright}
     43    public enum State { init, metadata, wpt, rte, trk, ext, author, link, trkseg, copyright}
    4444
    4545    private class Parser extends DefaultHandler {
    … …  
    5151        private WayPoint currentWayPoint;
    5252
    53         private state currentState = state.init;
     53        private State currentState = State.init;
    5454
    5555        private GpxLink currentLink;
    56         private Stack<state> states;
     56        private Stack<State> states;
    5757
    5858        private StringBuffer accumulator = new StringBuffer();
    … …  
    6262        @Override public void startDocument() {
    6363            accumulator = new StringBuffer();
    64             states = new Stack<state>();
     64            states = new Stack<State>();
    6565            currentData = new GpxData();
    6666        }
    … …  
    8282        @Override public void startElement(String namespaceURI, String qName, String rqName, Attributes atts) throws SAXException {
    8383            switch(currentState) {
    84                 case init:
    85                     if (qName.equals("metadata")) {
    86                         states.push(currentState);
    87                         currentState = state.metadata;
    88                     } else if (qName.equals("wpt")) {
    89                         states.push(currentState);
    90                         currentState = state.wpt;
    91                         currentWayPoint = new WayPoint(parseLatLon(atts));
    92                     } else if (qName.equals("rte")) {
    93                         states.push(currentState);
    94                         currentState = state.rte;
    95                         currentRoute = new GpxRoute();
    96                     } else if (qName.equals("trk")) {
    97                         states.push(currentState);
    98                         currentState = state.trk;
    99                         currentTrack = new GpxTrack();
    100                     } else if (qName.equals("extensions")) {
    101                         states.push(currentState);
    102                         currentState = state.ext;
    103                     } else if (qName.equals("gpx") && atts.getValue("creator") != null && atts.getValue("creator").startsWith("Nokia Sports Tracker")) {
    104                         nokiaSportsTrackerBug = true;
    105                     }
    106                     break;
    107                 case author:
    108                     if (qName.equals("link")) {
    109                         states.push(currentState);
    110                         currentState = state.link;
    111                         currentLink = new GpxLink(atts.getValue("href"));
    112                     } else if (qName.equals("email")) {
    113                         currentData.attr.put(GpxData.META_AUTHOR_EMAIL, atts.getValue("id") + "@" + atts.getValue("domain"));
    114                     }
    115                     break;
    116                 case trk:
    117                     if (qName.equals("trkseg")) {
    118                         states.push(currentState);
    119                         currentState = state.trkseg;
    120                         currentTrackSeg = new ArrayList<WayPoint>();
    121                     } else if (qName.equals("link")) {
    122                         states.push(currentState);
    123                         currentState = state.link;
    124                         currentLink = new GpxLink(atts.getValue("href"));
    125                     } else if (qName.equals("extensions")) {
    126                         states.push(currentState);
    127                         currentState = state.ext;
    128                     }
    129                     break;
    130                 case metadata:
    131                     if (qName.equals("author")) {
    132                         states.push(currentState);
    133                         currentState = state.author;
    134                     } else if (qName.equals("extensions")) {
    135                         states.push(currentState);
    136                         currentState = state.ext;
    137                     } else if (qName.equals("copyright")) {
    138                         states.push(currentState);
    139                         currentState = state.copyright;
    140                         currentData.attr.put(GpxData.META_COPYRIGHT_AUTHOR, atts.getValue("author"));
    141                     } else if (qName.equals("link")) {
    142                         states.push(currentState);
    143                         currentState = state.link;
    144                         currentLink = new GpxLink(atts.getValue("href"));
    145                     }
    146                     break;
    147                 case trkseg:
    148                     if (qName.equals("trkpt")) {
    149                         states.push(currentState);
    150                         currentState = state.wpt;
    151                         currentWayPoint = new WayPoint(parseLatLon(atts));
    152                     }
    153                     break;
    154                 case wpt:
    155                     if (qName.equals("link")) {
    156                         states.push(currentState);
    157                         currentState = state.link;
    158                         currentLink = new GpxLink(atts.getValue("href"));
    159                     } else if (qName.equals("extensions")) {
    160                         states.push(currentState);
    161                         currentState = state.ext;
    162                     }
    163                     break;
    164                 case rte:
    165                     if (qName.equals("link")) {
    166                         states.push(currentState);
    167                         currentState = state.link;
    168                         currentLink = new GpxLink(atts.getValue("href"));
    169                     } else if (qName.equals("rtept")) {
    170                         states.push(currentState);
    171                         currentState = state.wpt;
    172                         currentWayPoint = new WayPoint(parseLatLon(atts));
    173                     } else if (qName.equals("extensions")) {
    174                         states.push(currentState);
    175                         currentState = state.ext;
    176                     }
    177                     break;
    178                 default:
     84            case init:
     85                if (qName.equals("metadata")) {
     86                    states.push(currentState);
     87                    currentState = State.metadata;
     88                } else if (qName.equals("wpt")) {
     89                    states.push(currentState);
     90                    currentState = State.wpt;
     91                    currentWayPoint = new WayPoint(parseLatLon(atts));
     92                } else if (qName.equals("rte")) {
     93                    states.push(currentState);
     94                    currentState = State.rte;
     95                    currentRoute = new GpxRoute();
     96                } else if (qName.equals("trk")) {
     97                    states.push(currentState);
     98                    currentState = State.trk;
     99                    currentTrack = new GpxTrack();
     100                } else if (qName.equals("extensions")) {
     101                    states.push(currentState);
     102                    currentState = State.ext;
     103                } else if (qName.equals("gpx") && atts.getValue("creator") != null && atts.getValue("creator").startsWith("Nokia Sports Tracker")) {
     104                    nokiaSportsTrackerBug = true;
     105                }
     106                break;
     107            case author:
     108                if (qName.equals("link")) {
     109                    states.push(currentState);
     110                    currentState = State.link;
     111                    currentLink = new GpxLink(atts.getValue("href"));
     112                } else if (qName.equals("email")) {
     113                    currentData.attr.put(GpxData.META_AUTHOR_EMAIL, atts.getValue("id") + "@" + atts.getValue("domain"));
     114                }
     115                break;
     116            case trk:
     117                if (qName.equals("trkseg")) {
     118                    states.push(currentState);
     119                    currentState = State.trkseg;
     120                    currentTrackSeg = new ArrayList<WayPoint>();
     121                } else if (qName.equals("link")) {
     122                    states.push(currentState);
     123                    currentState = State.link;
     124                    currentLink = new GpxLink(atts.getValue("href"));
     125                } else if (qName.equals("extensions")) {
     126                    states.push(currentState);
     127                    currentState = State.ext;
     128                }
     129                break;
     130            case metadata:
     131                if (qName.equals("author")) {
     132                    states.push(currentState);
     133                    currentState = State.author;
     134                } else if (qName.equals("extensions")) {
     135                    states.push(currentState);
     136                    currentState = State.ext;
     137                } else if (qName.equals("copyright")) {
     138                    states.push(currentState);
     139                    currentState = State.copyright;
     140                    currentData.attr.put(GpxData.META_COPYRIGHT_AUTHOR, atts.getValue("author"));
     141                } else if (qName.equals("link")) {
     142                    states.push(currentState);
     143                    currentState = State.link;
     144                    currentLink = new GpxLink(atts.getValue("href"));
     145                }
     146                break;
     147            case trkseg:
     148                if (qName.equals("trkpt")) {
     149                    states.push(currentState);
     150                    currentState = State.wpt;
     151                    currentWayPoint = new WayPoint(parseLatLon(atts));
     152                }
     153                break;
     154            case wpt:
     155                if (qName.equals("link")) {
     156                    states.push(currentState);
     157                    currentState = State.link;
     158                    currentLink = new GpxLink(atts.getValue("href"));
     159                } else if (qName.equals("extensions")) {
     160                    states.push(currentState);
     161                    currentState = State.ext;
     162                }
     163                break;
     164            case rte:
     165                if (qName.equals("link")) {
     166                    states.push(currentState);
     167                    currentState = State.link;
     168                    currentLink = new GpxLink(atts.getValue("href"));
     169                } else if (qName.equals("rtept")) {
     170                    states.push(currentState);
     171                    currentState = State.wpt;
     172                    currentWayPoint = new WayPoint(parseLatLon(atts));
     173                } else if (qName.equals("extensions")) {
     174                    states.push(currentState);
     175                    currentState = State.ext;
     176                }
     177                break;
     178            default:
    179179            }
    180180            accumulator.setLength(0);
    … …  
    201201        private Map<String, Object> getAttr() {
    202202            switch (currentState) {
    203                 case rte: return currentRoute.attr;
    204                 case metadata: return currentData.attr;
    205                 case wpt: return currentWayPoint.attr;
    206                 case trk: return currentTrack.attr;
    207                 default: return null;
    208             }
    209         }
    210 
     203            case rte: return currentRoute.attr;
     204            case metadata: return currentData.attr;
     205            case wpt: return currentWayPoint.attr;
     206            case trk: return currentTrack.attr;
     207            default: return null;
     208            }
     209        }
     210
     211        @SuppressWarnings("unchecked")
    211212        @Override public void endElement(String namespaceURI, String qName, String rqName) {
    212213            switch (currentState) {
    213                 case metadata:
    214                     if (qName.equals("name")) {
    215                         currentData.attr.put(GpxData.META_NAME, accumulator.toString());
    216                     } else if (qName.equals("desc")) {
    217                         currentData.attr.put(GpxData.META_DESC, accumulator.toString());
    218                     } else if (qName.equals("time")) {
    219                         currentData.attr.put(GpxData.META_TIME, accumulator.toString());
    220                     } else if (qName.equals("keywords")) {
    221                         currentData.attr.put(GpxData.META_KEYWORDS, accumulator.toString());
    222                     } else if (qName.equals("metadata")) {
    223                         currentState = states.pop();
     214            case metadata:
     215                if (qName.equals("name")) {
     216                    currentData.attr.put(GpxData.META_NAME, accumulator.toString());
     217                } else if (qName.equals("desc")) {
     218                    currentData.attr.put(GpxData.META_DESC, accumulator.toString());
     219                } else if (qName.equals("time")) {
     220                    currentData.attr.put(GpxData.META_TIME, accumulator.toString());
     221                } else if (qName.equals("keywords")) {
     222                    currentData.attr.put(GpxData.META_KEYWORDS, accumulator.toString());
     223                } else if (qName.equals("metadata")) {
     224                    currentState = states.pop();
     225                }
     226                //TODO: parse bounds, extensions
     227                break;
     228            case author:
     229                if (qName.equals("author")) {
     230                    currentState = states.pop();
     231                } else if (qName.equals("name")) {
     232                    currentData.attr.put(GpxData.META_AUTHOR_NAME, accumulator.toString());
     233                } else if (qName.equals("email")) {
     234                    // do nothing, has been parsed on startElement
     235                } else if (qName.equals("link")) {
     236                    currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
     237                }
     238                break;
     239            case copyright:
     240                if (qName.equals("copyright")) {
     241                    currentState = states.pop();
     242                } else if (qName.equals("year")) {
     243                    currentData.attr.put(GpxData.META_COPYRIGHT_YEAR, accumulator.toString());
     244                } else if (qName.equals("license")) {
     245                    currentData.attr.put(GpxData.META_COPYRIGHT_LICENSE, accumulator.toString());
     246                }
     247                break;
     248            case link:
     249                if (qName.equals("text")) {
     250                    currentLink.text = accumulator.toString();
     251                } else if (qName.equals("type")) {
     252                    currentLink.type = accumulator.toString();
     253                } else if (qName.equals("link")) {
     254                    if (currentLink.uri == null && accumulator != null && accumulator.toString().length() != 0) {
     255                        currentLink = new GpxLink(accumulator.toString());
    224256                    }
    225                     //TODO: parse bounds, extensions
    226                     break;
    227                 case author:
    228                     if (qName.equals("author")) {
    229                         currentState = states.pop();
    230                     } else if (qName.equals("name")) {
    231                         currentData.attr.put(GpxData.META_AUTHOR_NAME, accumulator.toString());
    232                     } else if (qName.equals("email")) {
    233                         // do nothing, has been parsed on startElement
    234                     } else if (qName.equals("link")) {
    235                         currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
     257                    currentState = states.pop();
     258                }
     259                if (currentState == State.author) {
     260                    currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
     261                } else if (currentState != State.link) {
     262                    Map<String, Object> attr = getAttr();
     263                    if (!attr.containsKey(GpxData.META_LINKS)) {
     264                        attr.put(GpxData.META_LINKS, new LinkedList<GpxLink>());
    236265                    }
    237                     break;
    238                 case copyright:
    239                     if (qName.equals("copyright")) {
    240                         currentState = states.pop();
    241                     } else if (qName.equals("year")) {
    242                         currentData.attr.put(GpxData.META_COPYRIGHT_YEAR, accumulator.toString());
    243                     } else if (qName.equals("license")) {
    244                         currentData.attr.put(GpxData.META_COPYRIGHT_LICENSE, accumulator.toString());
     266                    ((Collection<GpxLink>) attr.get(GpxData.META_LINKS)).add(currentLink);
     267                }
     268                break;
     269            case wpt:
     270                if (   qName.equals("ele")  || qName.equals("magvar")
     271                        || qName.equals("name") || qName.equals("geoidheight")
     272                        || qName.equals("type") || qName.equals("sym")) {
     273                    currentWayPoint.attr.put(qName, accumulator.toString());
     274                } else if(qName.equals("hdop") /*|| qName.equals("vdop") ||
     275                        qName.equals("pdop")*/) {
     276                    try {
     277                        currentWayPoint.attr.put(qName, Float.parseFloat(accumulator.toString()));
     278                    } catch(Exception e) {
     279                        currentWayPoint.attr.put(qName, new Float(0));
    245280                    }
    246                     break;
    247                 case link:
    248                     if (qName.equals("text")) {
    249                         currentLink.text = accumulator.toString();
    250                     } else if (qName.equals("type")) {
    251                         currentLink.type = accumulator.toString();
    252                     } else if (qName.equals("link")) {
    253                         if (currentLink.uri == null && accumulator != null && accumulator.toString().length() != 0) {
    254                             currentLink = new GpxLink(accumulator.toString());
    255                         }
    256                         currentState = states.pop();
    257                     }
    258                     if (currentState == state.author) {
    259                         currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
    260                     } else if (currentState != state.link) {
    261                         Map<String, Object> attr = getAttr();
    262                         if (!attr.containsKey(GpxData.META_LINKS)) {
    263                             attr.put(GpxData.META_LINKS, new LinkedList<GpxLink>());
    264                         }
    265                         ((Collection<GpxLink>) attr.get(GpxData.META_LINKS)).add(currentLink);
    266                     }
    267                     break;
    268                 case wpt:
    269                     if (   qName.equals("ele")  || qName.equals("magvar")
    270                             || qName.equals("name") || qName.equals("geoidheight")
    271                             || qName.equals("type") || qName.equals("sym")) {
    272                         currentWayPoint.attr.put(qName, accumulator.toString());
    273                     } else if(qName.equals("hdop") /*|| qName.equals("vdop") ||
    274                         qName.equals("pdop")*/) {
    275                         try {
    276                             currentWayPoint.attr.put(qName, Float.parseFloat(accumulator.toString()));
    277                         } catch(Exception e) {
    278                             currentWayPoint.attr.put(qName, new Float(0));
    279                         }
    280                     } else if (qName.equals("time")) {
    281                         currentWayPoint.attr.put(qName, accumulator.toString());
    282                         currentWayPoint.setTime();
    283                     } else if (qName.equals("cmt") || qName.equals("desc")) {
    284                         currentWayPoint.attr.put(qName, accumulator.toString());
    285                         currentWayPoint.setTime();
    286                     } else if (qName.equals("rtept")) {
    287                         currentState = states.pop();
    288                         currentRoute.routePoints.add(currentWayPoint);
    289                     } else if (qName.equals("trkpt")) {
    290                         currentState = states.pop();
    291                         currentTrackSeg.add(currentWayPoint);
    292                     } else if (qName.equals("wpt")) {
    293                         currentState = states.pop();
    294                         currentData.waypoints.add(currentWayPoint);
    295                     }
    296                     break;
    297                 case trkseg:
    298                     if (qName.equals("trkseg")) {
    299                         currentState = states.pop();
    300                         currentTrack.trackSegs.add(currentTrackSeg);
    301                     }
    302                     break;
    303                 case trk:
    304                     if (qName.equals("trk")) {
    305                         currentState = states.pop();
    306                         currentData.tracks.add(currentTrack);
    307                     } else if (qName.equals("name") || qName.equals("cmt")
    308                             || qName.equals("desc") || qName.equals("src")
    309                             || qName.equals("type") || qName.equals("number")
    310                             || qName.equals("url")) {
    311                         currentTrack.attr.put(qName, accumulator.toString());
    312                     }
    313                     break;
    314                 case ext:
    315                     if (qName.equals("extensions")) {
    316                         currentState = states.pop();
    317                     }
    318                     break;
    319                 default:
    320                     if (qName.equals("wpt")) {
    321                         currentState = states.pop();
    322                     } else if (qName.equals("rte")) {
    323                         currentState = states.pop();
    324                         currentData.routes.add(currentRoute);
    325                     }
     281                } else if (qName.equals("time")) {
     282                    currentWayPoint.attr.put(qName, accumulator.toString());
     283                    currentWayPoint.setTime();
     284                } else if (qName.equals("cmt") || qName.equals("desc")) {
     285                    currentWayPoint.attr.put(qName, accumulator.toString());
     286                    currentWayPoint.setTime();
     287                } else if (qName.equals("rtept")) {
     288                    currentState = states.pop();
     289                    currentRoute.routePoints.add(currentWayPoint);
     290                } else if (qName.equals("trkpt")) {
     291                    currentState = states.pop();
     292                    currentTrackSeg.add(currentWayPoint);
     293                } else if (qName.equals("wpt")) {
     294                    currentState = states.pop();
     295                    currentData.waypoints.add(currentWayPoint);
     296                }
     297                break;
     298            case trkseg:
     299                if (qName.equals("trkseg")) {
     300                    currentState = states.pop();
     301                    currentTrack.trackSegs.add(currentTrackSeg);
     302                }
     303                break;
     304            case trk:
     305                if (qName.equals("trk")) {
     306                    currentState = states.pop();
     307                    currentData.tracks.add(currentTrack);
     308                } else if (qName.equals("name") || qName.equals("cmt")
     309                        || qName.equals("desc") || qName.equals("src")
     310                        || qName.equals("type") || qName.equals("number")
     311                        || qName.equals("url")) {
     312                    currentTrack.attr.put(qName, accumulator.toString());
     313                }
     314                break;
     315            case ext:
     316                if (qName.equals("extensions")) {
     317                    currentState = states.pop();
     318                }
     319                break;
     320            default:
     321                if (qName.equals("wpt")) {
     322                    currentState = states.pop();
     323                } else if (qName.equals("rte")) {
     324                    currentState = states.pop();
     325                    currentData.routes.add(currentRoute);
     326                }
    326327            }
    327328        }

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

    r2688 r2789  
    4949            reference = reference.getParent();
    5050        }
    51         if (reference == null || ! (reference instanceof Window))
     51        if (reference == null)
    5252            return new WindowGeometry(new Point(0,0), extent);
    5353        parentWindow = (Window)reference;

  • trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryNodeTest.java

    r2252 r2789  
    1414
    1515    @Test
    16     public void HistoryNode() {
     16    public void historyNode() {
    1717        Date d = new Date();
    1818        HistoryNode node = new HistoryNode(
Note: See TracChangeset for help on using the changeset viewer.