Changeset 4874 in josm


Ignore:
Timestamp:
Jan 26, 2012 9:52:34 PM (16 months ago)
Author:
jttt
Message:

Use static class were appropriate

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

Legend:

Unmodified
Added
Removed

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

    r4733 r4874  
    22package org.openstreetmap.josm.actions; 
    33 
     4import static org.openstreetmap.josm.gui.help.HelpUtil.ht; 
    45import static org.openstreetmap.josm.tools.I18n.tr; 
    5 import static org.openstreetmap.josm.gui.help.HelpUtil.ht; 
    66 
    77import java.awt.event.ActionEvent; 
    … …  
    1616 
    1717import org.openstreetmap.josm.Main; 
    18 import org.openstreetmap.josm.actions.ExtensionFileFilter; 
    1918import org.openstreetmap.josm.gui.HelpAwareOptionPane; 
    2019import org.openstreetmap.josm.gui.PleaseWaitRunnable; 
    … …  
    5756    } 
    5857 
    59     public class Loader extends PleaseWaitRunnable { 
     58    public static class Loader extends PleaseWaitRunnable { 
    6059 
    6160        private boolean canceled; 
    … …  
    8988                    for (Runnable task : postLoadTasks) { 
    9089                        if (canceled) return; 
    91                         if (task == null) continue; 
     90                        if (task == null) { 
     91                            continue; 
     92                        } 
    9293                        task.run(); 
    9394                    } 
    … …  
    112113                        JOptionPane.ERROR_MESSAGE, 
    113114                        null 
    114                 ); 
     115                        ); 
    115116                cancel(); 
    116117            } catch (IOException e) { 
    … …  
    122123                        JOptionPane.ERROR_MESSAGE, 
    123124                        null 
    124                 ); 
     125                        ); 
    125126                cancel(); 
    126127            } catch (RuntimeException e) { 

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

    r4076 r4874  
    3434 */ 
    3535public class ValidateAction extends JosmAction { 
    36      
     36 
    3737    /** Serializable ID */ 
    3838    private static final long serialVersionUID = -2304521273582574603L; 
    … …  
    4646    public ValidateAction() { 
    4747        super(tr("Validation"), "dialogs/validator", tr("Performs the data validation"), 
    48         Shortcut.registerShortcut("tools:validate", tr("Tool: {0}", tr("Validation")), KeyEvent.VK_V, Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true); 
     48                Shortcut.registerShortcut("tools:validate", tr("Tool: {0}", tr("Validation")), KeyEvent.VK_V, Shortcut.GROUP_EDIT, Shortcut.SHIFT_DEFAULT), true); 
    4949    } 
    5050 
    … …  
    106106     * 
    107107     */ 
    108     class ValidationTask extends PleaseWaitRunnable { 
     108    static class ValidationTask extends PleaseWaitRunnable { 
    109109        private Collection<Test> tests; 
    110110        private Collection<OsmPrimitive> validatedPrimitives; 
    … …  
    154154        @Override 
    155155        protected void realRun() throws SAXException, IOException, 
    156                 OsmTransferException { 
     156        OsmTransferException { 
    157157            if (tests == null || tests.isEmpty()) 
    158158                return; 

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmChangeTask.java

    r4734 r4874  
    5757        return Main.worker.submit(downloadTask); 
    5858    } 
    59      
     59 
    6060    protected class DownloadTask extends DownloadOsmTask.DownloadTask { 
    6161 
    … …  
    114114     * 
    115115     */ 
    116     private class HistoryListener implements HistoryDataSetListener { 
     116    private static class HistoryListener implements HistoryDataSetListener { 
    117117 
    118118        private final List<Pair<OsmPrimitive, Date>> toMonitor; 
    119          
     119 
    120120        public HistoryListener(List<Pair<OsmPrimitive, Date>> toMonitor) { 
    121121            this.toMonitor = toMonitor; 
    … …  
    133133                    if (hp != null) { 
    134134                        PrimitiveData data = null; 
    135                          
     135 
    136136                        switch (pair.a.getType()) { 
    137                             case NODE: 
    138                                 data = new NodeData(); 
    139                                 ((NodeData)data).setCoor(((HistoryNode)hp).getCoords()); 
    140                                 break; 
    141                             case WAY:  
    142                                 data = new WayData(); 
    143                                 ((WayData)data).setNodes(((HistoryWay)hp).getNodes()); 
    144                                 break; 
    145                             case RELATION:  
    146                                 data = new RelationData(); 
    147                                 ((RelationData)data).setMembers(((HistoryRelation)hp).getMembers()); 
    148                                 break; 
    149                             default: throw new AssertionError(); 
     137                        case NODE: 
     138                            data = new NodeData(); 
     139                            ((NodeData)data).setCoor(((HistoryNode)hp).getCoords()); 
     140                            break; 
     141                        case WAY: 
     142                            data = new WayData(); 
     143                            ((WayData)data).setNodes(((HistoryWay)hp).getNodes()); 
     144                            break; 
     145                        case RELATION: 
     146                            data = new RelationData(); 
     147                            ((RelationData)data).setMembers(((HistoryRelation)hp).getMembers()); 
     148                            break; 
     149                        default: throw new AssertionError(); 
    150150                        } 
    151                          
     151 
    152152                        data.setUser(hp.getUser()); 
    153153                        data.setVisible(hp.isVisible()); 
    … …  
    155155                        data.setKeys(hp.getTags()); 
    156156                        data.setOsmId(hp.getChangesetId(), (int) hp.getVersion()); 
    157                          
     157 
    158158                        // Load the history data 
    159159                        pair.a.load(data); 

  • trunk/src/org/openstreetmap/josm/data/APIDataSet.java

    r4534 r4874  
    7373        OsmPrimitiveComparator c = new OsmPrimitiveComparator(); 
    7474        c.relationsFirst = true; 
    75         Collections.sort(toDelete, c);  
    76         Collections.sort(toAdd, c);  
    77         Collections.sort(toUpdate, c);  
     75        Collections.sort(toDelete, c); 
     76        Collections.sort(toAdd, c); 
     77        Collections.sort(toUpdate, c); 
    7878    } 
    7979 
    … …  
    154154        OsmPrimitiveComparator c = new OsmPrimitiveComparator(); 
    155155        c.relationsFirst = true; 
    156         Collections.sort(toDelete, c);  
    157         Collections.sort(toAdd, c);  
    158         Collections.sort(toUpdate, c);  
     156        Collections.sort(toDelete, c); 
     157        Collections.sort(toAdd, c); 
     158        Collections.sort(toUpdate, c); 
    159159    } 
    160160 
    … …  
    278278     * 
    279279     */ 
    280     private class RelationUploadDependencyGraph { 
     280    private static class RelationUploadDependencyGraph { 
    281281        private HashMap<Relation, Set<Relation>> children; 
    282282        private Collection<Relation> relations; 
    … …  
    353353                        } 
    354354                    } 
    355             ); 
     355                    ); 
    356356            return ret; 
    357357        } 

  • trunk/src/org/openstreetmap/josm/data/ServerSidePreferences.java

    r4612 r4874  
    1515import java.net.URL; 
    1616import java.net.URLConnection; 
    17 import java.util.Collection; 
    18 import java.util.Collections; 
    19 import java.util.LinkedList; 
    20 import java.util.StringTokenizer; 
    21 import java.util.Map.Entry; 
    2217 
    2318import javax.swing.JOptionPane; 
    … …  
    2621import org.openstreetmap.josm.Main; 
    2722import org.openstreetmap.josm.io.OsmConnection; 
    28 import org.openstreetmap.josm.io.OsmTransferException; 
    2923import org.openstreetmap.josm.tools.Base64; 
    3024 
    … …  
    3630 */ 
    3731public class ServerSidePreferences extends Preferences { 
    38     public class MissingPassword extends Exception{ 
     32    public static class MissingPassword extends Exception{ 
    3933        public String realm; 
    4034        public MissingPassword(String r) { 
    … …  
    5650                String username = get("applet.username"); 
    5751                String password = get("applet.password"); 
    58                 if(password.isEmpty() && username.isEmpty()) 
     52                if(password.isEmpty() && username.isEmpty()) { 
    5953                    con.addRequestProperty("Authorization", "Basic "+Base64.encode(username+":"+password)); 
     54                } 
    6055                con.connect(); 
    6156                if(username.isEmpty() && con instanceof HttpURLConnection 
    62                     && ((HttpURLConnection) con).getResponseCode() 
    63                     == HttpURLConnection.HTTP_UNAUTHORIZED) { 
     57                        && ((HttpURLConnection) con).getResponseCode() 
     58                        == HttpURLConnection.HTTP_UNAUTHORIZED) { 
    6459                    String t = ((HttpURLConnection) con).getHeaderField("WWW-Authenticate"); 
    6560                    t = t.replace("Basic realm=\"","").replace("\"",""); 
    … …  
    8883                String username = get("applet.username"); 
    8984                String password = get("applet.password"); 
    90                 if(password.isEmpty() && username.isEmpty()) 
     85                if(password.isEmpty() && username.isEmpty()) { 
    9186                    con.addRequestProperty("Authorization", "Basic "+Base64.encode(username+":"+password)); 
     87                } 
    9288                con.setRequestMethod("POST"); 
    9389                con.setDoOutput(true); 
    … …  
    10399                        tr("Information"), 
    104100                        JOptionPane.INFORMATION_MESSAGE 
    105                 ); 
     101                        ); 
    106102            } catch (Exception e) { 
    107103                e.printStackTrace(); 
    … …  
    111107                        tr("Error"), 
    112108                        JOptionPane.ERROR_MESSAGE 
    113                 ); 
     109                        ); 
    114110            } 
    115111        } 
    … …  
    127123                    tr("Error"), 
    128124                    JOptionPane.ERROR_MESSAGE 
    129             ); 
     125                    ); 
    130126        } 
    131127        this.connection = connection; 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/DeprecatedTags.java

    r4806 r4874  
    11package org.openstreetmap.josm.data.validation.tests; 
    22 
    3 import org.openstreetmap.josm.data.osm.Node; 
    4 import org.openstreetmap.josm.data.osm.Relation; 
    5 import org.openstreetmap.josm.data.osm.Way; 
    63import static org.openstreetmap.josm.tools.I18n.tr; 
    74 
    … …  
    96import java.util.LinkedList; 
    107import java.util.List; 
     8 
    119import org.openstreetmap.josm.command.ChangePropertyCommand; 
    1210import org.openstreetmap.josm.command.Command; 
    1311import org.openstreetmap.josm.command.SequenceCommand; 
     12import org.openstreetmap.josm.data.osm.Node; 
    1413import org.openstreetmap.josm.data.osm.OsmPrimitive; 
     14import org.openstreetmap.josm.data.osm.Relation; 
    1515import org.openstreetmap.josm.data.osm.Tag; 
     16import org.openstreetmap.josm.data.osm.Way; 
    1617import org.openstreetmap.josm.data.validation.Severity; 
    1718import org.openstreetmap.josm.data.validation.Test; 
    … …  
    9798    } 
    9899 
    99     private class DeprecationCheck { 
     100    private static class DeprecationCheck { 
    100101 
    101102        int code; 
    … …  
    148149                String key = tag.getKey(); 
    149150                String value = tag.getValue(); 
    150                 if (value.isEmpty() && !p.hasKey(key)) { 
     151                if (value.isEmpty() && !p.hasKey(key)) 
    151152                    return false; 
    152                 } 
    153                 if (!value.isEmpty() && !value.equals(p.get(key))) { 
     153                if (!value.isEmpty() && !value.equals(p.get(key))) 
    154154                    return false; 
    155                 } 
    156155            } 
    157156            return true; 
    … …  
    167166 
    168167        String getDescription() { 
    169             if (alternatives.isEmpty()) { 
     168            if (alternatives.isEmpty()) 
    170169                return tr("{0} is deprecated", Utils.join(", ", test)); 
    171             } else { 
     170            else 
    172171                return tr("{0} is deprecated, use {1} instead", Utils.join(", ", test), Utils.join(tr(" or "), alternatives)); 
    173             } 
    174172        } 
    175173    } 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateNode.java

    r4869 r4874  
    3939public class DuplicateNode extends Test { 
    4040 
    41     private class NodeHash implements Hash<Object, Object> { 
     41    private static class NodeHash implements Hash<Object, Object> { 
    4242 
    4343        double precision = Main.pref.getDouble("validator.duplicatenodes.precision", 0.); 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateRelation.java

    r4869 r4874  
    3434{ 
    3535 
    36     public class RelMember { 
     36    public static class RelMember { 
    3737        private String role; 
    3838        private OsmPrimitiveType type; 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/DuplicateWay.java

    r4869 r4874  
    3434{ 
    3535 
    36     private class WayPair { 
     36    private static class WayPair { 
    3737        public List<LatLon> coor; 
    3838        public Map<String, String> keys; 
    … …  
    5656    } 
    5757 
    58     private class WayPairNoTags { 
     58    private static class WayPairNoTags { 
    5959        public List<LatLon> coor; 
    6060        public WayPairNoTags(List<LatLon> _coor) { 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r4869 r4874  
    7070    } 
    7171 
    72     public class RoleInfo { 
     72    public static class RoleInfo { 
    7373        int total = 0; 
    7474        Collection<Node> nodes = new LinkedList<Node>(); 

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

    r4869 r4874  
    12171217    } 
    12181218 
    1219     private class CursorInfo { 
     1219    private static class CursorInfo { 
    12201220        public Cursor cursor; 
    12211221        public Object object; 

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

    r4363 r4874  
    143143 
    144144        createLayout(p, true, Arrays.asList(new SideButton[] { 
    145             new SideButton(onoffAction), 
    146             new SideButton(upAction), 
    147             new SideButton(downAction), 
    148             new SideButton(new LaunchMapPaintPreferencesAction()) 
     145                new SideButton(onoffAction), 
     146                new SideButton(upAction), 
     147                new SideButton(downAction), 
     148                new SideButton(new LaunchMapPaintPreferencesAction()) 
    149149        })); 
    150150    } 
    … …  
    359359     * Opens preferences window and selects the mappaint tab. 
    360360     */ 
    361     class LaunchMapPaintPreferencesAction extends AbstractAction { 
     361    static class LaunchMapPaintPreferencesAction extends AbstractAction { 
    362362        public LaunchMapPaintPreferencesAction() { 
    363363            putValue(SMALL_ICON, ImageProvider.get("dialogs", "mappaintpreference")); 

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

    r3788 r4874  
    3030 */ 
    3131public class RelationNodeMap { 
    32     private class NodesWays{ 
     32    private static class NodesWays{ 
    3333        public Map<Node, Set<Integer>> nodes = new TreeMap<Node, Set<Integer>>(); 
    3434        public Map<Integer, Set<Node>> ways = new TreeMap<Integer, Set<Node>>(); 
    … …  
    108108         * cannot be used in future. (only for performance) 
    109109         */ 
    110 //        Iterator<Map.Entry<Node,TreeSet<Integer>>> it = map.nodes.entrySet().iterator(); 
    111 //        while (it.hasNext()) { 
    112 //            Map.Entry<Node,TreeSet<Integer>> nodeLinks = it.next(); 
    113 // 
    114 //            if (nodeLinks.getValue().size() < 2) { 
    115 //                if (nodeLinks.getValue().size() != 1) throw new AssertionError(); 
    116 // 
    117 //                Integer d_way = nodeLinks.getValue().iterator().next(); 
    118 //                TreeSet<Node> d_way_nodes = map.ways.get(d_way); 
    119 //                d_way_nodes.remove(nodeLinks.getKey()); 
    120 // 
    121 //                it.remove(); 
    122 //                continue; 
    123 //            } 
    124 //        } 
    125             } 
     110        //        Iterator<Map.Entry<Node,TreeSet<Integer>>> it = map.nodes.entrySet().iterator(); 
     111        //        while (it.hasNext()) { 
     112        //            Map.Entry<Node,TreeSet<Integer>> nodeLinks = it.next(); 
     113        // 
     114        //            if (nodeLinks.getValue().size() < 2) { 
     115        //                if (nodeLinks.getValue().size() != 1) throw new AssertionError(); 
     116        // 
     117        //                Integer d_way = nodeLinks.getValue().iterator().next(); 
     118        //                TreeSet<Node> d_way_nodes = map.ways.get(d_way); 
     119        //                d_way_nodes.remove(nodeLinks.getKey()); 
     120        // 
     121        //                it.remove(); 
     122        //                continue; 
     123        //            } 
     124        //        } 
     125    } 
    126126 
    127127    private void addPair(Node n, int i) { 
    … …  
    211211            } 
    212212        } 
    213          
     213 
    214214        firstOneway = way; 
    215215        return popForwardOnewayPart(way); 
    … …  
    220220            for (Node n : onewayMap.ways.get(way)) { 
    221221                Integer i = findAdjacentWay(onewayMap, n); 
    222                 if(i == null) continue; 
     222                if(i == null) { 
     223                    continue; 
     224                } 
    223225 
    224226                lastOnewayNode = processBackwardIfEndOfLoopReached(i); 
    225                 if(lastOnewayNode != null){ 
     227                if(lastOnewayNode != null) 
    226228                    return popBackwardOnewayPart(firstOneway); 
    227                 } 
    228229 
    229230                deleteWayNode(onewayMap, i, n); 
    … …  
    231232            } 
    232233        } 
    233          
     234 
    234235        firstOneway = null; 
    235236        return null; 
    … …  
    240241            for (Node n : onewayReverseMap.ways.get(way)) { 
    241242                if((map.nodes.containsKey(n)) 
    242                         || (onewayMap.nodes.containsKey(n) && onewayMap.nodes.get(n).size() > 1)) { 
     243                        || (onewayMap.nodes.containsKey(n) && onewayMap.nodes.get(n).size() > 1)) 
    243244                    return n; 
    244                 } 
    245                 if(firstCircular != null && firstCircular == n) { 
     245                if(firstCircular != null && firstCircular == n) 
    246246                    return firstCircular; 
    247                 } 
    248247            } 
    249248        } 
    250249        return null; 
    251250    } 
    252      
     251 
    253252    private Integer popBackwardOnewayPart(int way){ 
    254253        if (lastOnewayNode != null) { 
    255254            TreeSet<Node> nodes = new TreeSet<Node>(); 
    256             if (onewayReverseMap.ways.containsKey(way)) nodes.addAll(onewayReverseMap.ways.get(way)); 
    257             if (map.ways.containsKey(way)) nodes.addAll(map.ways.get(way)); 
     255            if (onewayReverseMap.ways.containsKey(way)) { 
     256                nodes.addAll(onewayReverseMap.ways.get(way)); 
     257            } 
     258            if (map.ways.containsKey(way)) { 
     259                nodes.addAll(map.ways.get(way)); 
     260            } 
    258261            for (Node n : nodes) { 
    259262                if(n == lastOnewayNode) { //if oneway part ends 
    … …  
    277280        firstOneway = null; 
    278281        lastOnewayNode = null; 
    279          
     282 
    280283        return null; 
    281284    } 
    … …  
    302305 
    303306    private void deleteWayNode(NodesWays nw, Integer way, Node n){ 
    304         if(nw.oneWay) 
     307        if(nw.oneWay) { 
    305308            doneOneway(way); 
    306         else 
     309        } else { 
    307310            done(way); 
     311        } 
    308312        nw.ways.get(way).remove(n); 
    309313    } 
    … …  
    343347        Set<Node> nodesForward = remainingOneway.get(i); 
    344348        for (Node n : nodesForward) { 
    345             if(onewayMap.nodes.containsKey(n)) onewayMap.nodes.get(n).remove(i); 
    346             if(onewayReverseMap.nodes.containsKey(n)) onewayReverseMap.nodes.get(n).remove(i); 
     349            if(onewayMap.nodes.containsKey(n)) { 
     350                onewayMap.nodes.get(n).remove(i); 
     351            } 
     352            if(onewayReverseMap.nodes.containsKey(n)) { 
     353                onewayReverseMap.nodes.get(n).remove(i); 
     354            } 
    347355        } 
    348356        remainingOneway.remove(i); 

  • trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

    r4729 r4874  
    66import java.io.BufferedReader; 
    77import java.io.DataOutputStream; 
     8import java.io.IOException; 
    89import java.io.InputStreamReader; 
    9 import java.io.IOException; 
    1010import java.io.UnsupportedEncodingException; 
    1111import java.lang.reflect.Field; 
    … …  
    4646    private HttpURLConnection connection; 
    4747 
    48     private class SessionId { 
     48    private static class SessionId { 
    4949        String id; 
    5050        String token; 
    … …  
    209209            while((c = r.readLine()) != null) { 
    210210                Matcher m = p.matcher(c); 
    211                 if(m.find()) { 
     211                if(m.find()) 
    212212                    return m.group(1); 
    213                 } 
    214213            } 
    215214        } catch (IOException e) { 

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

    r4802 r4874  
    147147                            if(!previouslyShownUnsupportedCrsError) { 
    148148                                JOptionPane.showMessageDialog(null, tr("That layer does not support any of JOSM''s projections,\n" + 
    149                                 "so you can not use it. This message will not show again."), 
    150                                 tr("WMS Error"), JOptionPane.ERROR_MESSAGE); 
     149                                        "so you can not use it. This message will not show again."), 
     150                                        tr("WMS Error"), JOptionPane.ERROR_MESSAGE); 
    151151                                previouslyShownUnsupportedCrsError = true; 
    152152                            } 
    … …  
    223223        resultingLayerField = new JTextArea(3, 40); 
    224224        resultingLayerField.setLineWrap(true); 
    225         JScrollPane bottomScrollPane = new JScrollPane(resultingLayerField,  
     225        JScrollPane bottomScrollPane = new JScrollPane(resultingLayerField, 
    226226                JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED, JScrollPane.HORIZONTAL_SCROLLBAR_NEVER); 
    227227        bottomScrollPane.setMinimumSize(new Dimension(60, 60)); 
    … …  
    250250        StringBuilder a = new StringBuilder("tms"); 
    251251        String z = sanitize(tmsZoom.getText()); 
    252         if(!z.isEmpty()) 
     252        if(!z.isEmpty()) { 
    253253            a.append("["+z+"]"); 
     254        } 
    254255        a.append(":"); 
    255256        a.append(sanitize(tmsURL.getText())); 
    … …  
    493494        for (Projection proj : Projections.getProjections()) { 
    494495            if (proj instanceof ProjectionSubPrefs) { 
    495                 if (((ProjectionSubPrefs) proj).getPreferencesFromCode(crs) == null) { 
     496                if (((ProjectionSubPrefs) proj).getPreferencesFromCode(crs) == null) 
    496497                    return true; 
    497                 } 
    498498            } else { 
    499                 if (proj.toCode().equals(crs)) { 
     499                if (proj.toCode().equals(crs)) 
    500500                    return true; 
    501                 } 
    502501            } 
    503502        } 
    … …  
    511510        } else if (selectedLayers != null) { 
    512511            HashSet<String> proj = new HashSet<String>(); 
    513             for(LayerDetails l : selectedLayers) 
     512            for(LayerDetails l : selectedLayers) { 
    514513                proj.addAll(l.getProjections()); 
     514            } 
    515515            info.setServerProjections(proj); 
    516516        } 
    … …  
    520520    private static String getChildContent(Element parent, String name, String missing, String empty) { 
    521521        Element child = getChild(parent, name); 
    522         if (child == null) { 
     522        if (child == null) 
    523523            return missing; 
    524         } else { 
     524        else { 
    525525            String content = (String) getContent(child); 
    526526            return (content != null) ? content : empty; 
    … …  
    556556 
    557557    private static Element getChild(Element parent, String name) { 
    558         if (parent == null) { 
     558        if (parent == null) 
    559559            return null; 
    560         } 
    561560        for (Node child = parent.getFirstChild(); child != null; child = child.getNextSibling()) { 
    562             if (child instanceof Element && name.equals(child.getNodeName())) { 
     561            if (child instanceof Element && name.equals(child.getNodeName())) 
    563562                return (Element) child; 
    564             } 
    565563        } 
    566564        return null; 
    567565    } 
    568566 
    569     class LayerDetails { 
     567    static class LayerDetails { 
    570568 
    571569        private String name; 
    … …  
    597595        @Override 
    598596        public String toString() { 
    599             if(this.name == null || this.name.isEmpty()) { 
     597            if(this.name == null || this.name.isEmpty()) 
    600598                return this.ident; 
    601             } else { 
     599            else 
    602600                return this.name; 
    603             } 
    604         } 
    605  
    606     } 
    607  
    608     class LayerTreeCellRenderer extends DefaultTreeCellRenderer { 
     601        } 
     602 
     603    } 
     604 
     605    static class LayerTreeCellRenderer extends DefaultTreeCellRenderer { 
    609606        @Override 
    610607        public Component getTreeCellRendererComponent(JTree tree, Object value, 

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

    r4577 r4874  
    282282        p.add(pane,GBC.std().fill(GBC.BOTH)); 
    283283    } 
    284      
     284 
    285285    public ImageryProvidersPanel getProvidersPanel() { 
    286286        return imageryProviders; 
    … …  
    391391        public final ImageryLayerTableModel activeModel; 
    392392        public final ImageryDefaultLayerTableModel defaultModel; 
    393          
     393 
    394394        // Public JToolbars 
    395395        public final JToolBar activeToolbar; 
    … …  
    400400        private final PreferenceTabbedPane gui; 
    401401        private final ImageryLayerInfo layerInfo; 
    402          
    403         private class ImageryTableCellRenderer extends DefaultTableCellRenderer { 
    404              
     402 
     403        private static class ImageryTableCellRenderer extends DefaultTableCellRenderer { 
     404 
    405405            private List<ImageryInfo> layers; 
    406              
     406 
    407407            public ImageryTableCellRenderer(List<ImageryInfo> layers) { 
    408408                this.layers = layers; 
    409409            } 
    410              
     410 
    411411            @Override 
    412412            public Component getTableCellRendererComponent(JTable table, Object value, boolean 
    413             isSelected, boolean hasFocus, int row, int column) { 
     413                    isSelected, boolean hasFocus, int row, int column) { 
    414414                JLabel label = (JLabel) super.getTableCellRendererComponent( 
    415                 table, value, isSelected, hasFocus, row, column); 
     415                        table, value, isSelected, hasFocus, row, column); 
    416416                String t = value.toString(); 
    417417                label.setBackground(Main.pref.getUIColor("Table.background")); 
    … …  
    423423                    if(l.getExtendedUrl().equals(t)) { 
    424424                        label.setBackground(Main.pref.getColor( 
    425                         marktr("Imagery Background: Default"), 
    426                         new Color(200,255,200))); 
     425                                marktr("Imagery Background: Default"), 
     426                                new Color(200,255,200))); 
    427427                        break; 
    428428                    } 
    … …  
    456456 
    457457            defaultModel.addTableModelListener( 
    458                 new TableModelListener() { 
    459                     @Override 
    460                     public void tableChanged(TableModelEvent e) { 
    461                         activeTable.repaint(); 
    462                     } 
    463                 } 
    464             ); 
     458                    new TableModelListener() { 
     459                        @Override 
     460                        public void tableChanged(TableModelEvent e) { 
     461                            activeTable.repaint(); 
     462                        } 
     463                    } 
     464                    ); 
    465465 
    466466            activeModel.addTableModelListener( 
    467                 new TableModelListener() { 
    468                     @Override 
    469                     public void tableChanged(TableModelEvent e) { 
    470                         defaultTable.repaint(); 
    471                     } 
    472                 } 
    473             ); 
     467                    new TableModelListener() { 
     468                        @Override 
     469                        public void tableChanged(TableModelEvent e) { 
     470                            defaultTable.repaint(); 
     471                        } 
     472                    } 
     473                    ); 
    474474 
    475475            TableColumnModel mod = defaultTable.getColumnModel(); 
    … …  
    592592                            mapPolygons.remove(i); 
    593593                        } 
    594                      // Only display bounds when no polygons (shapes) are defined for this provider 
     594                        // Only display bounds when no polygons (shapes) are defined for this provider 
    595595                    } else { 
    596596                        if (defaultTable.getSelectionModel().isSelectedIndex(i)) { 
    … …  
    643643                        activeModel.addRow(p.getImageryInfo()); 
    644644                    } catch (IllegalArgumentException ex) { 
    645                         if (ex.getMessage() == null || ex.getMessage().isEmpty()) { 
     645                        if (ex.getMessage() == null || ex.getMessage().isEmpty()) 
    646646                            throw ex; 
    647                         } else { 
     647                        else { 
    648648                            JOptionPane.showMessageDialog(Main.parent, 
    649649                                    ex.getMessage(), tr("Error"), 
    … …  
    709709                    return; 
    710710                } 
    711                  
     711 
    712712                Set<String> acceptedEulas = new HashSet<String>(); 
    713713 

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

    r4839 r4874  
    3232 
    3333    public static final boolean registerSourceProvider(SourceProvider provider) { 
    34         if (provider != null) { 
     34        if (provider != null) 
    3535            return styleSourceProviders.add(provider); 
    36         } 
    3736        return false; 
    3837    } 
    39      
     38 
    4039    public static class Factory implements PreferenceSettingFactory { 
    4140        public PreferenceSetting createPreferenceSetting() { 
    … …  
    6968                    } 
    7069                } 
    71         ); 
    72     } 
    73  
    74     class MapPaintSourceEditor extends SourceEditor { 
     70                ); 
     71    } 
     72 
     73    static class MapPaintSourceEditor extends SourceEditor { 
    7574 
    7675        final private String iconpref = "mappaint.icon.sources"; 
    … …  
    118117        public String getStr(I18nString ident) { 
    119118            switch (ident) { 
    120                 case AVAILABLE_SOURCES: 
    121                     return tr("Available styles:"); 
    122                 case ACTIVE_SOURCES: 
    123                     return tr("Active styles:"); 
    124                 case NEW_SOURCE_ENTRY_TOOLTIP: 
    125                      return tr("Add a new style by entering filename or URL"); 
    126                 case NEW_SOURCE_ENTRY: 
    127                     return tr("New style entry:"); 
    128                 case REMOVE_SOURCE_TOOLTIP: 
    129                     return tr("Remove the selected styles from the list of active styles"); 
    130                 case EDIT_SOURCE_TOOLTIP: 
    131                     return tr("Edit the filename or URL for the selected active style"); 
    132                 case ACTIVATE_TOOLTIP: 
    133                     return tr("Add the selected available styles to the list of active styles"); 
    134                 case RELOAD_ALL_AVAILABLE: 
    135                     return marktr("Reloads the list of available styles from ''{0}''"); 
    136                 case LOADING_SOURCES_FROM: 
    137                     return marktr("Loading style sources from ''{0}''"); 
    138                 case FAILED_TO_LOAD_SOURCES_FROM: 
    139                     return marktr("<html>Failed to load the list of style sources from<br>" 
    140                             + "''{0}''.<br>" 
    141                             + "<br>" 
    142                             + "Details (untranslated):<br>{1}</html>"); 
    143                 case FAILED_TO_LOAD_SOURCES_FROM_HELP_TOPIC: 
    144                     return "/Preferences/Styles#FailedToLoadStyleSources"; 
    145                 case ILLEGAL_FORMAT_OF_ENTRY: 
    146                     return marktr("Warning: illegal format of entry in style list ''{0}''. Got ''{1}''"); 
    147                 default: throw new AssertionError(); 
     119            case AVAILABLE_SOURCES: 
     120                return tr("Available styles:"); 
     121            case ACTIVE_SOURCES: 
     122                return tr("Active styles:"); 
     123            case NEW_SOURCE_ENTRY_TOOLTIP: 
     124                return tr("Add a new style by entering filename or URL"); 
     125            case NEW_SOURCE_ENTRY: 
     126                return tr("New style entry:"); 
     127            case REMOVE_SOURCE_TOOLTIP: 
     128                return tr("Remove the selected styles from the list of active styles"); 
     129            case EDIT_SOURCE_TOOLTIP: 
     130                return tr("Edit the filename or URL for the selected active style"); 
     131            case ACTIVATE_TOOLTIP: 
     132                return tr("Add the selected available styles to the list of active styles"); 
     133            case RELOAD_ALL_AVAILABLE: 
     134                return marktr("Reloads the list of available styles from ''{0}''"); 
     135            case LOADING_SOURCES_FROM: 
     136                return marktr("Loading style sources from ''{0}''"); 
     137            case FAILED_TO_LOAD_SOURCES_FROM: 
     138                return marktr("<html>Failed to load the list of style sources from<br>" 
     139                        + "''{0}''.<br>" 
     140                        + "<br>" 
     141                        + "Details (untranslated):<br>{1}</html>"); 
     142            case FAILED_TO_LOAD_SOURCES_FROM_HELP_TOPIC: 
     143                return "/Preferences/Styles#FailedToLoadStyleSources"; 
     144            case ILLEGAL_FORMAT_OF_ENTRY: 
     145                return marktr("Warning: illegal format of entry in style list ''{0}''. Got ''{1}''"); 
     146            default: throw new AssertionError(); 
    148147            } 
    149148        } 
    … …  
    177176        public MapPaintPrefMigration() { 
    178177            super("mappaint.style.sources", 
    179                   "mappaint.style.enable-defaults", 
    180                   "mappaint.style.sources-list"); 
     178                    "mappaint.style.enable-defaults", 
     179                    "mappaint.style.sources-list"); 
    181180        } 
    182181 
    … …  
    215214        private boolean insertNewDefaults(List<SourceEntry> list) { 
    216215            boolean changed = false; 
    217              
     216 
    218217            Collection<String> knownDefaults = new TreeSet<String>(Main.pref.getCollection("mappaint.style.known-defaults")); 
    219218 
    … …  
    223222                int i = Utils.indexOf(list, 
    224223                        new Predicate<SourceEntry>() { 
    225                             @Override 
    226                             public boolean evaluate(SourceEntry se) { 
    227                                 return Utils.equal(def.url, se.url); 
    228                             } 
    229                         }); 
     224                    @Override 
     225                    public boolean evaluate(SourceEntry se) { 
     226                        return Utils.equal(def.url, se.url); 
     227                    } 
     228                }); 
    230229                if (i == -1 && !knownDefaults.contains(def.url)) { 
    231230                    list.add(insertionIdx, def); 
    … …  
    268267                    entry.url, 
    269268                    entry.name == null ? "" : entry.name, 
    270                     entry.title == null ? "" : entry.title, 
    271                     Boolean.toString(entry.active) 
     269                            entry.title == null ? "" : entry.title, 
     270                                    Boolean.toString(entry.active) 
    272271            }); 
    273272        } 

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

    r4843 r4874  
    1212import java.util.ArrayList; 
    1313import java.util.Collection; 
    14 import java.util.HashMap; 
    1514import java.util.Iterator; 
    1615import java.util.LinkedList; 
    1716import java.util.List; 
    18 import java.util.Map; 
    1917 
    2018import javax.swing.BorderFactory; 
    … …  
    5856    } 
    5957 
    60     private class TabData { 
     58    private static class TabData { 
    6159        public String icon; 
    6260        public JComponent tab; 
    … …  
    217215                            tr("Warning"), 
    218216                            JOptionPane.WARNING_MESSAGE 
    219                     ); 
     217                            ); 
    220218                } 
    221219                Main.parent.repaint(); 
    … …  
    234232                        } 
    235233                    } 
    236             ); 
     234                    ); 
    237235        } else { 
    238236            // no need for asynchronous activities. Simply run the remaining "save preference" 

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

    r4839 r4874  
    11461146            try { 
    11471147                sources.addAll(getDefault()); 
    1148                  
     1148 
    11491149                for (SourceProvider provider : sourceProviders) { 
    11501150                    for (SourceEntry src : provider.getSources()) { 
    … …  
    11541154                    } 
    11551155                } 
    1156                  
     1156 
    11571157                MirroredInputStream stream = new MirroredInputStream(url); 
    11581158                InputStreamReader r; 
    … …  
    12331233    } 
    12341234 
    1235     class SourceEntryTableCellRenderer extends DefaultTableCellRenderer { 
     1235    static class SourceEntryTableCellRenderer extends DefaultTableCellRenderer { 
    12361236        @Override 
    12371237        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { 

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

    r4839 r4874  
    5050    private SourceEditor sources; 
    5151    private JCheckBox sortMenu; 
    52      
     52 
    5353    public static final boolean registerSourceProvider(SourceProvider provider) { 
    54         if (provider != null) { 
     54        if (provider != null) 
    5555            return presetSourceProviders.add(provider); 
    56         } 
    5756        return false; 
    5857    } 
    … …  
    166165                    } 
    167166                } 
    168         ); 
     167                ); 
    169168        gui.addValidationListener(validationListener); 
    170169    } 
    171170 
    172     class TaggingPresetSourceEditor extends SourceEditor { 
     171    static class TaggingPresetSourceEditor extends SourceEditor { 
    173172 
    174173        final private String iconpref = "taggingpreset.icon.sources"; 
    … …  
    216215        public String getStr(I18nString ident) { 
    217216            switch (ident) { 
    218                 case AVAILABLE_SOURCES: 
    219                     return tr("Available presets:"); 
    220                 case ACTIVE_SOURCES: 
    221                     return tr("Active presets:"); 
    222                 case NEW_SOURCE_ENTRY_TOOLTIP: 
    223                      return tr("Add a new preset by entering filename or URL"); 
    224                 case NEW_SOURCE_ENTRY: 
    225                     return tr("New preset entry:"); 
    226                 case REMOVE_SOURCE_TOOLTIP: 
    227                     return tr("Remove the selected presets from the list of active presets"); 
    228                 case EDIT_SOURCE_TOOLTIP: 
    229                     return tr("Edit the filename or URL for the selected active preset"); 
    230                 case ACTIVATE_TOOLTIP: 
    231                     return tr("Add the selected available presets to the list of active presets"); 
    232                 case RELOAD_ALL_AVAILABLE: 
    233                     return marktr("Reloads the list of available presets from ''{0}''"); 
    234                 case LOADING_SOURCES_FROM: 
    235                     return marktr("Loading preset sources from ''{0}''"); 
    236                 case FAILED_TO_LOAD_SOURCES_FROM: 
    237                     return marktr("<html>Failed to load the list of preset sources from<br>" 
    238                             + "''{0}''.<br>" 
    239                             + "<br>" 
    240                             + "Details (untranslated):<br>{1}</html>"); 
    241                 case FAILED_TO_LOAD_SOURCES_FROM_HELP_TOPIC: 
    242                     return "/Preferences/Presets#FailedToLoadPresetSources"; 
    243                 case ILLEGAL_FORMAT_OF_ENTRY: 
    244                     return marktr("Warning: illegal format of entry in preset list ''{0}''. Got ''{1}''"); 
    245                 default: throw new AssertionError(); 
     217            case AVAILABLE_SOURCES: 
     218                return tr("Available presets:"); 
     219            case ACTIVE_SOURCES: 
     220                return tr("Active presets:"); 
     221            case NEW_SOURCE_ENTRY_TOOLTIP: 
     222                return tr("Add a new preset by entering filename or URL"); 
     223            case NEW_SOURCE_ENTRY: 
     224                return tr("New preset entry:"); 
     225            case REMOVE_SOURCE_TOOLTIP: 
     226                return tr("Remove the selected presets from the list of active presets"); 
     227            case EDIT_SOURCE_TOOLTIP: 
     228                return tr("Edit the filename or URL for the selected active preset"); 
     229            case ACTIVATE_TOOLTIP: 
     230                return tr("Add the selected available presets to the list of active presets"); 
     231            case RELOAD_ALL_AVAILABLE: 
     232                return marktr("Reloads the list of available presets from ''{0}''"); 
     233            case LOADING_SOURCES_FROM: 
     234                return marktr("Loading preset sources from ''{0}''"); 
     235            case FAILED_TO_LOAD_SOURCES_FROM: 
     236                return marktr("<html>Failed to load the list of preset sources from<br>" 
     237                        + "''{0}''.<br>" 
     238                        + "<br>" 
     239                        + "Details (untranslated):<br>{1}</html>"); 
     240            case FAILED_TO_LOAD_SOURCES_FROM_HELP_TOPIC: 
     241                return "/Preferences/Presets#FailedToLoadPresetSources"; 
     242            case ILLEGAL_FORMAT_OF_ENTRY: 
     243                return marktr("Warning: illegal format of entry in preset list ''{0}''. Got ''{1}''"); 
     244            default: throw new AssertionError(); 
    246245            } 
    247246        } 
    … …  
    304303        public PresetPrefMigration() { 
    305304            super("taggingpreset.sources", 
    306                   "taggingpreset.enable-defaults", 
    307                   "taggingpreset.sources-list"); 
     305                    "taggingpreset.enable-defaults", 
     306                    "taggingpreset.sources-list"); 
    308307        } 
    309308 

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

    r4840 r4874  
    1313import java.awt.event.MouseEvent; 
    1414import java.util.ArrayList; 
     15import java.util.Collection; 
     16import java.util.Collections; 
    1517import java.util.List; 
    1618import java.util.Map; 
    1719import java.util.Map.Entry; 
    18 import java.util.Collection; 
    19 import java.util.Collections; 
    2020 
    2121import javax.swing.Box; 
    2222import javax.swing.ButtonGroup; 
     23import javax.swing.DefaultCellEditor; 
    2324import javax.swing.JButton; 
    2425import javax.swing.JLabel; 
    … …  
    3334import javax.swing.table.DefaultTableCellRenderer; 
    3435import javax.swing.table.DefaultTableModel; 
    35 import javax.swing.DefaultCellEditor; 
    3636 
    3737import org.openstreetmap.josm.Main; 
    38 import org.openstreetmap.josm.gui.ExtendedDialog; 
    39 import org.openstreetmap.josm.gui.preferences.PreferenceSetting; 
    40 import org.openstreetmap.josm.gui.preferences.PreferenceSettingFactory; 
    41 import org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane; 
    4238import org.openstreetmap.josm.data.Preferences; 
    4339import org.openstreetmap.josm.data.Preferences.ListListSetting; 
    … …  
    4642import org.openstreetmap.josm.data.Preferences.Setting; 
    4743import org.openstreetmap.josm.data.Preferences.StringSetting; 
     44import org.openstreetmap.josm.gui.ExtendedDialog; 
     45import org.openstreetmap.josm.gui.preferences.PreferenceSetting; 
     46import org.openstreetmap.josm.gui.preferences.PreferenceSettingFactory; 
     47import org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane; 
    4848import org.openstreetmap.josm.tools.CheckParameterUtil; 
    4949import org.openstreetmap.josm.tools.GBC; 
    … …  
    244244    } 
    245245 
    246     private class SettingCellRenderer extends DefaultTableCellRenderer { 
     246    private static class SettingCellRenderer extends DefaultTableCellRenderer { 
    247247        @Override 
    248248        public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { 
    … …  
    264264    } 
    265265 
    266     private class SettingCellEditor extends DefaultCellEditor { 
     266    private static class SettingCellEditor extends DefaultCellEditor { 
    267267        public SettingCellEditor() { 
    268268            super(new JTextField()); 
    269269        } 
    270270 
     271        @Override 
    271272        public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column) { 
    272273            PrefEntry pe = (PrefEntry) value; 
    … …  
    322323                    tr("Warning"), 
    323324                    JOptionPane.WARNING_MESSAGE 
    324             ); 
     325                    ); 
    325326            return; 
    326327        } 
    … …  
    430431                    tr("Warning"), 
    431432                    JOptionPane.WARNING_MESSAGE 
    432             ); 
     433                    ); 
    433434            return; 
    434435        } 

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

    r4759 r4874  
    66import java.io.File; 
    77import java.io.FileInputStream; 
    8 import java.io.FileNotFoundException; 
    98import java.io.IOException; 
    109import java.io.InputStream; 
    … …  
    1211 
    1312import javax.swing.JOptionPane; 
    14 import javax.swing.SwingUtilities; 
    1513 
    1614import org.openstreetmap.josm.Main; 
    … …  
    2422public class GpxImporter extends FileImporter { 
    2523 
    26     protected class GpxImporterData { 
     24    protected static class GpxImporterData { 
    2725        public GpxLayer gpxLayer; 
    2826        public MarkerLayer markerLayer; 
    … …  
    5957 
    6058    public GpxImporterData loadLayers(InputStream is, final File associatedFile, 
    61                     final String gpxLayerName, String markerLayerName, ProgressMonitor progressMonitor) throws IOException { 
     59            final String gpxLayerName, String markerLayerName, ProgressMonitor progressMonitor) throws IOException { 
    6260        final GpxImporterData data = new GpxImporterData(); 
    6361        try { 

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

    r4713 r4874  
    3030    private String source; 
    3131 
    32     private enum State {  
     32    private enum State { 
    3333        INIT,               // initial state, should always be at the bottom of the stack 
    3434        IMAGERY,            // inside the imagery element 
    … …  
    6363    } 
    6464 
    65     private class Parser extends DefaultHandler { 
     65    private static class Parser extends DefaultHandler { 
    6666        private StringBuffer accumulator = new StringBuffer(); 
    6767 
    … …  
    9797            State newState = null; 
    9898            switch (states.peek()) { 
    99                 case INIT: 
    100                     if (qName.equals("imagery")) { 
    101                         newState = State.IMAGERY; 
    102                     } 
    103                     break; 
    104                 case IMAGERY: 
    105                     if (qName.equals("entry")) { 
    106                         entry = new ImageryInfo(); 
    107                         skipEntry = false; 
    108                         newState = State.ENTRY; 
    109                     } 
    110                     break; 
    111                 case ENTRY: 
    112                     if (Arrays.asList(new String[] { 
     99            case INIT: 
     100                if (qName.equals("imagery")) { 
     101                    newState = State.IMAGERY; 
     102                } 
     103                break; 
     104            case IMAGERY: 
     105                if (qName.equals("entry")) { 
     106                    entry = new ImageryInfo(); 
     107                    skipEntry = false; 
     108                    newState = State.ENTRY; 
     109                } 
     110                break; 
     111            case ENTRY: 
     112                if (Arrays.asList(new String[] { 
    113113                        "name", 
    114114                        "type", 
    … …  
    126126                        "country-code", 
    127127                        "icon", 
    128                     }).contains(qName)) { 
    129                         newState = State.ENTRY_ATTRIBUTE; 
    130                     } else if (qName.equals("bounds")) { 
    131                         try { 
    132                             bounds = new ImageryBounds( 
    133                                     atts.getValue("min-lat") + "," + 
    134                                     atts.getValue("min-lon") + "," + 
    135                                     atts.getValue("max-lat") + "," + 
    136                                     atts.getValue("max-lon"), ","); 
    137                         } catch (IllegalArgumentException e) { 
    138                             break; 
    139                         } 
    140                         newState = State.BOUNDS; 
    141                     } else if (qName.equals("projections")) { 
    142                         projections = new ArrayList<String>(); 
    143                         newState = State.PROJECTIONS; 
    144                     } 
    145                     break; 
    146                 case BOUNDS: 
    147                     if (qName.equals("shape")) { 
    148                         shape = new Shape(); 
    149                         newState = State.SHAPE; 
    150                     } 
    151                     break; 
    152                 case SHAPE: 
    153                     if (qName.equals("point")) { 
    154                         try { 
    155                             shape.addPoint(atts.getValue("lat"), atts.getValue("lon")); 
    156                         } catch (IllegalArgumentException e) { 
    157                             break; 
    158                         } 
    159                     } 
    160                     break; 
    161                 case PROJECTIONS: 
    162                     if (qName.equals("code")) { 
    163                         newState = State.CODE; 
    164                     } 
    165                     break; 
     128                }).contains(qName)) { 
     129                    newState = State.ENTRY_ATTRIBUTE; 
     130                } else if (qName.equals("bounds")) { 
     131                    try { 
     132                        bounds = new ImageryBounds( 
     133                                atts.getValue("min-lat") + "," + 
     134                                        atts.getValue("min-lon") + "," + 
     135                                        atts.getValue("max-lat") + "," + 
     136                                        atts.getValue("max-lon"), ","); 
     137                    } catch (IllegalArgumentException e) { 
     138                        break; 
     139                    } 
     140                    newState = State.BOUNDS; 
     141                } else if (qName.equals("projections")) { 
     142                    projections = new ArrayList<String>(); 
     143                    newState = State.PROJECTIONS; 
     144                } 
     145                break; 
     146            case BOUNDS: 
     147                if (qName.equals("shape")) { 
     148                    shape = new Shape(); 
     149                    newState = State.SHAPE; 
     150                } 
     151                break; 
     152            case SHAPE: 
     153                if (qName.equals("point")) { 
     154                    try { 
     155                        shape.addPoint(atts.getValue("lat"), atts.getValue("lon")); 
     156                    } catch (IllegalArgumentException e) { 
     157                        break; 
     158                    } 
     159                } 
     160                break; 
     161            case PROJECTIONS: 
     162                if (qName.equals("code")) { 
     163                    newState = State.CODE; 
     164                } 
     165                break; 
    166166            } 
    167167            /** 
    … …  
    189189        public void endElement(String namespaceURI, String qName, String rqName) { 
    190190            switch (states.pop()) { 
    191                 case INIT: 
    192                     throw new RuntimeException("parsing error: more closing than opening elements"); 
    193                 case ENTRY: 
    194                     if (qName.equals("entry")) { 
    195                         if (!skipEntry) { 
    196                             entries.add(entry); 
     191            case INIT: 
     192                throw new RuntimeException("parsing error: more closing than opening elements"); 
     193            case ENTRY: 
     194                if (qName.equals("entry")) { 
     195                    if (!skipEntry) { 
     196                        entries.add(entry); 
     197                    } 
     198                    entry = null; 
     199                } 
     200                break; 
     201            case ENTRY_ATTRIBUTE: 
     202                if (qName.equals("name")) { 
     203                    entry.setName(tr(accumulator.toString())); 
     204                } else if (qName.equals("type")) { 
     205                    boolean found = false; 
     206                    for (ImageryType type : ImageryType.values()) { 
     207                        if (equal(accumulator.toString(), type.getUrlString())) { 
     208                            entry.setImageryType(type); 
     209                            found = true; 
     210                            break; 
    197211                        } 
    198                         entry = null; 
    199                     } 
    200                     break; 
    201                 case ENTRY_ATTRIBUTE: 
    202                     if (qName.equals("name")) { 
    203                         entry.setName(tr(accumulator.toString())); 
    204                     } else if (qName.equals("type")) { 
    205                         boolean found = false; 
    206                         for (ImageryType type : ImageryType.values()) { 
    207                             if (equal(accumulator.toString(), type.getUrlString())) { 
    208                                 entry.setImageryType(type); 
    209                                 found = true; 
    210                                 break; 
    211                             } 
     212                    } 
     213                    if (!found) { 
     214                        skipEntry = true; 
     215                    } 
     216                } else if (qName.equals("default")) { 
     217                    if (accumulator.toString().equals("true")) { 
     218                        entry.setDefaultEntry(true); 
     219                    } else if (accumulator.toString().equals("false")) { 
     220                        entry.setDefaultEntry(false); 
     221                    } else { 
     222                        skipEntry = true; 
     223                    } 
     224                } else if (qName.equals("url")) { 
     225                    entry.setUrl(accumulator.toString()); 
     226                } else if (qName.equals("eula")) { 
     227                    entry.setEulaAcceptanceRequired(accumulator.toString()); 
     228                } else if (qName.equals("min-zoom") || qName.equals("max-zoom")) { 
     229                    Integer val = null; 
     230                    try { 
     231                        val = Integer.parseInt(accumulator.toString()); 
     232                    } catch(NumberFormatException e) { 
     233                        val = null; 
     234                    } 
     235                    if (val == null) { 
     236                        skipEntry = true; 
     237                    } else { 
     238                        if (qName.equals("min-zoom")) { 
     239                            entry.setDefaultMinZoom(val); 
     240                        } else { 
     241                            entry.setDefaultMaxZoom(val); 
    212242                        } 
    213                         if (!found) { 
    214                             skipEntry = true; 
    215                         } 
    216                     } else if (qName.equals("default")) { 
    217                         if (accumulator.toString().equals("true")) { 
    218                             entry.setDefaultEntry(true); 
    219                         } else if (accumulator.toString().equals("false")) { 
    220                             entry.setDefaultEntry(false); 
    221                         } else { 
    222                             skipEntry = true; 
    223                         } 
    224                     } else if (qName.equals("url")) { 
    225                         entry.setUrl(accumulator.toString()); 
    226                     } else if (qName.equals("eula")) { 
    227                         entry.setEulaAcceptanceRequired(accumulator.toString()); 
    228                     } else if (qName.equals("min-zoom") || qName.equals("max-zoom")) { 
    229                         Integer val = null; 
    230                         try { 
    231                             val = Integer.parseInt(accumulator.toString()); 
    232                         } catch(NumberFormatException e) { 
    233                             val = null; 
    234                         } 
    235                         if (val == null) { 
    236                             skipEntry = true; 
    237                         } else { 
    238                             if (qName.equals("min-zoom")) { 
    239                                 entry.setDefaultMinZoom(val); 
    240                             } else { 
    241                                 entry.setDefaultMaxZoom(val); 
    242                             } 
    243                         } 
    244                     } else if (qName.equals("attribution-text")) { 
    245                         entry.setAttributionText(accumulator.toString()); 
    246                     } else if (qName.equals("attribution-url")) { 
    247                         entry.setAttributionLinkURL(accumulator.toString()); 
    248                     } else if (qName.equals("logo-image")) { 
    249                         entry.setAttributionImage(accumulator.toString()); 
    250                     } else if (qName.equals("logo-url")) { 
    251                         entry.setAttributionImageURL(accumulator.toString()); 
    252                     } else if (qName.equals("terms-of-use-text")) { 
    253                         entry.setTermsOfUseText(accumulator.toString()); 
    254                     } else if (qName.equals("terms-of-use-url")) { 
    255                         entry.setTermsOfUseURL(accumulator.toString()); 
    256                     } else if (qName.equals("country-code")) { 
    257                         entry.setCountryCode(accumulator.toString()); 
    258                     } else if (qName.equals("icon")) { 
    259                         entry.setIcon(accumulator.toString()); 
    260                     } else { 
    261                     } 
    262                     break; 
    263                 case BOUNDS: 
    264                     entry.setBounds(bounds); 
    265                     bounds = null; 
    266                     break; 
    267                 case SHAPE: 
    268                     bounds.addShape(shape); 
    269                     shape = null; 
    270                     break; 
    271                 case CODE: 
    272                     projections.add(accumulator.toString()); 
    273                     break; 
    274                 case PROJECTIONS: 
    275                     entry.setServerProjections(projections); 
    276                     projections = null; 
    277                     break; 
     243                    } 
     244                } else if (qName.equals("attribution-text")) { 
     245                    entry.setAttributionText(accumulator.toString()); 
     246                } else if (qName.equals("attribution-url")) { 
     247                    entry.setAttributionLinkURL(accumulator.toString()); 
     248                } else if (qName.equals("logo-image")) { 
     249                    entry.setAttributionImage(accumulator.toString()); 
     250                } else if (qName.equals("logo-url")) { 
     251                    entry.setAttributionImageURL(accumulator.toString()); 
     252                } else if (qName.equals("terms-of-use-text")) { 
     253                    entry.setTermsOfUseText(accumulator.toString()); 
     254                } else if (qName.equals("terms-of-use-url")) { 
     255                    entry.setTermsOfUseURL(accumulator.toString()); 
     256                } else if (qName.equals("country-code")) { 
     257                    entry.setCountryCode(accumulator.toString()); 
     258                } else if (qName.equals("icon")) { 
     259                    entry.setIcon(accumulator.toString()); 
     260                } else { 
     261                } 
     262                break; 
     263            case BOUNDS: 
     264                entry.setBounds(bounds); 
     265                bounds = null; 
     266                break; 
     267            case SHAPE: 
     268                bounds.addShape(shape); 
     269                shape = null; 
     270                break; 
     271            case CODE: 
     272                projections.add(accumulator.toString()); 
     273                break; 
     274            case PROJECTIONS: 
     275                entry.setServerProjections(projections); 
     276                projections = null; 
     277                break; 
    278278            } 
    279279        } 

  • trunk/src/org/openstreetmap/josm/io/session/SessionReader.java

    r4868 r4874  
    407407     * needed to block the current thread and wait for the result of the modal dialog from EDT. 
    408408     */ 
    409     private class CancelOrContinueDialog { 
     409    private static class CancelOrContinueDialog { 
    410410 
    411411        private boolean cancel; 
Note: See TracChangeset for help on using the changeset viewer.