Changeset 6248 in josm


Ignore:
Timestamp:
2013-09-23T16:47:50+02:00 (11 years ago)
Author:
Don-vip
Message:

Rework console output:

  • new log level "error"
  • Replace nearly all calls to system.out and system.err to Main.(error|warn|info|debug)
  • Remove some unnecessary debug output
  • Some messages are modified (removal of "Info", "Warning", "Error" from the message itself -> notable i18n impact but limited to console error messages not seen by the majority of users, so that's ok)
Location:
trunk/src/org/openstreetmap/josm
Files:
137 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r6143 r6248  
    4141import javax.swing.KeyStroke;
    4242import javax.swing.UIManager;
     43import javax.swing.UnsupportedLookAndFeelException;
    4344
    4445import org.openstreetmap.gui.jmapviewer.FeatureAdapter;
    … …  
    199200
    200201    /**
    201      * Logging level (3 = debug, 2 = info, 1 = warn, 0 = none).
    202      */
    203     static public int log_level = 2;
    204     /**
    205      * Print a warning message if logging is on.
     202     * Logging level (4 = debug, 3 = info, 2 = warn, 1 = error, 0 = none).
     203     * @since 6248
     204     */
     205    public static int logLevel = 3;
     206   
     207    /**
     208     * Prints an error message if logging is on.
    206209     * @param msg The message to print.
    207      */
    208     static public void warn(String msg) {
    209         if (log_level < 1)
     210     * @since 6248
     211     */
     212    public static void error(String msg) {
     213        if (logLevel < 1)
     214            return;
     215        System.err.println(tr("ERROR: {0}", msg));
     216    }
     217   
     218    /**
     219     * Prints a warning message if logging is on.
     220     * @param msg The message to print.
     221     */
     222    public static void warn(String msg) {
     223        if (logLevel < 2)
    210224            return;
    211225        System.err.println(tr("WARNING: {0}", msg));
    212226    }
    213     /**
    214      * Print an informational message if logging is on.
     227   
     228    /**
     229     * Prints an informational message if logging is on.
    215230     * @param msg The message to print.
    216231     */
    217     static public void info(String msg) {
    218         if (log_level < 2)
     232    public static void info(String msg) {
     233        if (logLevel < 3)
    219234            return;
    220         System.err.println(tr("INFO: {0}", msg));
    221     }
    222     /**
    223      * Print an debug message if logging is on.
     235        System.out.println(tr("INFO: {0}", msg));
     236    }
     237   
     238    /**
     239     * Prints a debug message if logging is on.
    224240     * @param msg The message to print.
    225241     */
    226     static public void debug(String msg) {
    227         if (log_level < 3)
     242    public static void debug(String msg) {
     243        if (logLevel < 4)
    228244            return;
    229         System.err.println(tr("DEBUG: {0}", msg));
    230     }
    231     /**
    232      * Print a formated warning message if logging is on. Calls {@link MessageFormat#format}
     245        System.out.println(tr("DEBUG: {0}", msg));
     246    }
     247   
     248    /**
     249     * Prints a formated error message if logging is on. Calls {@link MessageFormat#format}
    233250     * function to format text.
    234251     * @param msg The formated message to print.
    235252     * @param objects The objects to insert into format string.
    236      */
    237     static public void warn(String msg, Object... objects) {
    238         warn(MessageFormat.format(msg, objects));
    239     }
    240     /**
    241      * Print a formated informational message if logging is on. Calls {@link MessageFormat#format}
     253     * @since 6248
     254     */
     255    public static void error(String msg, Object... objects) {
     256        error(MessageFormat.format(msg, objects));
     257    }
     258   
     259    /**
     260     * Prints a formated warning message if logging is on. Calls {@link MessageFormat#format}
    242261     * function to format text.
    243262     * @param msg The formated message to print.
    244263     * @param objects The objects to insert into format string.
    245264     */
    246     static public void info(String msg, Object... objects) {
    247         info(MessageFormat.format(msg, objects));
    248     }
    249     /**
    250      * Print a formated debug message if logging is on. Calls {@link MessageFormat#format}
     265    public static void warn(String msg, Object... objects) {
     266        warn(MessageFormat.format(msg, objects));
     267    }
     268   
     269    /**
     270     * Prints a formated informational message if logging is on. Calls {@link MessageFormat#format}
    251271     * function to format text.
    252272     * @param msg The formated message to print.
    253273     * @param objects The objects to insert into format string.
    254274     */
    255     static public void debug(String msg, Object... objects) {
     275    public static void info(String msg, Object... objects) {
     276        info(MessageFormat.format(msg, objects));
     277    }
     278   
     279    /**
     280     * Prints a formated debug message if logging is on. Calls {@link MessageFormat#format}
     281     * function to format text.
     282     * @param msg The formated message to print.
     283     * @param objects The objects to insert into format string.
     284     */
     285    public static void debug(String msg, Object... objects) {
    256286        debug(MessageFormat.format(msg, objects));
     287    }
     288   
     289    /**
     290     * Prints an error message for the given Throwable.
     291     * @param t The throwable object causing the error
     292     * @since 6248
     293     */
     294    public static void error(Throwable t) {
     295        error(t.getClass().getName()+": "+t.getMessage());
     296    }
     297   
     298    /**
     299     * Prints a warning message for the given Throwable.
     300     * @param t The throwable object causing the error
     301     * @since 6248
     302     */
     303    public static void warn(Throwable t) {
     304        warn(t.getClass().getName()+": "+t.getMessage());
    257305    }
    258306
    … …  
    531579        Object existing = inputMap.get(keyStroke);
    532580        if (existing != null && !existing.equals(action)) {
    533             System.out.println(String.format("Keystroke %s is already assigned to %s, will be overridden by %s", keyStroke, existing, action));
     581            info(String.format("Keystroke %s is already assigned to %s, will be overridden by %s", keyStroke, existing, action));
    534582        }
    535583        inputMap.put(keyStroke, action);
    … …  
    600648                UIManager.setLookAndFeel(laf);
    601649            }
    602             catch (final java.lang.ClassNotFoundException e) {
    603                 System.out.println("Look and Feel not found: " + laf);
     650            catch (final ClassNotFoundException e) {
     651                info("Look and Feel not found: " + laf);
    604652                Main.pref.put("laf", defaultlaf);
    605653            }
    606             catch (final javax.swing.UnsupportedLookAndFeelException e) {
    607                 System.out.println("Look and Feel not supported: " + laf);
     654            catch (final UnsupportedLookAndFeelException e) {
     655                info("Look and Feel not supported: " + laf);
    608656                Main.pref.put("laf", defaultlaf);
    609657            }
    … …  
    853901        String os = System.getProperty("os.name");
    854902        if (os == null) {
    855             System.err.println("Your operating system has no name, so I'm guessing its some kind of *nix.");
     903            warn("Your operating system has no name, so I'm guessing its some kind of *nix.");
    856904            platform = new PlatformHookUnixoid();
    857905        } else if (os.toLowerCase().startsWith("windows")) {
    … …  
    864912            platform = new PlatformHookOsx();
    865913        } else {
    866             System.err.println("I don't know your operating system '"+os+"', so I'm guessing its some kind of *nix.");
     914            warn("I don't know your operating system '"+os+"', so I'm guessing its some kind of *nix.");
    867915            platform = new PlatformHookUnixoid();
    868916        }
    … …  
    948996            }
    949997        }
    950         System.err.println("Error: Could not recognize Java Version: "+version);
     998        error("Could not recognize Java Version: "+version);
    951999    }
    9521000

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

    r6084 r6248  
    4343        String ret =  pattern.matcher(baseUrl).replaceAll("/browse");
    4444        if (ret.equals(baseUrl)) {
    45             System.out.println(tr("WARNING: unexpected format of API base URL. Redirection to info or history page for OSM object will probably fail. API base URL is: ''{0}''",baseUrl));
     45            Main.warn(tr("Unexpected format of API base URL. Redirection to info or history page for OSM object will probably fail. API base URL is: ''{0}''",baseUrl));
    4646        }
    4747        if (ret.startsWith("http://api.openstreetmap.org/")) {
    … …  
    6262        String ret =  pattern.matcher(baseUrl).replaceAll("/user");
    6363        if (ret.equals(baseUrl)) {
    64             System.out.println(tr("WARNING: unexpected format of API base URL. Redirection to user page for OSM user will probably fail. API base URL is: ''{0}''",baseUrl));
     64            Main.warn(tr("Unexpected format of API base URL. Redirection to user page for OSM user will probably fail. API base URL is: ''{0}''",baseUrl));
    6565        }
    6666        if (ret.startsWith("http://api.openstreetmap.org/")) {

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

    r6069 r6248  
    99import java.io.IOException;
    1010import java.net.MalformedURLException;
     11
    1112import javax.swing.Action;
    1213import javax.swing.ImageIcon;
    … …  
    1516import javax.swing.JPanel;
    1617import javax.swing.JScrollPane;
     18
    1719import org.openstreetmap.josm.Main;
    1820import org.openstreetmap.josm.data.imagery.ImageryInfo;
    … …  
    2123import org.openstreetmap.josm.gui.actionsupport.AlignImageryPanel;
    2224import org.openstreetmap.josm.gui.layer.ImageryLayer;
     25import org.openstreetmap.josm.gui.preferences.imagery.WMSLayerTree;
    2326import org.openstreetmap.josm.io.imagery.WMSImagery;
    24 import org.openstreetmap.josm.gui.preferences.imagery.WMSLayerTree;
    2527import org.openstreetmap.josm.tools.GBC;
    2628import org.openstreetmap.josm.tools.ImageProvider;
    … …  
    7880            wms.attemptGetCapabilities(info.getUrl());
    7981
    80             System.out.println(wms.getLayers());
    8182            final WMSLayerTree tree = new WMSLayerTree();
    8283            tree.updateTree(wms);
    … …  
    108109            JOptionPane.showMessageDialog(Main.parent, tr("Could not parse WMS layer list."),
    109110                    tr("WMS Error"), JOptionPane.ERROR_MESSAGE);
    110             System.err.println("Could not parse WMS layer list. Incoming data:");
    111             System.err.println(ex.getIncomingData());
     111            Main.error("Could not parse WMS layer list. Incoming data:\n"+ex.getIncomingData());
    112112        }
    113113        return null;

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

    r6084 r6248  
    303303                        Utils.close(reader);
    304304                    } catch (Exception e) {
    305                         System.err.println(e.getMessage());
     305                        Main.error(e);
    306306                    }
    307307                }

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

    r6131 r6248  
    3333import org.openstreetmap.josm.tools.Shortcut;
    3434
     35/**
     36 * Delete unnecessary nodes from a way
     37 */
    3538public class SimplifyWayAction extends JosmAction {
     39   
     40    /**
     41     * Constructs a new {@code SimplifyWayAction}.
     42     */
    3643    public SimplifyWayAction() {
    3744        super(tr("Simplify Way"), "simplify", tr("Delete unnecessary nodes from a way."), Shortcut.registerShortcut("tools:simplify", tr("Tool: {0}", tr("Simplify Way")),
    … …  
    4148
    4249    protected boolean confirmWayWithNodesOutsideBoundingBox(List<? extends OsmPrimitive> primitives) {
    43         System.out.println(primitives);
    4450        return DeleteCommand.checkAndConfirmOutlyingDelete(Main.map.mapView.getEditLayer(), primitives, null);
    4551    }

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

    r6031 r6248  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.actions.downloadtasks;
     3
     4import static org.openstreetmap.josm.tools.I18n.tr;
    35
    46import java.util.Date;
    … …  
    810import java.util.Map;
    911import java.util.concurrent.Future;
    10 
    11 import static org.openstreetmap.josm.tools.I18n.tr;
    1212
    1313import org.openstreetmap.josm.Main;
    … …  
    184184                            data.setVisible(hp.isVisible());
    185185                        } catch (IllegalStateException e) {
    186                             System.err.println("Cannot change visibility for "+p+": "+e.getMessage());
     186                            Main.error("Cannot change visibility for "+p+": "+e.getMessage());
    187187                        }
    188188                        data.setTimestamp(hp.getTimestamp());
    … …  
    196196                            it.remove();
    197197                        } catch (AssertionError e) {
    198                             System.err.println("Cannot load "+p + ": " + e.getMessage());
     198                            Main.error("Cannot load "+p + ": " + e.getMessage());
    199199                        }
    200200                    }

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

    r6069 r6248  
    206206            } catch(Exception e) {
    207207                if (isCanceled()) {
    208                     System.out.println(tr("Ignoring exception because download has been canceled. Exception was: {0}", e.toString()));
     208                    Main.info(tr("Ignoring exception because download has been canceled. Exception was: {0}", e.toString()));
    209209                    return;
    210210                }

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DeleteAction.java

    r6084 r6248  
    116116            Toolkit.getDefaultToolkit().addAWTEventListener(this, AWTEvent.KEY_EVENT_MASK);
    117117        } catch (SecurityException ex) {
    118             System.out.println(ex);
     118            Main.warn(ex);
    119119        }
    120120    }
    … …  
    127127            Toolkit.getDefaultToolkit().removeAWTEventListener(this);
    128128        } catch (SecurityException ex) {
    129             System.out.println(ex);
     129            Main.warn(ex);
    130130        }
    131131        removeHighlighting();

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r6246 r6248  
    13651365                    snapAngles[i] = 360-Double.parseDouble(s); i++;
    13661366                } catch (NumberFormatException e) {
    1367                     System.err.println("Warning: incorrect number in draw.anglesnap.angles preferences: "+s);
     1367                    Main.warn("Incorrect number in draw.anglesnap.angles preferences: "+s);
    13681368                    snapAngles[i]=0;i++;
    13691369                    snapAngles[i]=0;i++;

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r6106 r6248  
    683683                    break;
    684684                } else {
    685                     System.out.println("Unknown char in SearchSettings: " + s);
     685                    Main.warn("Unknown char in SearchSettings: " + s);
    686686                    break;
    687687                }

  • trunk/src/org/openstreetmap/josm/actions/upload/ApiPreconditionCheckerHook.java

    r6084 r6248  
    5353            for (String key: osmPrimitive.keySet()) {
    5454                String value = osmPrimitive.get(key);
    55                 if(key.length() > 255) {
     55                if (key.length() > 255) {
    5656                    if (osmPrimitive.isDeleted()) {
    57                         // if OsmPrimitive is going to be deleted we automatically shorten the
    58                         // value
    59                         System.out.println(
    60                                 tr("Warning: automatically truncating value of tag ''{0}'' on deleted object {1}",
     57                        // if OsmPrimitive is going to be deleted we automatically shorten the value
     58                        Main.warn(
     59                                tr("Automatically truncating value of tag ''{0}'' on deleted object {1}",
    6160                                        key,
    6261                                        Long.toString(osmPrimitive.getId())

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

    r5077 r6248  
    4949    @Override public void undoCommand() {
    5050        if (! Main.map.mapView.hasLayer(getLayer())) {
    51             System.out.println(tr("Warning: Layer ''{0}'' does not exist any more. Cannot remove conflict for object ''{1}''.",
     51            Main.warn(tr("Layer ''{0}'' does not exist any more. Cannot remove conflict for object ''{1}''.",
    5252                    getLayer().getName(),
    5353                    conflict.getMy().getDisplayName(DefaultNameFormatter.getInstance())

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

    r5816 r6248  
    6161
    6262        if (! Main.map.mapView.hasLayer(getLayer())) {
    63             System.out.println(tr("Cannot undo command ''{0}'' because layer ''{1}'' is not present any more",
     63            Main.warn(tr("Cannot undo command ''{0}'' because layer ''{1}'' is not present any more",
    6464                    this.toString(),
    6565                    getLayer().toString()

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

    r5881 r6248  
    8181    public void undoCommand() {
    8282        if (! Main.map.mapView.hasLayer(layer)) {
    83             System.out.println(tr("Cannot undo command ''{0}'' because layer ''{1}'' is not present any more",
     83            Main.warn(tr("Cannot undo command ''{0}'' because layer ''{1}'' is not present any more",
    8484                    this.toString(),
    8585                    layer.toString()

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

    r5903 r6248  
    99import javax.swing.Icon;
    1010
     11import org.openstreetmap.josm.Main;
    1112import org.openstreetmap.josm.data.conflict.Conflict;
    1213import org.openstreetmap.josm.data.osm.Node;
    … …  
    5455        super.executeCommand();
    5556
    56         // replace the list of nodes of 'my' way by the list of merged
    57         // nodes
     57        // replace the list of nodes of 'my' way by the list of merged nodes
    5858        //
    5959        for (Node n:mergedNodeList) {
    6060            if (! getLayer().data.getNodes().contains(n)) {
    61                 System.out.println(tr("Main dataset does not include node {0}", n.toString()));
     61                Main.warn(tr("Main dataset does not include node {0}", n.toString()));
    6262            }
    6363        }

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

    r6104 r6248  
    8989
    9090            if (!autosaveDir.exists() && !autosaveDir.mkdirs()) {
    91                 System.out.println(tr("Unable to create directory {0}, autosave will be disabled", autosaveDir.getAbsolutePath()));
     91                Main.warn(tr("Unable to create directory {0}, autosave will be disabled", autosaveDir.getAbsolutePath()));
    9292                return;
    9393            }
    9494            if (!deletedLayersDir.exists() && !deletedLayersDir.mkdirs()) {
    95                 System.out.println(tr("Unable to create directory {0}, autosave will be disabled", deletedLayersDir.getAbsolutePath()));
     95                Main.warn(tr("Unable to create directory {0}, autosave will be disabled", deletedLayersDir.getAbsolutePath()));
    9696                return;
    9797            }
    … …  
    159159                        Utils.close(ps);
    160160                    } catch (Throwable t) {
    161                         System.err.println(t.getMessage());
     161                        Main.error(t);
    162162                    }
    163163                    return result;
    164164                } else {
    165                     System.out.println(tr("Unable to create file {0}, other filename will be used", result.getAbsolutePath()));
     165                    Main.warn(tr("Unable to create file {0}, other filename will be used", result.getAbsolutePath()));
    166166                    if (index > PROP_INDEX_LIMIT.get())
    167167                        throw new IOException("index limit exceeded");
    168168                }
    169169            } catch (IOException e) {
    170                 System.err.println(tr("IOError while creating file, autosave will be skipped: {0}", e.getMessage()));
     170                Main.error(tr("IOError while creating file, autosave will be skipped: {0}", e.getMessage()));
    171171                return null;
    172172            }
    … …  
    190190            File oldFile = info.backupFiles.remove();
    191191            if (!oldFile.delete()) {
    192                 System.out.println(tr("Unable to delete old backup file {0}", oldFile.getAbsolutePath()));
     192                Main.warn(tr("Unable to delete old backup file {0}", oldFile.getAbsolutePath()));
    193193            } else {
    194194                getPidFile(oldFile).delete();
    … …  
    207207            } catch (Throwable t) {
    208208                // Don't let exception stop time thread
    209                 System.err.println("Autosave failed: ");
     209                Main.error("Autosave failed:");
     210                Main.error(t);
    210211                t.printStackTrace();
    211212            }
    … …  
    257258                            }
    258259                        } catch (IOException e) {
    259                             System.err.println(tr("Error while creating backup of removed layer: {0}", e.getMessage()));
     260                            Main.error(tr("Error while creating backup of removed layer: {0}", e.getMessage()));
    260261                        }
    261262
    … …  
    300301                            }
    301302                        } catch (Throwable t) {
    302                             System.err.println(t.getClass()+":"+t.getMessage());
     303                            Main.error(t);
    303304                        } finally {
    304305                            Utils.close(reader);
    305306                        }
    306307                    } catch (Throwable t) {
    307                         System.err.println(t.getClass()+":"+t.getMessage());
     308                        Main.error(t);
    308309                    }
    309310                }
    … …  
    359360            deletedLayers.remove(backupFile);
    360361            if (!backupFile.delete()) {
    361                 System.err.println(String.format("Warning: Could not delete old backup file %s", backupFile));
     362                Main.warn(String.format("Could not delete old backup file %s", backupFile));
    362363            }
    363364        }
    … …  
    366367            pidFile.delete();
    367368        } else {
    368             System.err.println(String.format("Warning: Could not move autosaved file %s to %s folder", f.getName(), deletedLayersDir.getName()));
     369            Main.warn(String.format("Could not move autosaved file %s to %s folder", f.getName(), deletedLayersDir.getName()));
    369370            // we cannot move to deleted folder, so just try to delete it directly
    370371            if (!f.delete()) {
    371                 System.err.println(String.format("Warning: Could not delete backup file %s", f));
     372                Main.warn(String.format("Could not delete backup file %s", f));
    372373            } else {
    373374                pidFile.delete();
    … …  
    380381            }
    381382            if (!next.delete()) {
    382                 System.err.println(String.format("Warning: Could not delete archived backup file %s", next));
     383                Main.warn(String.format("Could not delete archived backup file %s", next));
    383384            }
    384385        }

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

    r6235 r6248  
    236236            root = document.getDocumentElement();
    237237        } catch (Exception ex) {
    238             System.out.println("Error getting preferences to save:" +ex.getMessage());
     238            Main.warn("Error getting preferences to save:" +ex.getMessage());
    239239        }
    240240        if (root==null) return;
    … …  
    265265            ts.transform(new DOMSource(exportDocument), new StreamResult(f.toURI().getPath()));
    266266        } catch (Exception ex) {
    267             System.out.println("Error saving preferences part: " +ex.getMessage());
     267            Main.warn("Error saving preferences part: " +ex.getMessage());
    268268            ex.printStackTrace();
    269269        }
    … …  
    345345                        List<PluginInformation> toDeletePlugins = new ArrayList<PluginInformation>();
    346346                        for (PluginInformation pi: availablePlugins) {
    347                             //System.out.print(pi.name+";");
    348347                            String name = pi.name.toLowerCase();
    349348                            if (installList.contains(name)) toInstallPlugins.add(pi);
    … …  
    355354                            Main.worker.submit(pluginDownloadTask);
    356355                        }
    357                             Collection<String> pls = new ArrayList<String>(Main.pref.getCollection("plugins"));
    358                             for (PluginInformation pi: toInstallPlugins) {
    359                                 if (!pls.contains(pi.name)) pls.add(pi.name);
     356                        Collection<String> pls = new ArrayList<String>(Main.pref.getCollection("plugins"));
     357                        for (PluginInformation pi: toInstallPlugins) {
     358                            if (!pls.contains(pi.name)) {
     359                                pls.add(pi.name);
    360360                            }
    361                             for (PluginInformation pi: toRemovePlugins) {
    362                                 pls.remove(pi.name);
    363                             }
    364                             for (PluginInformation pi: toDeletePlugins) {
    365                                 pls.remove(pi.name);
    366                                 new File(Main.pref.getPluginsDirectory(),pi.name+".jar").deleteOnExit();
    367                             }
    368                             System.out.println(pls);
    369                             Main.pref.putCollection("plugins",pls);
    370361                        }
     362                        for (PluginInformation pi: toRemovePlugins) {
     363                            pls.remove(pi.name);
     364                        }
     365                        for (PluginInformation pi: toDeletePlugins) {
     366                            pls.remove(pi.name);
     367                            new File(Main.pref.getPluginsDirectory(), pi.name+".jar").deleteOnExit();
     368                        }
     369                        Main.pref.putCollection("plugins",pls);
     370                    }
    371371                });
    372372            }
    … …  
    949949    }
    950950
    951 
    952 
    953951    private static void defaultUnknownWarning(String key) {
    954952        log("Warning: Unknown default value of %s , skipped\n", key);
    … …  
    961959
    962960    private static void showPrefs(Preferences tmpPref) {
    963         System.out.println("properties: " + tmpPref.properties);
    964         System.out.println("collections: " + tmpPref.collectionProperties);
    965         System.out.println("arrays: " + tmpPref.arrayProperties);
    966         System.out.println("maps: " + tmpPref.listOfStructsProperties);
     961        Main.info("properties: " + tmpPref.properties);
     962        Main.info("collections: " + tmpPref.collectionProperties);
     963        Main.info("arrays: " + tmpPref.arrayProperties);
     964        Main.info("maps: " + tmpPref.listOfStructsProperties);
    967965    }
    968966
    … …  
    973971    }
    974972
    975 
    976      /**
     973    /**
    977974     * Convert JavaScript preferences object to preferences data structures
    978975     * @param engine - JS engine to put object
    … …  
    10591056            tmpPref.listOfStructsProperties.put(e.getKey(), e.getValue());
    10601057        }
    1061 
    1062     }
    1063 
     1058    }
    10641059
    10651060    /**
    … …  
    11531148            "}\n";
    11541149
    1155         //System.out.println("map1: "+stringMap );
    1156         //System.out.println("lists1: "+listMap );
    1157         //System.out.println("listlist1: "+listlistMap );
    1158         //System.out.println("listmap1: "+listmapMap );
    1159 
    11601150        // Execute conversion script
    11611151        engine.eval(init);
    1162 
    11631152    }
    11641153    }

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

    r6235 r6248  
    373373        }
    374374        if (!cacheDirFile.exists() && !cacheDirFile.mkdirs()) {
    375             System.err.println(tr("Warning: Failed to create missing cache directory: {0}", cacheDirFile.getAbsoluteFile()));
     375            Main.warn(tr("Failed to create missing cache directory: {0}", cacheDirFile.getAbsoluteFile()));
    376376            JOptionPane.showMessageDialog(
    377377                    Main.parent,
    … …  
    499499            defaults.put(key, def);
    500500        } else if(def != null && !defaults.get(key).equals(def)) {
    501             System.out.println("Defaults for " + key + " differ: " + def + " != " + defaults.get(key));
     501            Main.info("Defaults for " + key + " differ: " + def + " != " + defaults.get(key));
    502502        }
    503503    }
    … …  
    556556                try {
    557557                    save();
    558                 } catch(IOException e){
    559                     System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
     558                } catch (IOException e) {
     559                    Main.warn(tr("Failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
    560560                }
    561561                changed = true;
    … …  
    652652        if (prefDir.exists()) {
    653653            if(!prefDir.isDirectory()) {
    654                 System.err.println(tr("Warning: Failed to initialize preferences. Preference directory ''{0}'' is not a directory.", prefDir.getAbsoluteFile()));
     654                Main.warn(tr("Failed to initialize preferences. Preference directory ''{0}'' is not a directory.", prefDir.getAbsoluteFile()));
    655655                JOptionPane.showMessageDialog(
    656656                        Main.parent,
    … …  
    663663        } else {
    664664            if (! prefDir.mkdirs()) {
    665                 System.err.println(tr("Warning: Failed to initialize preferences. Failed to create missing preference directory: {0}", prefDir.getAbsoluteFile()));
     665                Main.warn(tr("Failed to initialize preferences. Failed to create missing preference directory: {0}", prefDir.getAbsoluteFile()));
    666666                JOptionPane.showMessageDialog(
    667667                        Main.parent,
    … …  
    677677        try {
    678678            if (!preferenceFile.exists()) {
    679                 System.out.println(tr("Info: Missing preference file ''{0}''. Creating a default preference file.", preferenceFile.getAbsoluteFile()));
     679                Main.info(tr("Missing preference file ''{0}''. Creating a default preference file.", preferenceFile.getAbsoluteFile()));
    680680                resetToDefault();
    681681                save();
    682682            } else if (reset) {
    683                 System.out.println(tr("Warning: Replacing existing preference file ''{0}'' with default preference file.", preferenceFile.getAbsoluteFile()));
     683                Main.warn(tr("Replacing existing preference file ''{0}'' with default preference file.", preferenceFile.getAbsoluteFile()));
    684684                resetToDefault();
    685685                save();
    … …  
    712712            } catch(IOException e1) {
    713713                e1.printStackTrace();
    714                 System.err.println(tr("Warning: Failed to initialize preferences. Failed to reset preference file to default: {0}", getPreferenceFile()));
     714                Main.warn(tr("Failed to initialize preferences. Failed to reset preference file to default: {0}", getPreferenceFile()));
    715715            }
    716716        }
    … …  
    906906            try {
    907907                save();
    908             } catch(IOException e){
    909                 System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
     908            } catch (IOException e){
     909                Main.warn(tr("Failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
    910910            }
    911911        }
    … …  
    10061006            try {
    10071007                save();
    1008             } catch(IOException e){
    1009                 System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
     1008            } catch (IOException e){
     1009                Main.warn(tr("Failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
    10101010            }
    10111011        }
    … …  
    10751075            try {
    10761076                save();
    1077             } catch(IOException e){
    1078                 System.out.println(tr("Warning: failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
     1077            } catch (IOException e) {
     1078                Main.warn(tr("Failed to persist preferences to ''{0}''", getPreferenceFile().getAbsoluteFile()));
    10791079            }
    10801080        }
    … …  
    16571657        for (String key : obsolete) {
    16581658            boolean removed = false;
    1659             if(properties.containsKey(key)) { properties.remove(key); removed = true; }
    1660             if(collectionProperties.containsKey(key)) { collectionProperties.remove(key); removed = true; }
    1661             if(arrayProperties.containsKey(key)) { arrayProperties.remove(key); removed = true; }
    1662             if(listOfStructsProperties.containsKey(key)) { listOfStructsProperties.remove(key); removed = true; }
    1663             if(removed)
    1664                 System.out.println(tr("Preference setting {0} has been removed since it is no longer used.", key));
     1659            if (properties.containsKey(key)) { properties.remove(key); removed = true; }
     1660            if (collectionProperties.containsKey(key)) { collectionProperties.remove(key); removed = true; }
     1661            if (arrayProperties.containsKey(key)) { arrayProperties.remove(key); removed = true; }
     1662            if (listOfStructsProperties.containsKey(key)) { listOfStructsProperties.remove(key); removed = true; }
     1663            if (removed) {
     1664                Main.info(tr("Preference setting {0} has been removed since it is no longer used.", key));
     1665            }
    16651666        }
    16661667    }

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

    r5874 r6248  
    4747        public String download() throws MissingPassword {
    4848            try {
    49                 System.out.println("reading preferences from "+serverUrl);
     49                Main.info("reading preferences from "+serverUrl);
    5050                URLConnection con = serverUrl.openConnection();
    5151                String username = get("applet.username");
    … …  
    8080            try {
    8181                URL u = new URL(getPreferencesDir());
    82                 System.out.println("uploading preferences to "+u);
     82                Main.info("uploading preferences to "+u);
    8383                HttpURLConnection con = (HttpURLConnection)u.openConnection();
    8484                String username = get("applet.username");

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

    r6246 r6248  
    4848            s = sb.toString();
    4949        } catch (IOException e) {
    50             System.err.println(tr("Failed to load resource ''{0}'', error is {1}.", resource.toString(), e.toString()));
     50            Main.error(tr("Failed to load resource ''{0}'', error is {1}.", resource.toString(), e.toString()));
    5151            e.printStackTrace();
    5252        }
    … …  
    114114            } catch(NumberFormatException e) {
    115115                version = 0;
    116                 System.err.println(tr("Warning: unexpected JOSM version number in revision file, value is ''{0}''", value));
     116                Main.warn(tr("Unexpected JOSM version number in revision file, value is ''{0}''", value));
    117117            }
    118118        } else {
    … …  
    158158        URL u = Main.class.getResource("/REVISION");
    159159        if (u == null) {
    160             System.err.println(tr("Warning: the revision file ''/REVISION'' is missing."));
     160            Main.warn(tr("The revision file ''/REVISION'' is missing."));
    161161            version = 0;
    162162            releaseDescription = "";

  • trunk/src/org/openstreetmap/josm/data/coor/LatLon.java

    r6203 r6248  
    254254        // (This should almost never happen.)
    255255        if (java.lang.Double.isNaN(d)) {
    256             System.err.println("Error: NaN in greatCircleDistance");
     256            Main.error("NaN in greatCircleDistance");
    257257            d = PI * R;
    258258        }

  • trunk/src/org/openstreetmap/josm/data/imagery/GeorefImage.java

    r4065 r6248  
    148148            }
    149149            long freeMem = Runtime.getRuntime().maxMemory() - Runtime.getRuntime().totalMemory();
    150             //System.out.println("Free Memory:           "+ (freeMem/1024/1024) +" MB");
     150            //Main.debug("Free Memory:           "+ (freeMem/1024/1024) +" MB");
    151151            // Notice that this value can get negative due to integer overflows
    152             //System.out.println("Img Size:              "+ (width*height*3/1024/1024) +" MB");
     152            //Main.debug("Img Size:              "+ (width*height*3/1024/1024) +" MB");
    153153
    154154            int multipl = alphaChannel ? 4 : 3;

  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r6069 r6248  
    229229                    }
    230230                } catch (IllegalArgumentException ex) {
    231                     Main.warn(ex.toString());
     231                    Main.warn(ex);
    232232                }
    233233            }

  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryLayerInfo.java

    r6143 r6248  
    99import java.util.List;
    1010
    11 import org.xml.sax.SAXException;
    12 
    1311import org.openstreetmap.josm.Main;
    1412import org.openstreetmap.josm.data.imagery.ImageryInfo.ImageryPreferenceEntry;
    … …  
    1614import org.openstreetmap.josm.io.imagery.ImageryReader;
    1715import org.openstreetmap.josm.tools.Utils;
     16import org.xml.sax.SAXException;
    1817
    1918/**
    … …  
    5049                    add(i);
    5150                } catch (IllegalArgumentException e) {
    52                     System.err.println("Warning: Unable to load imagery preference entry:"+e);
     51                    Main.warn("Unable to load imagery preference entry:"+e);
    5352                }
    5453            }

  • trunk/src/org/openstreetmap/josm/data/imagery/OffsetBookmark.java

    r5460 r6248  
    5656        }
    5757        if (projectionCode == null) {
    58             System.err.println(tr("Projection ''{0}'' is not found, bookmark ''{1}'' is not usable", projectionCode, name));
     58            Main.error(tr("Projection ''{0}'' is not found, bookmark ''{1}'' is not usable", projectionCode, name));
    5959        }
    6060    }

  • trunk/src/org/openstreetmap/josm/data/imagery/WmsCache.java

    r6093 r6248  
    123123                layersIndex.load(fis);
    124124            } catch (FileNotFoundException e) {
    125                 System.out.println("Unable to load layers index for wms cache (file " + layerIndexFile + " not found)");
     125                Main.error("Unable to load layers index for wms cache (file " + layerIndexFile + " not found)");
    126126            } catch (IOException e) {
    127                 System.err.println("Unable to load layers index for wms cache");
     127                Main.error("Unable to load layers index for wms cache");
    128128                e.printStackTrace();
    129129            }
    … …  
    151151                    layersIndex.store(fos, "");
    152152                } catch (IOException e) {
    153                     System.err.println("Unable to save layer index for wms cache");
     153                    Main.error("Unable to save layer index for wms cache");
    154154                    e.printStackTrace();
    155155                }
    … …  
    187187            totalFileSize = cacheEntries.getTotalFileSize();
    188188            if (cacheEntries.getTileSize() != tileSize) {
    189                 System.out.println("Cache created with different tileSize, cache will be discarded");
     189                Main.info("Cache created with different tileSize, cache will be discarded");
    190190                return;
    191191            }
    … …  
    202202            if (indexFile.exists()) {
    203203                e.printStackTrace();
    204                 System.out.println("Unable to load index for wms-cache, new file will be created");
     204                Main.info("Unable to load index for wms-cache, new file will be created");
    205205            } else {
    206                 System.out.println("Index for wms-cache doesn't exist, new file will be created");
     206                Main.info("Index for wms-cache doesn't exist, new file will be created");
    207207            }
    208208        }
    … …  
    297297            marshaller.marshal(index, new FileOutputStream(new File(cacheDir, INDEX_FILENAME)));
    298298        } catch (Exception e) {
    299             System.err.println("Failed to save wms-cache file");
     299            Main.error("Failed to save wms-cache file");
    300300            e.printStackTrace();
    301301        }
    … …  
    364364                return loadImage(projectionEntries, entry);
    365365            } catch (IOException e) {
    366                 System.err.println("Unable to load file from wms cache");
     366                Main.error("Unable to load file from wms cache");
    367367                e.printStackTrace();
    368368                return null;

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

    r5881 r6248  
    799799        OsmPrimitive result = getPrimitiveById(primitiveId);
    800800        if (result == null) {
    801             System.out.println(tr("JOSM expected to find primitive [{0} {1}] in dataset but it is not there. Please report this "
     801            Main.warn(tr("JOSM expected to find primitive [{0} {1}] in dataset but it is not there. Please report this "
    802802                    + "at http://josm.openstreetmap.de/. This is not a critical error, it should be safe to continue in your work.",
    803803                    primitiveId.getType(), Long.toString(primitiveId.getUniqueId())));

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

    r6233 r6248  
    745745            reversedDirectionKeys = SearchCompiler.compile(Main.pref.get("tags.reversed_direction", reversedDirectionDefault), false, false);
    746746        } catch (ParseError e) {
    747             System.err.println("Unable to compile pattern for tags.reversed_direction, trying default pattern: " + e.getMessage());
     747            Main.error("Unable to compile pattern for tags.reversed_direction, trying default pattern: " + e.getMessage());
    748748
    749749            try {
    … …  
    756756            directionKeys = SearchCompiler.compile(Main.pref.get("tags.direction", directionDefault), false, false);
    757757        } catch (ParseError e) {
    758             System.err.println("Unable to compile pattern for tags.direction, trying default pattern: " + e.getMessage());
     758            Main.error("Unable to compile pattern for tags.direction, trying default pattern: " + e.getMessage());
    759759
    760760            try {

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapRendererFactory.java

    r6069 r6248  
    106106            }
    107107        }
    108         System.err.println(tr("Error: failed to load map renderer class ''{0}''. The class wasn''t found.", className));
     108        Main.error(tr("Failed to load map renderer class ''{0}''. The class wasn''t found.", className));
    109109        return null;
    110110    }
    … …  
    126126        Class<?> c = loadRendererClass(rendererClassName);
    127127        if (c == null){
    128             System.err.println(tr("Can''t activate map renderer class ''{0}'', because the class wasn''t found.", rendererClassName));
    129             System.err.println(tr("Activating the standard map renderer instead."));
     128            Main.error(tr("Can''t activate map renderer class ''{0}'', because the class wasn''t found.", rendererClassName));
     129            Main.error(tr("Activating the standard map renderer instead."));
    130130            activateDefault();
    131131        } else if (! AbstractMapRenderer.class.isAssignableFrom(c)) {
    132             System.err.println(tr("Can''t activate map renderer class ''{0}'', because it isn''t a subclass of ''{1}''.", rendererClassName, AbstractMapRenderer.class.getName()));
    133             System.err.println(tr("Activating the standard map renderer instead."));
     132            Main.error(tr("Can''t activate map renderer class ''{0}'', because it isn''t a subclass of ''{1}''.", rendererClassName, AbstractMapRenderer.class.getName()));
     133            Main.error(tr("Activating the standard map renderer instead."));
    134134            activateDefault();
    135135        } else {
    136136            Class<? extends AbstractMapRenderer> renderer = c.asSubclass(AbstractMapRenderer.class);
    137137            if (! isRegistered(renderer)) {
    138                 System.err.println(tr("Can''t activate map renderer class ''{0}'', because it isn''t registered as map renderer.", rendererClassName));
    139                 System.err.println(tr("Activating the standard map renderer instead."));
     138                Main.error(tr("Can''t activate map renderer class ''{0}'', because it isn''t registered as map renderer.", rendererClassName));
     139                Main.error(tr("Activating the standard map renderer instead."));
    140140                activateDefault();
    141141            } else {

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r6203 r6248  
    14031403    @Override
    14041404    public void render(final DataSet data, boolean renderVirtualNodes, Bounds bounds) {
    1405         //long start = System.currentTimeMillis();
    14061405        BBox bbox = bounds.toBBox();
    14071406        getSettings(renderVirtualNodes);
    … …  
    14201419        collectWayStyles(data, sc, bbox);
    14211420        collectRelationStyles(data, sc, bbox);
    1422         //long phase1 = System.currentTimeMillis();
    14231421        sc.drawAll();
    14241422        sc = null;
    14251423        drawVirtualNodes(data, bbox);
    1426 
    1427         //long now = System.currentTimeMillis();
    1428         //System.err.println(String.format("PAINTING TOOK %d [PHASE1 took %d] (at scale %s)", now - start, phase1 - start, circum));
    14291424    }
    14301425}

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/relations/Multipolygon.java

    r6219 r6248  
    292292                if (ds == null) {
    293293                    // DataSet still not found. This should not happen, but a warning does no harm
    294                     System.err.println("Warning: DataSet not found while resetting nodes in Multipolygon. This should not happen, you may report it to JOSM developers.");
     294                    Main.warn("DataSet not found while resetting nodes in Multipolygon. This should not happen, you may report it to JOSM developers.");
    295295                } else if (wayIds.size() == 1) {
    296296                    Way w = (Way) ds.getPrimitiveById(wayIds.iterator().next(), OsmPrimitiveType.WAY);

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

    r6135 r6248  
    104104        return degree + (minute/60.0) + (second/3600.0);
    105105    }
    106 
    107     public void dump() {
    108         System.err.println("x_0="+x_0);
    109         System.err.println("y_0="+y_0);
    110         System.err.println("lon_0="+lon_0);
    111         System.err.println("k_0="+k_0);
    112         System.err.println("ellps="+ellps);
    113         System.err.println("proj="+proj);
    114         System.err.println("datum="+datum);
    115     }
    116 
    117106}

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

    r6135 r6248  
    138138                        inits.put("EPSG:" + m.group(1), Pair.create(name, m.group(2).trim()));
    139139                    } else {
    140                         System.err.println("Warning: failed to parse line from the epsg projection definition: "+line);
     140                        Main.warn("Failed to parse line from the EPSG projection definition: "+line);
    141141                    }
    142142                }

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

    r6142 r6248  
    259259                                checkerData.add(d);
    260260                            } else {
    261                                 System.err.println(tr("Invalid tagchecker line - {0}: {1}", err, line));
     261                                Main.error(tr("Invalid tagchecker line - {0}: {1}", err, line));
    262262                            }
    263263                        }
    … …  
    267267                        spellCheckKeyData.put(line.substring(1), okValue);
    268268                    } else {
    269                         System.err.println(tr("Invalid spellcheck line: {0}", line));
     269                        Main.error(tr("Invalid spellcheck line: {0}", line));
    270270                    }
    271271                }
    … …  
    310310                                presetsValueData.putAll(ky.key, ky.getValues());
    311311                            } catch (NullPointerException e) {
    312                                 System.err.println(p+": Unable to initialize "+ky);
     312                                Main.error(p+": Unable to initialize "+ky);
    313313                            }
    314314                        }

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

    r6203 r6248  
    112112                    bookmarks.add(new Bookmark(entry));
    113113                }
    114                 catch(Exception e) {
    115                     System.err.println(tr("Error reading bookmark entry: %s", e.getMessage()));
     114                catch (Exception e) {
     115                    Main.error(tr("Error reading bookmark entry: %s", e.getMessage()));
    116116                }
    117117            }
    … …  
    126126                LinkedList<Bookmark> bookmarks = new LinkedList<Bookmark>();
    127127                if (bookmarkFile.exists()) {
    128                     System.out.println("Try loading obsolete bookmarks file");
     128                    Main.info("Try loading obsolete bookmarks file");
    129129                    BufferedReader in = new BufferedReader(new InputStreamReader(
    130130                            new FileInputStream(bookmarkFile), "utf-8"));
    … …  
    133133                        Matcher m = Pattern.compile("^(.+)[,\u001e](-?\\d+.\\d+)[,\u001e](-?\\d+.\\d+)[,\u001e](-?\\d+.\\d+)[,\u001e](-?\\d+.\\d+)$").matcher(line);
    134134                        if (!m.matches() || m.groupCount() != 5) {
    135                             System.err.println(tr("Error: Unexpected line ''{0}'' in bookmark file ''{1}''",line, bookmarkFile.toString()));
     135                            Main.error(tr("Unexpected line ''{0}'' in bookmark file ''{1}''",line, bookmarkFile.toString()));
    136136                            continue;
    137137                        }
    … …  
    142142                            try {
    143143                                values[i] = Double.parseDouble(m.group(i+2));
    144                             } catch(NumberFormatException e) {
    145                                 System.err.println(tr("Error: Illegal double value ''{0}'' on line ''{1}'' in bookmark file ''{2}''",m.group(i+2),line, bookmarkFile.toString()));
     144                            } catch (NumberFormatException e) {
     145                                Main.error(tr("Illegal double value ''{0}'' on line ''{1}'' in bookmark file ''{2}''",m.group(i+2),line, bookmarkFile.toString()));
    146146                                continue;
    147147                            }
    … …  
    156156                    }
    157157                    save();
    158                     System.out.println("Removing obsolete bookmarks file");
     158                    Main.info("Removing obsolete bookmarks file");
    159159                    bookmarkFile.delete();
    160160                }

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

    r6143 r6248  
    130130                        content = new MotdContent().updateIfRequiredString();
    131131                    } catch (IOException ex) {
    132                         System.out.println(tr("Warning: failed to read MOTD. Exception was: {0}", ex.toString()));
     132                        Main.warn(tr("Failed to read MOTD. Exception was: {0}", ex.toString()));
    133133                        content = "<html>" + STYLE + "<h1>" + "JOSM - " + tr("Java OpenStreetMap Editor")
    134134                                + "</h1>\n<h2 align=\"center\">(" + tr("Message of the day not available") + ")</h2></html>";

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

    r6246 r6248  
    320320            CustomConfigurator.XMLCommandProcessor config = new CustomConfigurator.XMLCommandProcessor(Main.pref);
    321321            for (String i : args.get(Option.LOAD_PREFERENCES)) {
    322                 System.out.println("Reading preferences from " + i);
     322                info("Reading preferences from " + i);
    323323                try {
    324324                    config.openAndReadXML(Utils.openURL(new URL(i)));
    … …  
    446446        if (Main.pref.getBoolean("debug.edt-checker.enable", Version.getInstance().isLocalBuild())) {
    447447            // Repaint manager is registered so late for a reason - there is lots of violation during startup process but they don't seem to break anything and are difficult to fix
    448             System.out.println("Enabled EDT checker, wrongful access to gui from non EDT thread will be printed to console");
     448            info("Enabled EDT checker, wrongful access to gui from non EDT thread will be printed to console");
    449449            RepaintManager.setCurrentManager(new CheckThreadViolationRepaintManager());
    450450        }

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

    r6106 r6248  
    130130        public void appendLogMessage(String message) {
    131131            if (message != null && !message.isEmpty()) {
    132                 System.out.println("appendLogMessage not implemented for background tasks. Message was: " + message);
     132                Main.info("appendLogMessage not implemented for background tasks. Message was: " + message);
    133133            }
    134134        }

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

    r6093 r6248  
    3131import java.beans.PropertyChangeListener;
    3232import java.beans.PropertyChangeSupport;
    33 import java.io.IOException;
    3433import java.io.Reader;
    3534import java.io.StreamTokenizer;
    … …  
    4544import javax.swing.UIManager;
    4645
     46import org.openstreetmap.josm.Main;
    4747import org.openstreetmap.josm.tools.Utils;
    4848
    … …  
    12631263        }
    12641264        catch (Exception e) {
    1265             System.err.println(e);
     1265            Main.error(e);
    12661266        }
    12671267        finally {
    … …  
    13201320        return parseModel(new StringReader(s));
    13211321    }
    1322 
    1323     private static void printModel(String indent, Node root) {
    1324         if (root instanceof Split) {
    1325             Split split = (Split)root;
    1326             System.out.println(indent + split);
    1327             for(Node child : split.getChildren()) {
    1328                 printModel(indent + "  ", child);
    1329             }
    1330         }
    1331         else {
    1332             System.out.println(indent + root);
    1333         }
    1334     }
    1335 
    1336     /**
    1337      * Print the tree with enough detail for simple debugging.
    1338      */
    1339     public static void printModel(Node root) {
    1340         printModel("", root);
    1341     }
    13421322}

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

    r6203 r6248  
    860860
    861861                    if (perDistSq < snapDistanceSq && a < c + snapDistanceSq && b < c + snapDistanceSq) {
    862                         //System.err.println(Double.toHexString(perDistSq));
    863 
    864862                        List<WaySegment> wslist;
    865863                        if (nearestMap.containsKey(perDistSq)) {

  • trunk/src/org/openstreetmap/josm/gui/bbox/TileSelectionBBoxChooser.java

    r6246 r6248  
    6969 *            // listen for BBOX events
    7070 *            if (evt.getPropertyName().equals(BBoxChooser.BBOX_PROP)) {
    71  *               System.out.println("new bbox based on OSM tiles selected: " + (Bounds)evt.getNewValue());
     71 *               Main.info("new bbox based on OSM tiles selected: " + (Bounds)evt.getNewValue());
    7272 *            }
    7373 *        }

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/ListMergeModel.java

    r6084 r6248  
    719719                return ((RelationMember)value).getMember();
    720720            } else {
    721                 System.err.println("Unknown object type: "+value);
     721                Main.error("Unknown object type: "+value);
    722722                return null;
    723723            }

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

    r6104 r6248  
    317317    @Override
    318318    public void onConflictsRemoved(ConflictCollection conflicts) {
    319         System.err.println("1 conflict has been resolved.");
     319        Main.info("1 conflict has been resolved.");
    320320        refreshView();
    321321    }

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

    r6084 r6248  
    204204        }
    205205        if (!toReload.isEmpty()) {
    206             System.out.println(trn("Reloading {0} map style.", "Reloading {0} map styles.", toReload.size(), toReload.size()));
     206            Main.info(trn("Reloading {0} map style.", "Reloading {0} map styles.", toReload.size(), toReload.size()));
    207207            Main.worker.submit(new MapPaintStyleLoader(toReload));
    208208        }

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

    r6246 r6248  
    846846                        buttonActions.add(action);
    847847                    } else {
    848                         System.err.println("Button " + button + " doesn't have action defined");
     848                        Main.warn("Button " + button + " doesn't have action defined");
    849849                        new Exception().printStackTrace();
    850850                    }

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

    r6085 r6248  
    203203            if (users.isEmpty()) return;
    204204            if (users.size() > 10) {
    205                 System.out.println(tr("Warning: only launching info browsers for the first {0} of {1} selected users", 10, users.size()));
     205                Main.warn(tr("Only launching info browsers for the first {0} of {1} selected users", 10, users.size()));
    206206            }
    207207            int num = Math.min(10, users.size());

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

    r6084 r6248  
    191191                q = BasicQuery.valueOf(BasicQuery.class, value);
    192192            } catch(IllegalArgumentException e) {
    193                 System.err.println(tr("Warning: unexpected value for preference ''{0}'', got ''{1}''. Resetting to default query.","changeset-query.basic.query", value));
     193                Main.warn(tr("Unexpected value for preference ''{0}'', got ''{1}''. Resetting to default query.","changeset-query.basic.query", value));
    194194                q = BasicQuery.MOST_RECENT_CHANGESETS;
    195195            }

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

    r6084 r6248  
    428428            } catch (Exception e) {
    429429                if (canceled) {
    430                     System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e
    431                             .toString()));
     430                    Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    432431                    return;
    433432                }
    … …  
    521520            } catch (Exception e) {
    522521                if (canceled) {
    523                     System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e
    524                             .toString()));
     522                    Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    525523                    return;
    526524                }

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

    r6093 r6248  
    145145        } catch (Exception e) {
    146146            if (canceled) {
    147                 System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e
    148                         .toString()));
     147                Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    149148                return;
    150149            }

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

    r6084 r6248  
    108108        } catch (Exception e) {
    109109            if (canceled) {
    110                 System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e
    111                         .toString()));
     110                Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    112111                return;
    113112            }

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

    r6084 r6248  
    193193        } catch(Exception e) {
    194194            if (canceled) {
    195                 System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e.toString()));
     195                Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    196196                return;
    197197            }

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

    r6084 r6248  
    159159            } catch(Exception e) {
    160160                if (canceled) {
    161                     System.out.println(tr("Warning: Ignoring exception because task was canceled. Exception: {0}", e.toString()));
     161                    Main.warn(tr("Ignoring exception because task was canceled. Exception: {0}", e.toString()));
    162162                    return;
    163163                }

  • trunk/src/org/openstreetmap/josm/gui/help/HelpBrowser.java

    r6105 r6248  
    145145            }
    146146        } catch(Exception e) {
    147             System.err.println(tr("Failed to read CSS file ''help-browser.css''. Exception is: {0}", e.toString()));
     147            Main.error(tr("Failed to read CSS file ''help-browser.css''. Exception is: {0}", e.toString()));
    148148            e.printStackTrace();
    149149            return ss;
    … …  
    499499        @Override
    500500        public void update(Observable o, Object arg) {
    501             //System.out.println("BackAction: canGoBoack=" + history.canGoBack() );
    502501            setEnabled(history.canGoBack());
    503502        }
    … …  
    559558                        return true;
    560559                    }
    561                 } catch(BadLocationException e) {
    562                     System.err.println(tr("Warning: bad location in HTML document. Exception was: {0}", e.toString()));
     560                } catch (BadLocationException e) {
     561                    Main.warn(tr("Bad location in HTML document. Exception was: {0}", e.toString()));
    563562                    e.printStackTrace();
    564563                }

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

    r6199 r6248  
    265265            }
    266266        }
    267         System.out.println(tr("Warning: error header \"{0}\" did not match with an expected pattern", errorHeader));
     267        Main.warn(tr("Error header \"{0}\" did not match with an expected pattern", errorHeader));
    268268        handleUploadConflictForUnknownConflict();
    269269    }
    … …  
    280280            handleUploadPreconditionFailedConflict(e, conflict);
    281281        } else {
    282             System.out.println(tr("Warning: error header \"{0}\" did not match with an expected pattern", e.getErrorHeader()));
     282            Main.warn(tr("Error header \"{0}\" did not match with an expected pattern", e.getErrorHeader()));
    283283            ExceptionDialogUtil.explainPreconditionFailed(e);
    284284        }

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

    r6087 r6248  
    106106            setAlwaysOnTop(true);
    107107        } catch(SecurityException e) {
    108             System.out.println(tr("Warning: failed to put Credential Dialog always on top. Caught security exception."));
     108            Main.warn(tr("Failed to put Credential Dialog always on top. Caught security exception."));
    109109        }
    110110        build();

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

    r6070 r6248  
    1818import java.util.zip.ZipFile;
    1919
     20import org.openstreetmap.josm.Main;
    2021import org.openstreetmap.josm.gui.PleaseWaitDialog;
    2122import org.openstreetmap.josm.gui.PleaseWaitRunnable;
    … …  
    121122            Utils.close(out);
    122123            if (!canceled) {
    123                 System.out.println(tr("Download finished"));
     124                Main.info(tr("Download finished"));
    124125                if (unpack) {
    125                     System.out.println(tr("Unpacking {0} into {1}", file.getAbsolutePath(), file.getParent()));
     126                    Main.info(tr("Unpacking {0} into {1}", file.getAbsolutePath(), file.getParent()));
    126127                    unzipFileRecursively(file, file.getParent());
    127128                    file.delete();
    … …  
    129130            }
    130131        } catch(MalformedURLException e) {
    131             String msg = tr("Warning: Cannot download file ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", file.getName(), address);
    132             System.err.println(msg);
     132            String msg = tr("Cannot download file ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", file.getName(), address);
     133            Main.warn(msg);
    133134            throw new DownloadException(msg);
    134135        } catch (IOException e) {

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

    r5890 r6248  
    117117                im.setPartiallyIdentified(im.getUserName());
    118118            }
    119             System.err.println(tr("Warning: Failed to retrieve user infos for the current JOSM user. Exception was: {0}", e.toString()));
     119            Main.warn(tr("Failed to retrieve user infos for the current JOSM user. Exception was: {0}", e.toString()));
    120120        }
    121121    }

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

    r5266 r6248  
    77import java.util.HashSet;
    88
     9import org.openstreetmap.josm.Main;
    910import org.openstreetmap.josm.actions.upload.CyclicUploadDependencyException;
    1011import org.openstreetmap.josm.data.APIDataSet;
    … …  
    9394            // we tried to delete an already deleted primitive.
    9495            //
    95             System.out.println(tr("Warning: object ''{0}'' is already deleted on the server. Skipping this object and retrying to upload.", p.getDisplayName(DefaultNameFormatter.getInstance())));
     96            Main.warn(tr("Object ''{0}'' is already deleted on the server. Skipping this object and retrying to upload.", p.getDisplayName(DefaultNameFormatter.getInstance())));
    9697            processedPrimitives.addAll(writer.getProcessedPrimitives());
    9798            processedPrimitives.add(p);
    … …  
    138139        } catch (Exception sxe) {
    139140            if (isCanceled()) {
    140                 System.out.println("Ignoring exception caught because upload is canceled. Exception is: " + sxe.toString());
     141                Main.info("Ignoring exception caught because upload is canceled. Exception is: " + sxe.toString());
    141142                return;
    142143            }

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

    r6132 r6248  
    2626import org.openstreetmap.josm.gui.HelpAwareOptionPane;
    2727import org.openstreetmap.josm.gui.HelpAwareOptionPane.ButtonSpec;
     28import org.openstreetmap.josm.gui.Notification;
    2829import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    2930import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    … …  
    3738import org.openstreetmap.josm.tools.ImageProvider;
    3839import org.xml.sax.SAXException;
    39 
    40 import org.openstreetmap.josm.gui.Notification;
    4140
    4241/**
    … …  
    206205            }
    207206            monitor.appendLogMessage(msg);
    208             System.out.println(tr("Warning: {0}", msg));
     207            Main.warn(msg);
    209208            processedPrimitives.addAll(writer.getProcessedPrimitives());
    210209            processedPrimitives.add(p);
    … …  
    301300        } catch (Exception e) {
    302301            if (uploadCanceled) {
    303                 System.out.println(tr("Ignoring caught exception because upload is canceled. Exception is: {0}", e.toString()));
     302                Main.info(tr("Ignoring caught exception because upload is canceled. Exception is: {0}", e.toString()));
    304303            } else {
    305304                lastException = e;

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

    r6084 r6248  
    151151    public List<OsmPrimitive> getSelectedPrimitives() {
    152152        ArrayList<OsmPrimitive> ret  = new ArrayList<OsmPrimitive>();
    153         System.out.println("selected length:" +lstSelectedPrimitives.getSelectedIndices().length);
    154         for (int i=0; i< lstSelectedPrimitives.getSelectedIndices().length;i++) {
    155             System.out.println("selected:" +lstSelectedPrimitives.getSelectedIndices()[i]);
    156         }
    157153        ret.addAll(lstSelectedPrimitives.getOsmPrimitiveListModel().getPrimitives(lstSelectedPrimitives.getSelectedIndices()));
    158154        ret.addAll(lstDeletedPrimitives.getOsmPrimitiveListModel().getPrimitives(lstDeletedPrimitives.getSelectedIndices()));

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

    r5266 r6248  
    8686        UploadStrategy strategy = fromPreference(v);
    8787        if (strategy == null) {
    88             System.err.println(tr("Warning: unexpected value for key ''{0}'' in preferences, got ''{1}''", "osm-server.upload-strategy", v ));
     88            Main.warn(tr("Unexpected value for key ''{0}'' in preferences, got ''{1}''", "osm-server.upload-strategy", v ));
    8989            return DEFAULT_UPLOAD_STRATEGY;
    9090        }

  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r6232 r6248  
    135135            }
    136136        }
    137         //System.out.println("scanning "+data.tracks.size()+" tracks, found min,max"+min+"--"+max);
    138137        if (min==1e100 || max==-1e100) return null;
    139138        return new Date[]{new Date((long) (min * 1000)), new Date((long) (max * 1000)), };
    … …  
    425424         ********** STEP 1 - GET CONFIG VALUES **************************
    426425         ****************************************************************/
    427         // Long startTime = System.currentTimeMillis();
    428426        Color neutralColor = getColor(true);
    429427        String spec="layer "+getName();
    … …  
    788786            g.setStroke(storedStroke);
    789787        }
    790         // Long duration = System.currentTimeMillis() - startTime;
    791         // System.out.println(duration);
    792788    } // end paint
    793789

  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r6084 r6248  
    396396            data.setVersion(from.getVersion());
    397397        } else if ("0.5".equals(data.getVersion()) ^ "0.5".equals(from.getVersion())) {
    398             System.err.println(tr("Warning: mixing 0.6 and 0.5 data results in version 0.5"));
     398            Main.warn(tr("Mixing 0.6 and 0.5 data results in version 0.5"));
    399399            data.setVersion("0.5");
    400400        }

  • trunk/src/org/openstreetmap/josm/gui/layer/TMSLayer.java

    r6093 r6248  
    329329                String r = new Scanner(in).useDelimiter("\\A").next();
    330330                Utils.close(in);
    331                 System.out.println("Successfully loaded Bing attribution data.");
     331                Main.info("Successfully loaded Bing attribution data.");
    332332                return r.getBytes("utf-8");
    333333            }
    … …  
    347347                            return parseAttributionText(new InputSource(new StringReader((xml))));
    348348                        } catch (IOException ex) {
    349                             System.err.println("Could not connect to Bing API. Will retry in " + waitTimeSec + " seconds.");
     349                            Main.warn("Could not connect to Bing API. Will retry in " + waitTimeSec + " seconds.");
    350350                            Thread.sleep(waitTimeSec * 1000L);
    351351                            waitTimeSec *= 2;

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

    r6246 r6248  
    10521052            delta = Math.round(diff - timezone*60*60); // seconds
    10531053
    1054             /*System.out.println("phto " + firstExifDate);
    1055             System.out.println("gpx  " + firstGPXDate);
    1056             System.out.println("diff " + diff);
    1057             System.out.println("difh " + diffInH);
    1058             System.out.println("days " + dayOffset);
    1059             System.out.println("time " + tz);
    1060             System.out.println("fix  " + timezone);
    1061             System.out.println("offt " + delta);*/
     1054            /*Main.debug("phto " + firstExifDate);
     1055            Main.debug("gpx  " + firstGPXDate);
     1056            Main.debug("diff " + diff);
     1057            Main.debug("difh " + diffInH);
     1058            Main.debug("days " + dayOffset);
     1059            Main.debug("time " + tz);
     1060            Main.debug("fix  " + timezone);
     1061            Main.debug("offt " + delta);*/
    10621062
    10631063            tfTimezone.getDocument().removeDocumentListener(statusBarUpdater);
    … …  
    11601160
    11611161                        } catch(ParseException e) {
    1162                             System.err.println("Error while parsing date \"" + curWpTimeStr + '"');
     1162                            Main.error("Error while parsing date \"" + curWpTimeStr + '"');
    11631163                            e.printStackTrace();
    11641164                            prevWp = null;

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

    r6209 r6248  
    558558
    559559        } catch (Exception ex) { // (other exceptions, e.g. #5271)
    560             System.err.println("Error reading EXIF from file: "+ex);
     560            Main.error("Error reading EXIF from file: "+ex);
    561561            e.setExifCoor(null);
    562562            e.setPos(null);
    … …  
    651651
    652652                if (toDelete.getFile().delete()) {
    653                     System.out.println("File "+toDelete.getFile().toString()+" deleted. ");
     653                    Main.info("File "+toDelete.getFile().toString()+" deleted. ");
    654654                } else {
    655655                    JOptionPane.showMessageDialog(

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

    r6084 r6248  
    3737    @Override
    3838    public void run() {
    39         System.err.println("Load Thumbnails");
     39        Main.debug("Load Thumbnails");
    4040        tracker = new MediaTracker(Main.map.mapView);
    4141        for (int i = 0; i < data.size(); i++) {
    … …  
    7171        if (!cacheOff) {
    7272            BufferedImage cached = cache.getImg(cacheIdent);
    73             if(cached != null) {
    74                 System.err.println(" from cache");
     73            if (cached != null) {
     74                Main.debug(" from cache");
    7575                return cached;
    7676            }
    … …  
    8282            tracker.waitForID(0);
    8383        } catch (InterruptedException e) {
    84             System.err.println(" InterruptedException");
     84            Main.error(" InterruptedException");
    8585            return null;
    8686        }
    8787        if (tracker.isErrorID(1) || img.getWidth(null) <= 0 || img.getHeight(null) <= 0) {
    88             System.err.println(" Invalid image");
     88            Main.error(" Invalid image");
    8989            return null;
    9090        }
    … …  
    104104
    105105        if (scaledBI.getWidth() <= 0 || scaledBI.getHeight() <= 0) {
    106             System.err.println(" Invalid image");
     106            Main.error(" Invalid image");
    107107            return null;
    108108        }
    … …  
    112112        }
    113113
    114         System.err.println("");
    115114        return scaledBI;
    116115    }

  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/ImportAudioAction.java

    r6104 r6248  
    131131            url = wavFile.toURI().toURL();
    132132        } catch (MalformedURLException e) {
    133             System.err.println("Unable to convert filename " + wavFile.getAbsolutePath() + " to URL");
     133            Main.error("Unable to convert filename " + wavFile.getAbsolutePath() + " to URL");
    134134        }
    135135        Collection<WayPoint> waypoints = new ArrayList<WayPoint>();

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

    r6242 r6248  
    291291            }
    292292        } catch (URISyntaxException e) {
    293             Main.warn("URISyntaxException: "+e.getMessage());
     293            Main.warn(e);
    294294        }
    295295        return true;

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

    r6142 r6248  
    1111import java.util.regex.Pattern;
    1212
     13import org.openstreetmap.josm.Main;
    1314import org.openstreetmap.josm.gui.mappaint.mapcss.CSSColors;
    1415import org.openstreetmap.josm.tools.Utils;
    … …  
    4950        if (res == null) {
    5051            if (!suppressWarnings) {
    51                 System.err.println(String.format("Warning: unable to convert property %s to type %s: found %s of type %s!", key, klass, o, o.getClass()));
     52                Main.warn(String.format("Unable to convert property %s to type %s: found %s of type %s!", key, klass, o, o.getClass()));
    5253            }
    5354            return def;

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

    r6174 r6248  
    100100                .setHeight(height)
    101101                .setOptional(true).get();
    102         if(i == null)
    103         {
    104             System.out.println("Mappaint style \""+namespace+"\" ("+ref.source.getDisplayString()+") icon \"" + ref.iconName + "\" not found.");
     102        if (i == null) {
     103            Main.warn("Mappaint style \""+namespace+"\" ("+ref.source.getDisplayString()+") icon \"" + ref.iconName + "\" not found.");
    105104            return null;
    106105        }
    … …  
    243242                    }
    244243                }
    245                 System.err.println("Warning: Could not detect style type. Using default (xml).");
     244                Main.warn("Could not detect style type. Using default (xml).");
    246245                return new XmlStyleSource(entry);
    247246            }
    248247        } catch (IOException e) {
    249             System.err.println(tr("Warning: failed to load Mappaint styles from ''{0}''. Exception was: {1}", entry.url, e.toString()));
     248            Main.warn(tr("Failed to load Mappaint styles from ''{0}''. Exception was: {1}", entry.url, e.toString()));
    250249            e.printStackTrace();
    251250        } finally {

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

    r6142 r6248  
    525525                throw new RuntimeException(ex);
    526526            } catch (InvocationTargetException ex) {
    527                 System.err.println(ex);
     527                Main.error(ex);
    528528                return null;
    529529            }
    … …  
    574574                throw new RuntimeException(ex);
    575575            } catch (InvocationTargetException ex) {
    576                 System.err.println(ex);
     576                Main.error(ex);
    577577                return null;
    578578            }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParser.jj

    r5705 r6248  
    2727import org.openstreetmap.josm.tools.Pair;
    2828import org.openstreetmap.josm.tools.Utils;
     29import org.openstreetmap.josm.Main;
    2930
    3031public class MapCSSParser {
    … …  
    573574    }
    574575   
    575     System.err.println("Skipping to the next rule, because of an error:");
    576     System.err.println(e);
     576    Main.error("Skipping to the next rule, because of an error:");
     577    Main.error(e);
    577578    if (sheet != null) {
    578579        sheet.logError(e);
    … …  
    598599                t.image.contains("\n")) {
    599600            ParseException e = new ParseException(String.format("Warning: end of line while reading an unquoted string at line %s column %s.", t.beginLine, t.beginColumn));
    600             System.err.println(e);
     601            Main.error(e);
    601602            if (sheet != null) {
    602603                sheet.logError(e);

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

    r6175 r6248  
    1515import java.util.zip.ZipFile;
    1616
     17import org.openstreetmap.josm.Main;
    1718import org.openstreetmap.josm.data.osm.Node;
    1819import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    7071            loadMeta();
    7172            loadCanvas();
    72         } catch(IOException e) {
    73             System.err.println(tr("Warning: failed to load Mappaint styles from ''{0}''. Exception was: {1}", url, e.toString()));
     73        } catch (IOException e) {
     74            Main.warn(tr("Failed to load Mappaint styles from ''{0}''. Exception was: {1}", url, e.toString()));
    7475            e.printStackTrace();
    7576            logError(e);
    7677        } catch (TokenMgrError e) {
    77             System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
     78            Main.warn(tr("Failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
    7879            e.printStackTrace();
    7980            logError(e);
    8081        } catch (ParseException e) {
    81             System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
     82            Main.warn(tr("Failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
    8283            e.printStackTrace();
    8384            logError(new ParseException(e.getMessage())); // allow e to be garbage collected, it links to the entire token stream

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

    r6175 r6248  
    55import java.util.regex.PatternSyntaxException;
    66
     7import org.openstreetmap.josm.Main;
    78import org.openstreetmap.josm.data.osm.Node;
    89import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    224225                    if (!c.applies(env)) return false;
    225226                } catch (PatternSyntaxException e) {
    226                     System.err.println("PatternSyntaxException while applying condition" + c +": "+e.getMessage());
     227                    Main.error("PatternSyntaxException while applying condition" + c +": "+e.getMessage());
    227228                    return false;
    228229                }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/LinePrototype.java

    r3856 r6248  
    55import java.util.List;
    66
     7import org.openstreetmap.josm.Main;
    78import org.openstreetmap.josm.data.osm.visitor.paint.MapPaintSettings;
    89import org.openstreetmap.josm.data.osm.visitor.paint.PaintColors;
    … …  
    6263            }
    6364            if (f < 0) {
    64                 System.err.println(I18n.tr("Illegal dash pattern, values must be positive"));
     65                Main.error(I18n.tr("Illegal dash pattern, values must be positive"));
    6566                this.dashed = null;
    6667                return;
    … …  
    7071            this.dashed = dashed;
    7172        } else {
    72             System.err.println(I18n.tr("Illegal dash pattern, at least one value must be > 0"));
     73            Main.error(I18n.tr("Illegal dash pattern, at least one value must be > 0"));
    7374        }
    7475    }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlStyleSource.java

    r6148 r6248  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import java.io.IOException;
    67import java.io.InputStream;
    78import java.io.InputStreamReader;
    8 import java.io.IOException;
    99import java.util.Collection;
    1010import java.util.Collections;
    … …  
    7676            }
    7777
    78         } catch(IOException e) {
    79             System.err.println(tr("Warning: failed to load Mappaint styles from ''{0}''. Exception was: {1}", url, e.toString()));
     78        } catch (IOException e) {
     79            Main.warn(tr("Failed to load Mappaint styles from ''{0}''. Exception was: {1}", url, e.toString()));
    8080            e.printStackTrace();
    8181            logError(e);
    82         } catch(SAXParseException e) {
    83             System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: [{1}:{2}] {3}", url, e.getLineNumber(), e.getColumnNumber(), e.getMessage()));
     82        } catch (SAXParseException e) {
     83            Main.warn(tr("Failed to parse Mappaint styles from ''{0}''. Error was: [{1}:{2}] {3}", url, e.getLineNumber(), e.getColumnNumber(), e.getMessage()));
    8484            e.printStackTrace();
    8585            logError(e);
    86         } catch(SAXException e) {
    87             System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
     86        } catch (SAXException e) {
     87            Main.warn(tr("Failed to parse Mappaint styles from ''{0}''. Error was: {1}", url, e.getMessage()));
    8888            e.printStackTrace();
    8989            logError(e);

  • trunk/src/org/openstreetmap/josm/gui/mappaint/xml/XmlStyleSourceHandler.java

    r6246 r6248  
    7373    private void error(String message) {
    7474        String warning = style.getDisplayString() + " (" + rule.cond.key + "=" + rule.cond.value + "): " + message;
    75         System.err.println(warning);
     75        Main.warn(warning);
    7676        style.logError(new Exception(warning));
    7777    }

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

    r6070 r6248  
    110110                    con.disconnect();
    111111                }
    112             } catch(NoSuchFieldException e) {
     112            } catch (NoSuchFieldException e) {
    113113                e.printStackTrace();
    114                 System.err.println(tr("Warning: failed to cancel running OAuth operation"));
    115             } catch(SecurityException e) {
     114                Main.warn(tr("Failed to cancel running OAuth operation"));
     115            } catch (SecurityException e) {
    116116                e.printStackTrace();
    117                 System.err.println(tr("Warning: failed to cancel running OAuth operation"));
    118             } catch(IllegalAccessException e) {
     117                Main.warn(tr("Failed to cancel running OAuth operation"));
     118            } catch (IllegalAccessException e) {
    119119                e.printStackTrace();
    120                 System.err.println(tr("Warning: failed to cancel running OAuth operation"));
     120                Main.warn(tr("Failed to cancel running OAuth operation"));
    121121            }
    122122        }

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

    r6106 r6248  
    12471247                        Matcher m = Pattern.compile("^\t([^:]+): *(.+)$").matcher(line);
    12481248                        if (! m.matches()) {
    1249                             System.err.println(tr(getStr(I18nString.ILLEGAL_FORMAT_OF_ENTRY), url, line));
     1249                            Main.error(tr(getStr(I18nString.ILLEGAL_FORMAT_OF_ENTRY), url, line));
    12501250                            continue;
    12511251                        }
    … …  
    12911291                            sources.add(last = new ExtendedSourceEntry(m.group(1), m.group(2)));
    12921292                        } else {
    1293                             System.err.println(tr(getStr(I18nString.ILLEGAL_FORMAT_OF_ENTRY), url, line));
     1293                            Main.error(tr(getStr(I18nString.ILLEGAL_FORMAT_OF_ENTRY), url, line));
    12941294                        }
    12951295                    }

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

    r6152 r6248  
    77import java.util.regex.Matcher;
    88import java.util.regex.Pattern;
     9
     10import org.openstreetmap.josm.Main;
    911
    1012/**
    … …  
    127129            return m.group(1);
    128130        } else {
    129             System.err.println("Warning: Unexpected URL format: "+url);
     131            Main.warn("Unexpected URL format: "+url);
    130132            return url;
    131133        }

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

    r6070 r6248  
    904904                    Object tb = action.getValue("toolbar");
    905905                    if(tb == null) {
    906                         System.out.println(tr("Toolbar action without name: {0}",
     906                        Main.info(tr("Toolbar action without name: {0}",
    907907                        action.getClass().getName()));
    908908                        continue;
    909909                    } else if (!(tb instanceof String)) {
    910910                        if(!(tb instanceof Boolean) || (Boolean)tb) {
    911                             System.out.println(tr("Strange toolbar value: {0}",
     911                            Main.info(tr("Strange toolbar value: {0}",
    912912                            action.getClass().getName()));
    913913                        }
    … …  
    917917                        Action r = actions.get(toolbar);
    918918                        if(r != null && r != action && !toolbar.startsWith("imagery_")) {
    919                             System.out.println(tr("Toolbar action {0} overwritten: {1} gets {2}",
     919                            Main.info(tr("Toolbar action {0} overwritten: {1} gets {2}",
    920920                            toolbar, r.getClass().getName(), action.getClass().getName()));
    921921                        }
    … …  
    988988                    result.add(a);
    989989                } else {
    990                     System.out.println("Could not load tool definition "+s);
     990                    Main.info("Could not load tool definition "+s);
    991991                }
    992992            }
    … …  
    10011001    public Action register(Action action) {
    10021002        String toolbar = (String) action.getValue("toolbar");
    1003         if(toolbar == null) {
    1004             System.out.println(tr("Registered toolbar action without name: {0}",
     1003        if (toolbar == null) {
     1004            Main.info(tr("Registered toolbar action without name: {0}",
    10051005            action.getClass().getName()));
    10061006        } else {
    10071007            Action r = regactions.get(toolbar);
    1008             if(r != null) {
    1009                 System.out.println(tr("Registered toolbar action {0} overwritten: {1} gets {2}",
     1008            if (r != null) {
     1009                Main.info(tr("Registered toolbar action {0} overwritten: {1} gets {2}",
    10101010                toolbar, r.getClass().getName(), action.getClass().getName()));
    10111011            }

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

    r6246 r6248  
    22package org.openstreetmap.josm.gui.preferences.advanced;
    33
     4import static org.openstreetmap.josm.tools.I18n.marktr;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    5 import static org.openstreetmap.josm.tools.I18n.marktr;
    66
    77import java.awt.Dimension;
    … …  
    1717import java.util.Map;
    1818import java.util.Map.Entry;
     19
    1920import javax.swing.AbstractAction;
    20 
    2121import javax.swing.Box;
    2222import javax.swing.JButton;
    … …  
    356356                   if (idx>=0) {
    357357                        String t=s.substring(0,idx);
    358                         System.out.println(t);
    359                         if (profileTypes.containsKey(t))
     358                        if (profileTypes.containsKey(t)) {
    360359                            p.add(new ImportProfileAction(s, f, t));
     360                        }
    361361                   }
    362362                }
    … …  
    366366                   if (idx>=0) {
    367367                        String t=s.substring(0,idx);
    368                         if (profileTypes.containsKey(t))
     368                        if (profileTypes.containsKey(t)) {
    369369                            p.add(new ImportProfileAction(s, f, t));
     370                        }
    370371                   }
    371372                }

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

    r6023 r6248  
    11// License: GPL. See LICENSE file for details.
    22package org.openstreetmap.josm.gui.preferences.advanced;
     3
     4import static org.openstreetmap.josm.tools.I18n.tr;
    35
    46import java.awt.event.ActionEvent;
    … …  
    68import java.util.ArrayList;
    79import java.util.Map;
     10
    811import javax.swing.AbstractAction;
    912import javax.swing.JFileChooser;
    1013import javax.swing.JOptionPane;
    1114import javax.swing.filechooser.FileFilter;
     15
    1216import org.openstreetmap.josm.Main;
    1317import org.openstreetmap.josm.actions.DiskAccessAction;
    … …  
    1519import org.openstreetmap.josm.data.Preferences;
    1620import org.openstreetmap.josm.data.Preferences.Setting;
    17 
    18 import static org.openstreetmap.josm.tools.I18n.tr;
    1921
    2022/**
    … …  
    7072            if (!sel.getName().endsWith(".xml")) sel=new File(sel.getAbsolutePath()+".xml");
    7173            if (!sel.getName().startsWith(schemaKey)) {
    72                 System.out.println(sel.getParentFile().getAbsolutePath()+"/"+schemaKey+"_"+sel.getName());
    73                 sel =new File(sel.getParentFile().getAbsolutePath()+"/"+schemaKey+"_"+sel.getName());
     74                sel = new File(sel.getParentFile().getAbsolutePath()+"/"+schemaKey+"_"+sel.getName());
    7475            }
    7576            return sel;

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

    r6084 r6248  
    2222import javax.swing.JScrollPane;
    2323
     24import org.openstreetmap.josm.Main;
    2425import org.openstreetmap.josm.data.imagery.ImageryInfo;
    2526import org.openstreetmap.josm.gui.bbox.SlippyMapBBoxChooser;
    … …  
    8384                    JOptionPane.showMessageDialog(getParent(), tr("Could not parse WMS layer list."),
    8485                            tr("WMS Error"), JOptionPane.ERROR_MESSAGE);
    85                     System.err.println("Could not parse WMS layer list. Incoming data:");
    86                     System.err.println(ex.getIncomingData());
     86                    Main.error("Could not parse WMS layer list. Incoming data:\n"+ex.getIncomingData());
    8787                }
    8888            }

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

    r6246 r6248  
    8484                            canLoad = true;
    8585                        } catch (IOException e) {
    86                             System.err.println(tr("Warning: Could not read tagging preset source: {0}", source));
     86                            Main.warn(tr("Could not read tagging preset source: {0}", source));
    8787                            ExtendedDialog ed = new ExtendedDialog(Main.parent, tr("Error"),
    8888                                    new String[] {tr("Yes"), tr("No"), tr("Cancel")});
    … …  
    108108                            // Should not happen, but at least show message
    109109                            String msg = tr("Could not read tagging preset source {0}", source);
    110                             System.err.println(msg);
     110                            Main.error(msg);
    111111                            JOptionPane.showMessageDialog(Main.parent, msg);
    112112                            return false;
    … …  
    135135
    136136                        if (errorMessage != null) {
    137                             System.err.println("Error: "+errorMessage);
     137                            Main.error(errorMessage);
    138138                            int result = JOptionPane.showConfirmDialog(Main.parent, new JLabel(errorMessage), tr("Error"),
    139139                                    JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.ERROR_MESSAGE);

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/AuthenticationPreferencesPanel.java

    r6084 r6248  
    104104            rbOAuth.setSelected(true);
    105105        } else {
    106             System.err.println(tr("Warning: Unsupported value in preference ''{0}'', got ''{1}''. Using authentication method ''Basic Authentication''.", "osm-server.auth-method", authMethod));
     106            Main.warn(tr("Unsupported value in preference ''{0}'', got ''{1}''. Using authentication method ''Basic Authentication''.", "osm-server.auth-method", authMethod));
    107107            rbBasicAuthentication.setSelected(true);
    108108        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/BasicAuthenticationPreferencesPanel.java

    r5886 r6248  
    1515import javax.swing.JPanel;
    1616
     17import org.openstreetmap.josm.Main;
    1718import org.openstreetmap.josm.gui.widgets.JosmPasswordField;
    1819import org.openstreetmap.josm.gui.widgets.JosmTextField;
    … …  
    103104        } catch(CredentialsAgentException e) {
    104105            e.printStackTrace();
    105             System.err.println(tr("Warning: failed to retrieve OSM credentials from credential manager."));
    106             System.err.println(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
     106            Main.warn(tr("Failed to retrieve OSM credentials from credential manager."));
     107            Main.warn(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
    107108            tfOsmUserName.setText("");
    108109            tfOsmPassword.setText("");
    … …  
    118119            );
    119120            cm.store(RequestorType.SERVER, OsmApi.getOsmApi().getHost(), pa);
    120         } catch(CredentialsAgentException e) {
     121        } catch (CredentialsAgentException e) {
    121122            e.printStackTrace();
    122             System.err.println(tr("Warning: failed to save OSM credentials to credential manager."));
    123             System.err.println(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
     123            Main.warn(tr("Failed to save OSM credentials to credential manager."));
     124            Main.warn(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
    124125        }
    125126    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/OAuthAccessTokenHolder.java

    r4245 r6248  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import org.openstreetmap.josm.Main;
    67import org.openstreetmap.josm.data.Preferences;
    78import org.openstreetmap.josm.data.oauth.OAuthToken;
    … …  
    144145        } catch(CredentialsAgentException e) {
    145146            e.printStackTrace();
    146             System.err.println(tr("Warning: Failed to retrieve OAuth Access Token from credential manager"));
    147             System.err.println(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
     147            Main.warn(tr("Failed to retrieve OAuth Access Token from credential manager"));
     148            Main.warn(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
    148149        }
    149150        saveToPreferences = pref.getBoolean("oauth.access-token.save-to-preferences", true);
    … …  
    175176        } catch(CredentialsAgentException e){
    176177            e.printStackTrace();
    177             System.err.println(tr("Warning: Failed to store OAuth Access Token to credentials manager"));
    178             System.err.println(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
     178            Main.warn(tr("Failed to store OAuth Access Token to credentials manager"));
     179            Main.warn(tr("Current credential manager is of type ''{0}''", cm.getClass().getName()));
    179180        }
    180181    }

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/ProxyPreferencesPanel.java

    r5899 r6248  
    316316
    317317        if (pp.equals(ProxyPolicy.USE_SYSTEM_SETTINGS) && ! DefaultProxySelector.willJvmRetrieveSystemProxies()) {
    318             System.err.println(tr("Warning: JOSM is configured to use proxies from the system setting, but the JVM is not configured to retrieve them. Resetting preferences to ''No proxy''"));
     318            Main.warn(tr("JOSM is configured to use proxies from the system setting, but the JVM is not configured to retrieve them. Resetting preferences to ''No proxy''"));
    319319            pp = ProxyPolicy.NO_PROXY;
    320320            rbProxyPolicy.get(pp).setSelected(true);

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

    r6070 r6248  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.gui.preferences.shortcut;
     3
     4import static org.openstreetmap.josm.tools.I18n.marktr;
     5import static org.openstreetmap.josm.tools.I18n.tr;
    36
    47import java.awt.Color;
    … …  
    912import java.awt.Insets;
    1013import java.awt.Toolkit;
    11 
    12 import static org.openstreetmap.josm.tools.I18n.marktr;
    13 import static org.openstreetmap.josm.tools.I18n.tr;
    14 
    1514import java.awt.event.KeyEvent;
    1615import java.lang.reflect.Field;
    … …  
    1817import java.util.LinkedHashMap;
    1918import java.util.Map;
    20 
    2119import java.util.regex.PatternSyntaxException;
     20
    2221import javax.swing.AbstractAction;
    2322import javax.swing.BorderFactory;
    … …  
    3837import javax.swing.event.ListSelectionListener;
    3938import javax.swing.table.AbstractTableModel;
    40 import javax.swing.table.TableModel;
    4139import javax.swing.table.DefaultTableCellRenderer;
    4240import javax.swing.table.TableColumnModel;
    43 
     41import javax.swing.table.TableModel;
    4442import javax.swing.table.TableRowSorter;
     43
    4544import org.openstreetmap.josm.Main;
    4645import org.openstreetmap.josm.gui.widgets.JosmComboBox;
     46import org.openstreetmap.josm.gui.widgets.JosmTextField;
    4747import org.openstreetmap.josm.gui.widgets.SelectAllOnFocusGainedDecorator;
    4848import org.openstreetmap.josm.tools.Shortcut;
    49 import org.openstreetmap.josm.gui.widgets.JosmTextField;
    5049
    5150/**
    … …  
    117116                    if (s != null && s.length() > 0 && !s.contains(unknown)) {
    118117                        list.put(Integer.valueOf(i), s);
    119                         //System.out.println(i+": "+s);
    120118                    }
    121119                } catch (Exception e) {

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

    r6092 r6248  
    1717import java.util.Collections;
    1818import java.util.EventObject;
    19 import java.util.HashMap;
    20 import java.util.List;
    21 import java.util.Map;
    2219import java.util.concurrent.CopyOnWriteArrayList;
    2320
    2421import javax.swing.AbstractAction;
    25 import static javax.swing.Action.SHORT_DESCRIPTION;
    26 import static javax.swing.Action.SMALL_ICON;
    2722import javax.swing.CellEditor;
    2823import javax.swing.DefaultListSelectionModel;
    … …  
    3833import javax.swing.table.TableColumn;
    3934import javax.swing.text.JTextComponent;
     35
    4036import org.openstreetmap.josm.Main;
    4137import org.openstreetmap.josm.actions.PasteTagsAction.TagPaster;
    4238import org.openstreetmap.josm.data.osm.OsmPrimitive;
    4339import org.openstreetmap.josm.data.osm.Relation;
    44 
    4540import org.openstreetmap.josm.gui.dialogs.relation.RunnableAction;
     41import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
    4642import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionManager;
    47 import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
    4843import org.openstreetmap.josm.tools.ImageProvider;
    4944
    … …  
    464459    public void setAutoCompletionManager(AutoCompletionManager autocomplete) {
    465460        if (autocomplete == null) {
    466             System.out.println("argument autocomplete should not be null. Aborting.");
     461            Main.warn("argument autocomplete should not be null. Aborting.");
    467462            Thread.dumpStack();
    468463            return;

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

    r6101 r6248  
    150150                    });
    151151                } else {
    152                     System.out.println("Could not get presets icon " + iconName);
     152                    Main.warn("Could not get presets icon " + iconName);
    153153                }
    154154            }
    … …  
    167167            this.nameTemplate = new TemplateParser(pattern).parse();
    168168        } catch (ParseError e) {
    169             System.err.println("Error while parsing " + pattern + ": " + e.getMessage());
     169            Main.error("Error while parsing " + pattern + ": " + e.getMessage());
    170170            throw new SAXException(e);
    171171        }
    … …  
    176176            this.nameTemplateFilter = SearchCompiler.compile(filter, false, false);
    177177        } catch (org.openstreetmap.josm.actions.search.SearchCompiler.ParseError e) {
    178             System.err.println("Error while parsing" + filter + ": " + e.getMessage());
     178            Main.error("Error while parsing" + filter + ": " + e.getMessage());
    179179            throw new SAXException(e);
    180180        }

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

    r6246 r6248  
    626626                        pnl.add(aibutton, GBC.std());
    627627                    } catch (ParseException x) {
    628                         System.err.println("Cannot parse auto-increment value of '" + ai + "' into an integer");
     628                        Main.error("Cannot parse auto-increment value of '" + ai + "' into an integer");
    629629                    }
    630630                }
    … …  
    673673            String v = getValue(value);
    674674            if (v == null) {
    675                 System.err.println("No 'last value' support for component " + value);
     675                Main.error("No 'last value' support for component " + value);
    676676                return;
    677677            }
    … …  
    921921            } else {
    922922                if (values != null) {
    923                     System.err.println(tr("Warning in tagging preset \"{0}-{1}\": "
     923                    Main.warn(tr("Warning in tagging preset \"{0}-{1}\": "
    924924                            + "Ignoring ''{2}'' attribute as ''{3}'' elements are given.",
    925925                            key, text, "values", "list_entry"));
    926926                }
    927927                if (display_values != null || locale_display_values != null) {
    928                     System.err.println(tr("Warning in tagging preset \"{0}-{1}\": "
     928                    Main.warn(tr("Warning in tagging preset \"{0}-{1}\": "
    929929                            + "Ignoring ''{2}'' attribute as ''{3}'' elements are given.",
    930930                            key, text, "display_values", "list_entry"));
    931931                }
    932932                if (short_descriptions != null || locale_short_descriptions != null) {
    933                     System.err.println(tr("Warning in tagging preset \"{0}-{1}\": "
     933                    Main.warn(tr("Warning in tagging preset \"{0}-{1}\": "
    934934                            + "Ignoring ''{2}'' attribute as ''{3}'' elements are given.",
    935935                            key, text, "short_descriptions", "list_entry"));
    … …  
    963963                            value_array = (String[]) method.invoke(null);
    964964                        } else {
    965                             System.err.println(tr("Broken tagging preset \"{0}-{1}\" - Java method given in ''values_from'' is not \"{2}\"", key, text,
     965                            Main.error(tr("Broken tagging preset \"{0}-{1}\" - Java method given in ''values_from'' is not \"{2}\"", key, text,
    966966                                    "public static String[] methodName()"));
    967967                        }
    968968                    } catch (Exception e) {
    969                         System.err.println(tr("Broken tagging preset \"{0}-{1}\" - Java method given in ''values_from'' threw {2} ({3})", key, text,
     969                        Main.error(tr("Broken tagging preset \"{0}-{1}\" - Java method given in ''values_from'' threw {2} ({3})", key, text,
    970970                                e.getClass().getName(), e.getMessage()));
    971971                    }
    … …  
    984984
    985985            if (display_array.length != value_array.length) {
    986                 System.err.println(tr("Broken tagging preset \"{0}-{1}\" - number of items in ''display_values'' must be the same as in ''values''", key, text));
     986                Main.error(tr("Broken tagging preset \"{0}-{1}\" - number of items in ''display_values'' must be the same as in ''values''", key, text));
    987987                display_array = value_array;
    988988            }
    989989
    990990            if (short_descriptions_array != null && short_descriptions_array.length != value_array.length) {
    991                 System.err.println(tr("Broken tagging preset \"{0}-{1}\" - number of items in ''short_descriptions'' must be the same as in ''values''", key, text));
     991                Main.error(tr("Broken tagging preset \"{0}-{1}\" - number of items in ''short_descriptions'' must be the same as in ''values''", key, text));
    992992                short_descriptions_array = null;
    993993            }

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

    r6198 r6248  
    187187                allPresets.addAll(readAll(source, validate));
    188188            } catch (IOException e) {
    189                 System.err.println(e.getClass().getName()+": "+e.getMessage());
    190                 System.err.println(source);
     189                Main.error(e);
     190                Main.error(source);
    191191                JOptionPane.showMessageDialog(
    192192                        Main.parent,
    … …  
    196196                        );
    197197            } catch (SAXException e) {
    198                 System.err.println(e.getClass().getName()+": "+e.getMessage());
    199                 System.err.println(source);
     198                Main.error(e);
     199                Main.error(source);
    200200                JOptionPane.showMessageDialog(
    201201                        Main.parent,

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

    r6084 r6248  
    291291        return list == null ? null : getFilteredItem(rowIndex);
    292292    }
    293 
    294     public void dump() {
    295         System.out.println("---------------------------------");
    296         for (AutoCompletionListItem item: list) {
    297             System.out.println(item);
    298         }
    299         System.out.println("---------------------------------");
    300     }
    301293}

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

    r6068 r6248  
    1212import java.util.Set;
    1313
     14import org.openstreetmap.josm.Main;
    1415import org.openstreetmap.josm.data.osm.DataSet;
    1516import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    156157                            presetTagCache.putAll(ki.key, ki.getValues());
    157158                        } catch (NullPointerException e) {
    158                             System.err.println(p+": Unable to cache "+ki);
     159                            Main.error(p+": Unable to cache "+ki);
    159160                        }
    160161                    }

  • trunk/src/org/openstreetmap/josm/gui/util/GuiHelper.java

    r6232 r6248  
    22package org.openstreetmap.josm.gui.util;
    33
     4import static org.openstreetmap.josm.tools.I18n.tr;
     5
    46import java.awt.BasicStroke;
    5 import static org.openstreetmap.josm.tools.I18n.tr;
    6 
    77import java.awt.Component;
    88import java.awt.Container;
    … …  
    196196                }
    197197            } catch (NumberFormatException ex) {
    198                 System.err.println("Error in stroke preference format: "+code);
     198                Main.error("Error in stroke preference format: "+code);
    199199                dash = new float[]{5.0f};
    200200            }
    201201            if (sumAbs < 1e-1) {
    202                 System.err.println("Error in stroke dash fomat (all zeros): "+code);
     202                Main.error("Error in stroke dash fomat (all zeros): "+code);
    203203                return new BasicStroke(w);
    204204            }

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

    r5899 r6248  
    44import javax.swing.text.JTextComponent;
    55
    6 import org.openstreetmap.josm.gui.widgets.JosmTextField;
     6import org.openstreetmap.josm.Main;
    77import org.openstreetmap.josm.tools.Utils;
    88
    … …  
    3939            }
    4040        } catch (Exception e) {
    41             System.err.println(e.getClass().getName()+": "+e.getMessage());
     41            Main.error(e);
    4242        } finally {
    4343            this.validator = validator;

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

    r5927 r6248  
    1010import javax.swing.text.Document;
    1111import javax.swing.text.JTextComponent;
     12
     13import org.openstreetmap.josm.Main;
    1214
    1315/**
    … …  
    103105                        pasteAction.actionPerformed(e);
    104106                    } catch (NullPointerException npe) {
    105                         System.err.println("NullPointerException occured because of JDK bug 6322854. "
     107                        Main.error("NullPointerException occured because of JDK bug 6322854. "
    106108                                +"Copy/Paste operation has not been performed. Please complain to Oracle: "+
    107109                                "http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6322854");

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

    r5123 r6248  
    1010import java.util.Map;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.data.osm.Changeset;
    1314import org.openstreetmap.josm.data.osm.DataSet;
    … …  
    101102                }
    102103                if (n.isDeleted()) {
    103                     System.out.println(tr("Deleted node {0} is part of way {1}", id, w.getId()));
     104                    Main.info(tr("Deleted node {0} is part of way {1}", id, w.getId()));
    104105                } else {
    105106                    wayNodes.add(n);
    … …  
    108109            w.setNodes(wayNodes);
    109110            if (w.hasIncompleteNodes()) {
    110                   System.out.println(tr("Way {0} with {1} nodes has incomplete nodes because at least one node was missing in the loaded data.",
     111                Main.info(tr("Way {0} with {1} nodes has incomplete nodes because at least one node was missing in the loaded data.",
    111112                          externalWayId, w.getNodesCount()));
    112113            }
    … …  
    174175                }
    175176                if (primitive.isDeleted()) {
    176                     System.out.println(tr("Deleted member {0} is used by relation {1}", primitive.getId(), relation.getId()));
     177                    Main.info(tr("Deleted member {0} is used by relation {1}", primitive.getId(), relation.getId()));
    177178                } else {
    178179                    relationMembers.add(new RelationMember(rm.getRole(), primitive));

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

    r6102 r6248  
    1414
    1515import org.openstreetmap.josm.Main;
     16import org.openstreetmap.josm.tools.Utils;
    1617
    1718/**
    … …  
    115116            new RandomAccessFile(data, "r").readFully(bytes);
    116117            return bytes;
    117         } catch(Exception e) {
    118             System.out.println(e.getMessage());
     118        } catch (Exception e) {
     119            Main.warn(e);
    119120        }
    120121        return null;
    … …  
    130131        try {
    131132            File f = getPath(ident);
    132             if(f.exists()) {
     133            if (f.exists()) {
    133134                f.delete();
    134135            }
    135136            // rws also updates the file meta-data, i.e. last mod time
    136             new RandomAccessFile(f, "rws").write(data);
    137         } catch(Exception e){
    138             System.out.println(e.getMessage());
     137            RandomAccessFile raf = new RandomAccessFile(f, "rws");
     138            try {
     139                raf.write(data);
     140            } finally {
     141                Utils.close(raf);
     142            }
     143        } catch (Exception e) {
     144            Main.warn(e);
    139145        }
    140146
    … …  
    164170            }
    165171            return ImageIO.read(img);
    166         } catch(Exception e) {
    167             System.out.println(e.getMessage());
     172        } catch (Exception e) {
     173            Main.warn(e);
    168174        }
    169175        return null;
    … …  
    176182     */
    177183    public void saveImg(String ident, BufferedImage image) {
    178         if(!enabled) return;
     184        if (!enabled) return;
    179185        try {
    180186            ImageIO.write(image, "png", getPath(ident, "png"));
    181         } catch(Exception e){
    182             System.out.println(e.getMessage());
     187        } catch (Exception e) {
     188            Main.warn(e);
    183189        }
    184190

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

    r6070 r6248  
    88import java.util.HashMap;
    99import java.util.List;
     10
     11import org.openstreetmap.josm.Main;
    1012
    1113/**
    … …  
    120122            int n = Integer.parseInt(v);
    121123            if (n <= 0) {
    122                 System.err.println(tr("Warning: illegal value of attribute ''{0}'' of element ''{1}'' in server capabilities. Got ''{2}''", "changesets", "maximum_elements", n ));
     124                Main.warn(tr("Illegal value of attribute ''{0}'' of element ''{1}'' in server capabilities. Got ''{2}''", "changesets", "maximum_elements", n ));
    123125                return -1;
    124126            }
    125127            return n;
    126         } catch(NumberFormatException e) {
    127             System.err.println(tr("Warning: illegal value of attribute ''{0}'' of element ''{1}'' in server capabilities. Got ''{2}''", "changesets", "maximum_elements", v ));
     128        } catch (NumberFormatException e) {
     129            Main.warn(tr("Illegal value of attribute ''{0}'' of element ''{1}'' in server capabilities. Got ''{2}''", "changesets", "maximum_elements", v ));
    128130            return -1;
    129131        }

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

    r5266 r6248  
    1111import java.util.regex.Matcher;
    1212import java.util.regex.Pattern;
     13
     14import org.openstreetmap.josm.Main;
    1315
    1416/**
    … …  
    8183                closedOn = formatter.parse(m.group(2));
    8284            } catch(ParseException ex) {
    83                 System.err.println(tr("Failed to parse date ''{0}'' replied by server.", m.group(2)));
     85                Main.error(tr("Failed to parse date ''{0}'' replied by server.", m.group(2)));
    8486                ex.printStackTrace();
    8587            }
    8688        } else {
    87             System.err.println(tr("Unexpected format of error header for conflict in changeset update. Got ''{0}''", errorHeader));
     89            Main.error(tr("Unexpected format of error header for conflict in changeset update. Got ''{0}''", errorHeader));
    8890        }
    8991    }

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

    r6087 r6248  
    77import java.net.InetSocketAddress;
    88import java.net.Proxy;
     9import java.net.Proxy.Type;
    910import java.net.ProxySelector;
    1011import java.net.SocketAddress;
    1112import java.net.URI;
    12 import java.net.Proxy.Type;
    1313import java.util.Collections;
    1414import java.util.List;
    … …  
    7979            port = Integer.parseInt(value);
    8080        } catch (NumberFormatException e) {
    81             System.err.println(tr("Unexpected format for port number in in preference ''{0}''. Got ''{1}''.", property, value));
    82             System.err.println(tr("The proxy will not be used."));
     81            Main.error(tr("Unexpected format for port number in in preference ''{0}''. Got ''{1}''.", property, value));
     82            Main.error(tr("The proxy will not be used."));
    8383            return 0;
    8484        }
    8585        if (port <= 0 || port >  65535) {
    86             System.err.println(tr("Illegal port number in preference ''{0}''. Got {1}.", property, port));
    87             System.err.println(tr("The proxy will not be used."));
     86            Main.error(tr("Illegal port number in preference ''{0}''. Got {1}.", property, port));
     87            Main.error(tr("The proxy will not be used."));
    8888            return 0;
    8989        }
    … …  
    102102            proxyPolicy= ProxyPolicy.fromName(value);
    103103            if (proxyPolicy == null) {
    104                 System.err.println(tr("Warning: unexpected value for preference ''{0}'' found. Got ''{1}''. Will use no proxy.", ProxyPreferencesPanel.PROXY_POLICY, value));
     104                Main.warn(tr("Unexpected value for preference ''{0}'' found. Got ''{1}''. Will use no proxy.", ProxyPreferencesPanel.PROXY_POLICY, value));
    105105                proxyPolicy = ProxyPolicy.NO_PROXY;
    106106            }
    … …  
    113113            httpProxySocketAddress = null;
    114114            if (proxyPolicy.equals(ProxyPolicy.USE_HTTP_PROXY)) {
    115                 System.err.println(tr("Warning: Unexpected parameters for HTTP proxy. Got host ''{0}'' and port ''{1}''.", host, port));
    116                 System.err.println(tr("The proxy will not be used."));
     115                Main.warn(tr("Unexpected parameters for HTTP proxy. Got host ''{0}'' and port ''{1}''.", host, port));
     116                Main.warn(tr("The proxy will not be used."));
    117117            }
    118118        }
    … …  
    125125            socksProxySocketAddress = null;
    126126            if (proxyPolicy.equals(ProxyPolicy.USE_SOCKS_PROXY)) {
    127                 System.err.println(tr("Warning: Unexpected parameters for SOCKS proxy. Got host ''{0}'' and port ''{1}''.", host, port));
    128                 System.err.println(tr("The proxy will not be used."));
     127                Main.warn(tr("Unexpected parameters for SOCKS proxy. Got host ''{0}'' and port ''{1}''.", host, port));
     128                Main.warn(tr("The proxy will not be used."));
    129129            }
    130130        }
    … …  
    133133    @Override
    134134    public void connectFailed(URI uri, SocketAddress sa, IOException ioe) {
    135         // Just log something. The network stack will also throw an exception which will be caught
    136         // somewhere else
     135        // Just log something. The network stack will also throw an exception which will be caught somewhere else
    137136        //
    138         System.out.println(tr("Error: Connection to proxy ''{0}'' for URI ''{1}'' failed. Exception was: {2}", sa.toString(), uri.toString(), ioe.toString()));
     137        Main.error(tr("Connection to proxy ''{0}'' for URI ''{1}'' failed. Exception was: {2}", sa.toString(), uri.toString(), ioe.toString()));
    139138    }
    140139
    … …  
    145144        case USE_SYSTEM_SETTINGS:
    146145            if (!JVM_WILL_USE_SYSTEM_PROXIES) {
    147                 System.err.println(tr("Warning: the JVM is not configured to lookup proxies from the system settings. The property ''java.net.useSystemProxies'' was missing at startup time.  Will not use a proxy."));
     146                Main.warn(tr("The JVM is not configured to lookup proxies from the system settings. The property ''java.net.useSystemProxies'' was missing at startup time.  Will not use a proxy."));
    148147                return Collections.singletonList(Proxy.NO_PROXY);
    149148            }

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

    r6102 r6248  
    6363    public boolean importDataHandleExceptions(File f, ProgressMonitor progressMonitor) {
    6464        try {
    65             System.out.println("Open file: " + f.getAbsolutePath() + " (" + f.length() + " bytes)");
     65            Main.info("Open file: " + f.getAbsolutePath() + " (" + f.length() + " bytes)");
    6666            importData(f, progressMonitor);
    6767            return true;
    … …  
    7979    public boolean importDataHandleExceptions(List<File> files, ProgressMonitor progressMonitor) {
    8080        try {
    81             System.out.println("Open "+files.size()+" files");
     81            Main.info("Open "+files.size()+" files");
    8282            importData(files, progressMonitor);
    8383            return true;

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

    r6148 r6248  
    150150            }
    151151        } catch (Exception e) {
    152             if(file.getName().endsWith(".zip")) {
    153                 System.err.println(tr("Warning: failed to open file with extension ''{2}'' and namepart ''{3}'' in zip file ''{0}''. Exception was: {1}",
     152            if (file.getName().endsWith(".zip")) {
     153                Main.warn(tr("Failed to open file with extension ''{2}'' and namepart ''{3}'' in zip file ''{0}''. Exception was: {1}",
    154154                        file.getName(), e.toString(), extension, namepart));
    155155            }
    … …  
    257257                {Long.toString(System.currentTimeMillis()), localFile.toString()}));
    258258            } else {
    259                 System.out.println(tr("Failed to rename file {0} to {1}.",
     259                Main.warn(tr("Failed to rename file {0} to {1}.",
    260260                destDirFile.getPath(), localFile.getPath()));
    261261            }
    262262        } catch (IOException e) {
    263263            if (age >= maxTime*1000 && age < maxTime*1000*2) {
    264                 System.out.println(tr("Failed to load {0}, use cached file and retry next time: {1}",
    265                 url, e));
     264                Main.warn(tr("Failed to load {0}, use cached file and retry next time: {1}", url, e));
    266265                return localFile;
    267266            } else {
    … …  
    319318                    throw new IOException(msg);
    320319                }
    321                 System.out.println(tr("Download redirected to ''{0}''", downloadUrl));
     320                Main.info(tr("Download redirected to ''{0}''", downloadUrl));
    322321                break;
    323322            default:

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

    r6070 r6248  
    4949 *    reader.parseOsm();
    5050 *    if (!reader.getMissingPrimitives().isEmpty()) {
    51  *        System.out.println("There are missing primitives: " + reader.getMissingPrimitives());
     51 *        Main.info("There are missing primitives: " + reader.getMissingPrimitives());
    5252 *    }
    5353 *    if (!reader.getSkippedWays().isEmpty()) {
    54  *       System.out.println("There are skipped ways: " + reader.getMissingPrimitives());
     54 *       Main.info("There are skipped ways: " + reader.getMissingPrimitives());
    5555 *    }
    5656 * </pre>
    … …  
    489489            } catch (OsmApiException e) {
    490490                if (e.getResponseCode() == HttpURLConnection.HTTP_NOT_FOUND) {
    491                     System.out.println(tr("Server replied with response code 404, retrying with an individual request for each object."));
     491                    Main.info(tr("Server replied with response code 404, retrying with an individual request for each object."));
    492492                    return singleGetIdPackage(type, pkg, progressMonitor);
    493493                } else {
    … …  
    567567                } catch (OsmApiException e) {
    568568                    if (e.getResponseCode() == HttpURLConnection.HTTP_NOT_FOUND) {
    569                         System.out.println(tr("Server replied with response code 404 for id {0}. Skipping.", Long.toString(id)));
     569                        Main.info(tr("Server replied with response code 404 for id {0}. Skipping.", Long.toString(id)));
    570570                        result.missingPrimitives.add(new SimplePrimitiveId(id, type));
    571571                    } else {

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

    r6087 r6248  
    247247                byte[] chb = chkstrings[0].getBytes();
    248248                int chk=0;
    249                 for(int i = 1; i < chb.length; i++) {
     249                for (int i = 1; i < chb.length; i++) {
    250250                    chk ^= chb[i];
    251251                }
    252                 if(Integer.parseInt(chkstrings[1].substring(0,2),16) != chk) {
    253                     //System.out.println("Checksum error");
     252                if (Integer.parseInt(chkstrings[1].substring(0,2),16) != chk) {
    254253                    ps.checksum_errors++;
    255254                    ps.p_Wp=null;
    … …  
    447446            return true;
    448447
    449         } catch(RuntimeException x) {
     448        } catch (RuntimeException x) {
    450449            // out of bounds and such
    451             // x.printStackTrace();
    452             // System.out.println("Malformed line: "+s.toString().trim());
    453450            ps.malformed++;
    454451            ps.p_Wp=null;

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

    r6070 r6248  
    229229                version = "0.6";
    230230            } else {
    231                 System.err.println(tr("This version of JOSM is incompatible with the configured server."));
    232                 System.err.println(tr("It supports protocol version 0.6, while the server says it supports {0} to {1}.",
     231                Main.error(tr("This version of JOSM is incompatible with the configured server."));
     232                Main.error(tr("It supports protocol version 0.6, while the server says it supports {0} to {1}.",
    233233                        capabilities.get("version", "minimum"), capabilities.get("version", "maximum")));
    234234                initialized = false; // FIXME gets overridden by next assignment
    … …  
    256256                for (Layer l : Main.map.mapView.getLayersOfType(ImageryLayer.class)) {
    257257                    if (((ImageryLayer) l).getInfo().isBlacklisted()) {
    258                         System.out.println(tr("Removed layer {0} because it is not allowed by the configured API.", l.getName()));
     258                        Main.info(tr("Removed layer {0} because it is not allowed by the configured API.", l.getName()));
    259259                        Main.main.removeLayer(l);
    260260                    }
    … …  
    548548            } catch (InterruptedException ex) {}
    549549        }
    550         System.out.println(tr("OK - trying again."));
     550        Main.info(tr("OK - trying again."));
    551551    }
    552552
    … …  
    628628
    629629                activeConnection.connect();
    630                 System.out.println(activeConnection.getResponseMessage());
     630                Main.info(activeConnection.getResponseMessage());
    631631                int retCode = activeConnection.getResponseCode();
    632632
    … …  
    634634                    if (retries-- > 0) {
    635635                        sleepAndListen(retries, monitor);
    636                         System.out.println(tr("Starting retry {0} of {1}.", getMaxRetries() - retries,getMaxRetries()));
     636                        Main.info(tr("Starting retry {0} of {1}.", getMaxRetries() - retries,getMaxRetries()));
    637637                        continue;
    638638                    }
    … …  
    666666                if (activeConnection.getHeaderField("Error") != null) {
    667667                    errorHeader = activeConnection.getHeaderField("Error");
    668                     System.err.println("Error header: " + errorHeader);
     668                    Main.error("Error header: " + errorHeader);
    669669                } else if (retCode != 200 && responseBody.length()>0) {
    670                     System.err.println("Error body: " + responseBody);
     670                    Main.error("Error body: " + responseBody);
    671671                }
    672672                activeConnection.disconnect();

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

    r6201 r6248  
    1313import javax.xml.parsers.SAXParserFactory;
    1414
     15import org.openstreetmap.josm.Main;
    1516import org.openstreetmap.josm.data.osm.ChangesetDataSet;
    1617import org.openstreetmap.josm.data.osm.ChangesetDataSet.ChangesetModificationType;
    … …  
    5758                currentModificationType = ChangesetModificationType.DELETED;
    5859            } else {
    59                 System.err.println(tr("Warning: unsupported start element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
     60                Main.warn(tr("Unsupported start element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
    6061            }
    6162        }
    … …  
    8586                // do nothing
    8687            } else {
    87                 System.err.println(tr("Warning: unsupported end element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
     88                Main.warn(tr("Unsupported end element ''{0}'' in changeset content at position ({1},{2}). Skipping.", qName, locator.getLineNumber(), locator.getColumnNumber()));
    8889            }
    8990        }

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

    r5881 r6248  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
     6import java.net.Authenticator.RequestorType;
    67import java.net.HttpURLConnection;
    7 import java.net.Authenticator.RequestorType;
    88import java.nio.ByteBuffer;
    99import java.nio.CharBuffer;
    … …  
    1919import org.openstreetmap.josm.gui.preferences.server.OAuthAccessTokenHolder;
    2020import org.openstreetmap.josm.io.auth.CredentialsAgentException;
     21import org.openstreetmap.josm.io.auth.CredentialsAgentResponse;
    2122import org.openstreetmap.josm.io.auth.CredentialsManager;
    22 import org.openstreetmap.josm.io.auth.CredentialsAgentResponse;
    2323import org.openstreetmap.josm.tools.Base64;
    2424
    … …  
    132132            addOAuthAuthorizationHeader(connection);
    133133        } else {
    134             String msg = tr("Warning: unexpected value for preference ''{0}''. Got ''{1}''.", "osm-server.auth-method", authMethod);
    135             System.err.println(msg);
     134            String msg = tr("Unexpected value for preference ''{0}''. Got ''{1}''.", "osm-server.auth-method", authMethod);
     135            Main.warn(msg);
    136136            throw new OsmTransferException(msg);
    137137        }

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

    r6093 r6248  
    1818import javax.xml.stream.XMLStreamReader;
    1919
     20import org.openstreetmap.josm.Main;
    2021import org.openstreetmap.josm.data.Bounds;
    2122import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    171172                Bounds copy = new Bounds(bounds);
    172173                bounds.normalize();
    173                 System.out.println("Bbox " + copy + " is out of the world, normalized to " + bounds);
     174                Main.info("Bbox " + copy + " is out of the world, normalized to " + bounds);
    174175            }
    175176            DataSource src = new DataSource(bounds, origin);
    … …  
    233234        }
    234235        if (w.isDeleted() && !nodeIds.isEmpty()) {
    235             System.out.println(tr("Deleted way {0} contains nodes", w.getUniqueId()));
     236            Main.info(tr("Deleted way {0} contains nodes", w.getUniqueId()));
    236237            nodeIds = new ArrayList<Long>();
    237238        }
    … …  
    280281        }
    281282        if (r.isDeleted() && !members.isEmpty()) {
    282             System.out.println(tr("Deleted relation {0} contains members", r.getUniqueId()));
     283            Main.info(tr("Deleted relation {0} contains members", r.getUniqueId()));
    283284            members = new ArrayList<RelationMemberData>();
    284285        }
    … …  
    356357    protected void parseUnknown(boolean printWarning) throws XMLStreamException {
    357358        if (printWarning) {
    358             System.out.println(tr("Undefined element ''{0}'' found in input stream. Skipping.", parser.getLocalName()));
     359            Main.info(tr("Undefined element ''{0}'' found in input stream. Skipping.", parser.getLocalName()));
    359360        }
    360361        while (true) {
    … …  
    445446                    throwException(tr("Illegal value for attribute ''version'' on OSM primitive with ID {0}. Got {1}.", Long.toString(current.getUniqueId()), versionString));
    446447                } else if (version < 0 && current.getUniqueId() <= 0) {
    447                     System.out.println(tr("WARNING: Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 0, "0.6"));
     448                    Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 0, "0.6"));
    448449                    version = 0;
    449450                }
    450451            } else if (ds.getVersion().equals("0.5")) {
    451452                if (version <= 0 && current.getUniqueId() > 0) {
    452                     System.out.println(tr("WARNING: Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
     453                    Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
    453454                    version = 1;
    454455                } else if (version < 0 && current.getUniqueId() <= 0) {
    455                     System.out.println(tr("WARNING: Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 0, "0.5"));
     456                    Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 0, "0.5"));
    456457                    version = 0;
    457458                }
    … …  
    467468            } else if (current.getUniqueId() > 0 && ds.getVersion() != null && ds.getVersion().equals("0.5")) {
    468469                // default version in 0.5 files for existing primitives
    469                 System.out.println(tr("WARNING: Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
     470                Main.warn(tr("Normalizing value of attribute ''version'' of element {0} to {2}, API version is ''{3}''. Got {1}.", current.getUniqueId(), version, 1, "0.5"));
    470471                version= 1;
    471472            } else if (current.getUniqueId() <= 0 && ds.getVersion() != null && ds.getVersion().equals("0.5")) {
    … …  
    496497                if (current.getUniqueId() <= 0) {
    497498                    // for a new primitive we just log a warning
    498                     System.out.println(tr("Illegal value for attribute ''changeset'' on new object {1}. Got {0}. Resetting to 0.", v, current.getUniqueId()));
     499                    Main.info(tr("Illegal value for attribute ''changeset'' on new object {1}. Got {0}. Resetting to 0.", v, current.getUniqueId()));
    499500                    current.setChangesetId(0);
    500501                } else {
    … …  
    506507                if (current.getUniqueId() <= 0) {
    507508                    // for a new primitive we just log a warning
    508                     System.out.println(tr("Illegal value for attribute ''changeset'' on new object {1}. Got {0}. Resetting to 0.", v, current.getUniqueId()));
     509                    Main.info(tr("Illegal value for attribute ''changeset'' on new object {1}. Got {0}. Resetting to 0.", v, current.getUniqueId()));
    509510                    current.setChangesetId(0);
    510511                } else {

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

    r6246 r6248  
    107107
    108108            try {
    109                 System.out.println("GET " + url);
     109                Main.info("GET " + url);
    110110                activeConnection.connect();
    111111            } catch (Exception e) {

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

    r4745 r6248  
    3030        String urlstring = url.toExternalForm();
    3131
    32         System.out.println("Grabbing HTML " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
     32        Main.info("Grabbing HTML " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
    3333
    3434        ArrayList<String> cmdParams = new ArrayList<String>();
    3535        StringTokenizer st = new StringTokenizer(MessageFormat.format(PROP_BROWSER.get(), urlstring));
    36         while( st.hasMoreTokens() ) {
     36        while (st.hasMoreTokens()) {
    3737            cmdParams.add(st.nextToken());
    3838        }
    … …  
    4343        try {
    4444            browser = builder.start();
    45         } catch(IOException ioe) {
     45        } catch (IOException ioe) {
    4646            throw new IOException( "Could not start browser. Please check that the executable path is correct.\n" + ioe.getMessage() );
    4747        }

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

    r6087 r6248  
    158158
    159159    protected BufferedImage grab(WMSRequest request, URL url, int attempt) throws IOException, OsmTransferException {
    160         System.out.println("Grabbing WMS " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
     160        Main.info("Grabbing WMS " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
    161161
    162162        HttpURLConnection conn = Utils.openHttpConnection(url);

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

    r6106 r6248  
    2020import javax.xml.parsers.DocumentBuilderFactory;
    2121
     22import org.openstreetmap.josm.Main;
    2223import org.openstreetmap.josm.data.Bounds;
    2324import org.openstreetmap.josm.data.imagery.ImageryInfo;
    … …  
    127128        }
    128129
    129         System.out.println("GET " + getCapabilitiesUrl.toString());
     130        Main.info("GET " + getCapabilitiesUrl.toString());
    130131        URLConnection openConnection = Utils.openHttpConnection(getCapabilitiesUrl);
    131132        InputStream inputStream = openConnection.getInputStream();
    … …  
    139140        String incomingData = ba.toString();
    140141
    141         //System.out.println("WMS capabilities:\n"+incomingData+"\n");
    142142        try {
    143143            DocumentBuilderFactory builderFactory = DocumentBuilderFactory.newInstance();
    … …  
    149149                @Override
    150150                public InputSource resolveEntity(String publicId, String systemId) throws SAXException, IOException {
    151                     System.out.println("Ignoring DTD " + publicId + ", " + systemId);
     151                    Main.info("Ignoring DTD " + publicId + ", " + systemId);
    152152                    return new InputSource(new StringReader(""));
    153153                }
    … …  
    175175                String baseURL = child.getAttribute("xlink:href");
    176176                if (baseURL != null && !baseURL.equals(serviceUrlStr)) {
    177                     System.out.println("GetCapabilities specifies a different service URL: " + baseURL);
     177                    Main.info("GetCapabilities specifies a different service URL: " + baseURL);
    178178                    serviceUrl = new URL(baseURL);
    179179                }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpServer.java

    r6084 r6248  
    66import java.io.IOException;
    77import java.net.BindException;
     8import java.net.InetAddress;
    89import java.net.ServerSocket;
    910import java.net.Socket;
    1011import java.net.SocketException;
    11 import java.net.InetAddress;
    1212
    1313import org.openstreetmap.josm.Main;
    … …  
    3737            instance.start();
    3838        } catch (BindException ex) {
    39             Main.warn(marktr("Warning: Cannot start remotecontrol server on port {0}: {1}"),
     39            Main.warn(marktr("Cannot start remotecontrol server on port {0}: {1}"),
    4040                    Integer.toString(port), ex.getLocalizedMessage());
    4141        } catch (IOException ioe) {

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r6223 r6248  
    2222import java.util.regex.Pattern;
    2323
     24import org.openstreetmap.josm.Main;
    2425import org.openstreetmap.josm.gui.help.HelpUtil;
    2526import org.openstreetmap.josm.io.remotecontrol.handler.AddNodeHandler;
    … …  
    110111        String commandWithSlash = "/" + command;
    111112        if (handlers.get(commandWithSlash) != null) {
    112             System.out.println("RemoteControl: ignoring duplicate command " + command
     113            Main.info("RemoteControl: ignoring duplicate command " + command
    113114                    + " with handler " + handler.getName());
    114115        } else {
    115116            if (!silent) {
    116                 System.out.println("RemoteControl: adding command \"" +
     117                Main.info("RemoteControl: adding command \"" +
    117118                    command + "\" (handled by " + handler.getSimpleName() + ")");
    118119            }
    … …  
    151152                return;
    152153            }
    153             System.out.println("RemoteControl received: " + get);
     154            Main.info("RemoteControl received: " + get);
    154155
    155156            StringTokenizer st = new StringTokenizer(get);

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddNodeHandler.java

    r6091 r6248  
    66import java.awt.Point;
    77import java.util.HashMap;
     8
    89import org.openstreetmap.josm.Main;
    910import org.openstreetmap.josm.actions.AutoScaleAction;
    … …  
    1516import org.openstreetmap.josm.io.remotecontrol.AddTagsDialog;
    1617import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
    17 import org.openstreetmap.josm.io.remotecontrol.handler.RequestHandler.RequestHandlerBadRequestException;
    1818
    1919/**
    … …  
    7777
    7878        // Parse the arguments
    79         System.out.println("Adding node at (" + lat + ", " + lon + ")");
     79        Main.info("Adding node at (" + lat + ", " + lon + ")");
    8080
    8181        // Create a new node

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/ImageryHandler.java

    r6091 r6248  
    22package org.openstreetmap.josm.io.remotecontrol.handler;
    33
    4 import java.util.Arrays;
    54import static org.openstreetmap.josm.tools.I18n.tr;
    65
     6import java.util.Arrays;
    77import java.util.HashMap;
    88
    … …  
    6161                imgInfo.setDefaultMinZoom(Integer.parseInt(min_zoom));
    6262            } catch (NumberFormatException e) {
    63                 System.err.println("NumberFormatException ("+e.getMessage()+")");
     63                Main.error(e);
    6464            }
    6565        }
    … …  
    6969                imgInfo.setDefaultMaxZoom(Integer.parseInt(max_zoom));
    7070            } catch (NumberFormatException e) {
    71                 System.err.println("NumberFormatException ("+e.getMessage()+")");
     71                Main.error(e);
    7272            }
    7373        }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/ImportHandler.java

    r6143 r6248  
    3535            }
    3636        } catch (Exception ex) {
    37             System.out.println("RemoteControl: Error parsing import remote control request:");
     37            Main.warn("RemoteControl: Error parsing import remote control request:");
    3838            ex.printStackTrace();
    3939            throw new RequestHandlerErrorException();

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    r6203 r6248  
    9797            boolean newLayer = isLoadInNewLayer();
    9898
    99             if(command.equals(myCommand))
    100             {
    101                 if (!PermissionPrefWithDefault.LOAD_DATA.isAllowed())
    102                 {
    103                     System.out.println("RemoteControl: download forbidden by preferences");
    104                 }
    105                 else
    106                 {
     99            if (command.equals(myCommand)) {
     100                if (!PermissionPrefWithDefault.LOAD_DATA.isAllowed()) {
     101                    Main.info("RemoteControl: download forbidden by preferences");
     102                } else {
    107103                    Area toDownload = null;
    108104                    if (!newLayer) {
    … …  
    116112                            toDownload = new Area(new Rectangle2D.Double(minlon,minlat,maxlon-minlon,maxlat-minlat));
    117113                            toDownload.subtract(present);
    118                             if (!toDownload.isEmpty())
    119                             {
     114                            if (!toDownload.isEmpty()) {
    120115                                // the result might not be a rectangle (L shaped etc)
    121116                                Rectangle2D downloadBounds = toDownload.getBounds2D();
    … …  
    127122                        }
    128123                    }
    129                     if (toDownload != null && toDownload.isEmpty())
    130                     {
    131                         System.out.println("RemoteControl: no download necessary");
    132                     }
    133                     else
    134                     {
     124                    if (toDownload != null && toDownload.isEmpty()) {
     125                        Main.info("RemoteControl: no download necessary");
     126                    } else {
    135127                        Future<?> future = osmTask.download(newLayer, new Bounds(minlat,minlon,maxlat,maxlon), null /* let the task manage the progress monitor */);
    136128                        Main.worker.submit(new PostDownloadHandler(osmTask, future));
    … …  
    139131            }
    140132        } catch (Exception ex) {
    141             System.out.println("RemoteControl: Error parsing load_and_zoom remote control request:");
     133            Main.warn("RemoteControl: Error parsing load_and_zoom remote control request:");
    142134            ex.printStackTrace();
    143135            throw new RequestHandlerErrorException();
    … …  
    262254                        relations.add(Long.parseLong(item.substring(3)));
    263255                    } else {
    264                         System.out.println("RemoteControl: invalid selection '"+item+"' ignored");
     256                        Main.warn("RemoteControl: invalid selection '"+item+"' ignored");
    265257                    }
    266258                } catch (NumberFormatException e) {
    267                     System.out.println("RemoteControl: invalid selection '"+item+"' ignored");
     259                    Main.warn("RemoteControl: invalid selection '"+item+"' ignored");
    268260                }
    269261            }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java

    r6091 r6248  
    5050    protected void handleRequest() throws RequestHandlerErrorException, RequestHandlerBadRequestException {
    5151        if (!PermissionPrefWithDefault.LOAD_DATA.isAllowed()) {
    52             System.out.println("RemoteControl: download forbidden by preferences");
     52            Main.info("RemoteControl: download forbidden by preferences");
    5353        }
    5454        if (!ps.isEmpty()) {
    … …  
    8888                ps.add(SimplePrimitiveId.fromString(i));
    8989            } catch (IllegalArgumentException e) {
    90                 System.out.println("RemoteControl: invalid selection '"+i+"' ignored");
     90                Main.warn("RemoteControl: invalid selection '"+i+"' ignored");
    9191            }
    9292        }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java

    r6091 r6248  
    137137            if (!Main.pref.getBoolean(permissionPref.pref, permissionPref.defaultVal)) {
    138138                String err = MessageFormat.format("RemoteControl: ''{0}'' forbidden by preferences", myCommand);
    139                 System.out.println(err);
     139                Main.info(err);
    140140                throw new RequestHandlerForbiddenException(err);
    141141            }
    … …  
    211211            if ((value == null) || (value.length() == 0)) {
    212212                error = true;
    213                 System.out.println("'" + myCommand + "' remote control request must have '" + key + "' parameter");
     213                Main.warn("'" + myCommand + "' remote control request must have '" + key + "' parameter");
    214214                missingKeys.add(key);
    215215            }

  • trunk/src/org/openstreetmap/josm/plugins/PluginDownloadTask.java

    r6073 r6248  
    114114        try {
    115115            if (pi.downloadlink == null) {
    116                 String msg = tr("Warning: Cannot download plugin ''{0}''. Its download link is not known. Skipping download.", pi.name);
    117                 System.err.println(msg);
     116                String msg = tr("Cannot download plugin ''{0}''. Its download link is not known. Skipping download.", pi.name);
     117                Main.warn(msg);
    118118                throw new PluginDownloadException(msg);
    119119            }
    … …  
    128128                out.write(buffer, 0, read);
    129129            }
    130         } catch(MalformedURLException e) {
    131             String msg = tr("Warning: Cannot download plugin ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", pi.name, pi.downloadlink);
    132             System.err.println(msg);
     130        } catch (MalformedURLException e) {
     131            String msg = tr("Cannot download plugin ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", pi.name, pi.downloadlink);
     132            Main.warn(msg);
    133133            throw new PluginDownloadException(msg);
    134134        } catch (IOException e) {

  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r6090 r6248  
    6666 * PluginHandler is basically a collection of static utility functions used to bootstrap
    6767 * and manage the loaded plugins.
    68  *
     68 * @since 1326
    6969 */
    7070public class PluginHandler {
    7171
    7272    /**
    73      * deprecated plugins that are removed on start
     73     * Deprecated plugins that are removed on start
    7474     */
    7575    public final static Collection<DeprecatedPlugin> DEPRECATED_PLUGINS;
    … …  
    117117    }
    118118
     119    /**
     120     * Description of a deprecated plugin
     121     */
    119122    public static class DeprecatedPlugin implements Comparable<DeprecatedPlugin> {
    120         public String name;
    121         // short explanation, can be null
    122         public String reason;
    123         // migration, can be null
    124         private Runnable migration;
    125 
     123        /** Plugin name */
     124        public final String name;
     125        /** Short explanation about deprecation, can be {@code null} */
     126        public final String reason;
     127        /** Code to run to perform migration, can be {@code null} */
     128        private final Runnable migration;
     129
     130        /**
     131         * Constructs a new {@code DeprecatedPlugin}.
     132         * @param name The plugin name
     133         */
    126134        public DeprecatedPlugin(String name) {
    127             this.name = name;
    128         }
    129 
     135            this(name, null, null);
     136        }
     137
     138        /**
     139         * Constructs a new {@code DeprecatedPlugin} with a given reason.
     140         * @param name The plugin name
     141         * @param reason The reason about deprecation
     142         */
    130143        public DeprecatedPlugin(String name, String reason) {
    131             this.name = name;
    132             this.reason = reason;
    133         }
    134 
     144            this(name, reason, null);
     145        }
     146
     147        /**
     148         * Constructs a new {@code DeprecatedPlugin}.
     149         * @param name The plugin name
     150         * @param reason The reason about deprecation
     151         * @param migration The code to run to perform migration
     152         */
    135153        public DeprecatedPlugin(String name, String reason, Runnable migration) {
    136154            this.name = name;
    … …  
    139157        }
    140158
     159        /**
     160         * Performs migration.
     161         */
    141162        public void migrate() {
    142163            if (migration != null) {
    … …  
    151172    }
    152173
     174    /**
     175     * List of unmaintained plugins. Not really up-to-date as the vast majority of plugins are not really maintained after a few months, sadly...
     176     */
    153177    final public static String [] UNMAINTAINED_PLUGINS = new String[] {"gpsbabelgui", "Intersect_way"};
    154178
    … …  
    319343        if (policy.equals("never")) {
    320344            if ("pluginmanager.version-based-update.policy".equals(togglePreferenceKey)) {
    321                 System.out.println(tr("Skipping plugin update after JOSM upgrade. Automatic update at startup is disabled."));
     345                Main.info(tr("Skipping plugin update after JOSM upgrade. Automatic update at startup is disabled."));
    322346            } else if ("pluginmanager.time-based-update.policy".equals(togglePreferenceKey)) {
    323                 System.out.println(tr("Skipping plugin update after elapsed update interval. Automatic update at startup is disabled."));
     347                Main.info(tr("Skipping plugin update after elapsed update interval. Automatic update at startup is disabled."));
    324348            }
    325349            return false;
    … …  
    328352        if (policy.equals("always")) {
    329353            if ("pluginmanager.version-based-update.policy".equals(togglePreferenceKey)) {
    330                 System.out.println(tr("Running plugin update after JOSM upgrade. Automatic update at startup is enabled."));
     354                Main.info(tr("Running plugin update after JOSM upgrade. Automatic update at startup is enabled."));
    331355            } else if ("pluginmanager.time-based-update.policy".equals(togglePreferenceKey)) {
    332                 System.out.println(tr("Running plugin update after elapsed update interval. Automatic update at startup is disabled."));
     356                Main.info(tr("Running plugin update after elapsed update interval. Automatic update at startup is disabled."));
    333357            }
    334358            return true;
    … …  
    336360
    337361        if (!policy.equals("ask")) {
    338             System.err.println(tr("Unexpected value ''{0}'' for preference ''{1}''. Assuming value ''ask''.", policy, togglePreferenceKey));
     362            Main.warn(tr("Unexpected value ''{0}'' for preference ''{1}''. Assuming value ''ask''.", policy, togglePreferenceKey));
    339363        }
    340364        int ret = HelpAwareOptionPane.showOptionDialog(
    … …  
    513537     * the class loader <code>pluginClassLoader</code>.
    514538     *
     539     * @param parent The parent component to be used for the displayed dialog
    515540     * @param plugin the plugin
    516541     * @param pluginClassLoader the plugin class loader
    … …  
    521546            Class<?> klass = plugin.loadClass(pluginClassLoader);
    522547            if (klass != null) {
    523                 System.out.println(tr("loading plugin ''{0}'' (version {1})", plugin.name, plugin.localversion));
     548                Main.info(tr("loading plugin ''{0}'' (version {1})", plugin.name, plugin.localversion));
    524549                PluginProxy pluginProxy = plugin.load(klass);
    525550                pluginList.add(pluginProxy);
    … …  
    528553            msg = null;
    529554        } catch (PluginException e) {
    530             System.err.println(e.getMessage());
     555            Main.error(e.getMessage());
    531556            Throwable cause = e.getCause();
    532557            if (cause != null) {
    533558                msg = cause.getLocalizedMessage();
    534559                if (msg != null) {
    535                     System.err.println("Cause: " + cause.getClass().getName()+": " + msg);
     560                    Main.error("Cause: " + cause.getClass().getName()+": " + msg);
    536561                } else {
    537562                    cause.printStackTrace();
    … …  
    545570            e.printStackTrace();
    546571        }
    547         if(msg != null && confirmDisablePlugin(parent, msg, plugin.name)) {
     572        if (msg != null && confirmDisablePlugin(parent, msg, plugin.name)) {
    548573            Main.pref.removeFromCollection("plugins", plugin.name);
    549574        }
    … …  
    554579     * memory.
    555580     *
     581     * @param parent The parent component to be used for the displayed dialog
    556582     * @param plugins the list of plugins
    557583     * @param monitor the progress monitor. Defaults to {@link NullProgressMonitor#INSTANCE} if null.
    … …  
    621647     * set to false.
    622648     *
     649     * @param parent The parent component to be used for the displayed dialog
    623650     * @param plugins the collection of plugins
    624651     * @param monitor the progress monitor. Defaults to {@link NullProgressMonitor#INSTANCE} if null.
    … …  
    698725     * messages.
    699726     *
     727     * @param parent The parent component to be used for the displayed dialog
    700728     * @param monitor the progress monitor. Defaults to {@link NullProgressMonitor#INSTANCE} if null.
    701729     * @return the set of plugins to load (as set of plugin names)
    … …  
    785813                    }
    786814                } catch (PluginException e) {
    787                     System.out.println(tr("Warning: failed to find plugin {0}", name));
     815                    Main.warn(tr("Failed to find plugin {0}", name));
    788816                    e.printStackTrace();
    789817                }
    … …  
    801829     * @throws IllegalArgumentException thrown if plugins is null
    802830     */
    803     public static List<PluginInformation>  updatePlugins(Component parent,
     831    public static List<PluginInformation> updatePlugins(Component parent,
    804832            List<PluginInformation> plugins, ProgressMonitor monitor)
    805833            throws IllegalArgumentException{
    … …  
    825853                allPlugins = task1.getAvailablePlugins();
    826854                plugins = buildListOfPluginsToLoad(parent,monitor.createSubTaskMonitor(1, false));
    827             } catch(ExecutionException e) {
    828                 System.out.println(tr("Warning: failed to download plugin information list"));
     855            } catch (ExecutionException e) {
     856                Main.warn(tr("Failed to download plugin information list"));
    829857                e.printStackTrace();
    830858                // don't abort in case of error, continue with downloading plugins below
    831             } catch(InterruptedException e) {
    832                 System.out.println(tr("Warning: failed to download plugin information list"));
     859            } catch (InterruptedException e) {
     860                Main.warn(tr("Failed to download plugin information list"));
    833861                e.printStackTrace();
    834862                // don't abort in case of error, continue with downloading plugins below
    … …  
    838866            //
    839867            Collection<PluginInformation> pluginsToUpdate = new ArrayList<PluginInformation>();
    840             for(PluginInformation pi: plugins) {
     868            for (PluginInformation pi: plugins) {
    841869                if (pi.isUpdateRequired()) {
    842870                    pluginsToUpdate.add(pi);
    … …  
    907935    /**
    908936     * Ask the user for confirmation that a plugin shall be disabled.
    909      *
     937     *
     938     * @param parent The parent component to be used for the displayed dialog
    910939     * @param reason the reason for disabling the plugin
    911940     * @param name the plugin name
    … …  
    940969    }
    941970
     971    /**
     972     * Returns the plugin of the specified name.
     973     * @param name The plugin name
     974     * @return The plugin of the specified name, if installed and loaded, or {@code null} otherwise.
     975     */
    942976    public static Object getPlugin(String name) {
    943977        for (PluginProxy plugin : pluginList)
    944             if(plugin.getPluginInformation().name.equals(name))
     978            if (plugin.getPluginInformation().name.equals(name))
    945979                return plugin.plugin;
    946980        return null;
    … …  
    9861020            if (plugin.exists()) {
    9871021                if (!plugin.delete() && dowarn) {
    988                     System.err.println(tr("Warning: failed to delete outdated plugin ''{0}''.", plugin.toString()));
    989                     System.err.println(tr("Warning: failed to install already downloaded plugin ''{0}''. Skipping installation. JOSM is still going to load the old plugin version.", pluginName));
     1022                    Main.warn(tr("Failed to delete outdated plugin ''{0}''.", plugin.toString()));
     1023                    Main.warn(tr("Failed to install already downloaded plugin ''{0}''. Skipping installation. JOSM is still going to load the old plugin version.", pluginName));
    9901024                    continue;
    9911025                }
    … …  
    9961030            } catch (Exception e) {
    9971031                if (dowarn) {
    998                     System.err.println(tr("Warning: failed to install plugin ''{0}'' from temporary download file ''{1}''. {2}", plugin.toString(), updatedPlugin.toString(), e.getLocalizedMessage()));
     1032                    Main.warn(tr("Failed to install plugin ''{0}'' from temporary download file ''{1}''. {2}", plugin.toString(), updatedPlugin.toString(), e.getLocalizedMessage()));
    9991033                }
    10001034                continue;
    … …  
    10021036            // Install plugin
    10031037            if (!updatedPlugin.renameTo(plugin) && dowarn) {
    1004                 System.err.println(tr("Warning: failed to install plugin ''{0}'' from temporary download file ''{1}''. Renaming failed.", plugin.toString(), updatedPlugin.toString()));
    1005                 System.err.println(tr("Warning: failed to install already downloaded plugin ''{0}''. Skipping installation. JOSM is still going to load the old plugin version.", pluginName));
     1038                Main.warn(tr("Failed to install plugin ''{0}'' from temporary download file ''{1}''. Renaming failed.", plugin.toString(), updatedPlugin.toString()));
     1039                Main.warn(tr("Failed to install already downloaded plugin ''{0}''. Skipping installation. JOSM is still going to load the old plugin version.", pluginName));
    10061040            }
    10071041        }
    … …  
    11771211    }
    11781212
     1213    /**
     1214     * Returns the list of loaded plugins as a {@code String} to be displayed in status report. Useful for bug reports.
     1215     * @return The list of loaded plugins (one plugin per line)
     1216     */
    11791217    public static String getBugReportText() {
    1180         String text = "";
     1218        StringBuilder text = new StringBuilder();
    11811219        LinkedList <String> pl = new LinkedList<String>(Main.pref.getCollection("plugins", new LinkedList<String>()));
    11821220        for (final PluginProxy pp : pluginList) {
    … …  
    11881226        Collections.sort(pl);
    11891227        for (String s : pl) {
    1190             text += "Plugin: " + s + "\n";
    1191         }
    1192         return text;
    1193     }
    1194 
     1228            text.append("Plugin: ").append(s).append("\n");
     1229        }
     1230        return text.toString();
     1231    }
     1232
     1233    /**
     1234     * Returns the list of loaded plugins as a {@code JPanel} to be displayed in About dialog.
     1235     * @return The list of loaded plugins (one "line" of Swing components per plugin)
     1236     */
    11951237    public static JPanel getInfoPanel() {
    11961238        JPanel pluginTab = new JPanel(new GridBagLayout());

  • trunk/src/org/openstreetmap/josm/plugins/PluginInformation.java

    r6232 r6248  
    2323import java.util.jar.JarInputStream;
    2424import java.util.jar.Manifest;
     25
    2526import javax.swing.ImageIcon;
    2627
    … …  
    186187                new URL(s);
    187188            } catch (MalformedURLException e) {
    188                 System.out.println(tr("Invalid URL ''{0}'' in plugin {1}", s, name));
     189                Main.info(tr("Invalid URL ''{0}'' in plugin {1}", s, name));
    189190                s = null;
    190191            }
    … …  
    200201                    s = tr(s);
    201202                } catch (IllegalArgumentException e) {
    202                     System.out.println(tr("Invalid plugin description ''{0}'' in plugin {1}", s, name));
     203                    Main.info(tr("Invalid plugin description ''{0}'' in plugin {1}", s, name));
    203204                }
    204205            }

  • trunk/src/org/openstreetmap/josm/plugins/ReadLocalPluginInformationTask.java

    r6084 r6248  
    1414import java.util.Map;
    1515
     16import org.openstreetmap.josm.Main;
    1617import org.openstreetmap.josm.gui.PleaseWaitRunnable;
    1718import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    … …  
    9091                processLocalPluginInformationFile(f);
    9192            } catch(PluginListParseException e) {
    92                 System.err.println(tr("Warning: Failed to scan file ''{0}'' for plugin information. Skipping.", fname));
     93                Main.warn(tr("Failed to scan file ''{0}'' for plugin information. Skipping.", fname));
    9394                e.printStackTrace();
    9495            }
    … …  
    151152                }
    152153            } catch (PluginException e){
    153                 System.err.println(e.getMessage());
    154                 System.err.println(tr("Warning: Failed to scan file ''{0}'' for plugin information. Skipping.", fname));
     154                Main.warn("PluginException: "+e.getMessage());
     155                Main.warn(tr("Failed to scan file ''{0}'' for plugin information. Skipping.", fname));
    155156            }
    156157            monitor.worked(1);

  • trunk/src/org/openstreetmap/josm/plugins/ReadRemotePluginInformationTask.java

    r6084 r6248  
    259259            if (!pluginDir.exists()) {
    260260                if (! pluginDir.mkdirs()) {
    261                     System.err.println(tr("Warning: failed to create plugin directory ''{0}''. Cannot cache plugin list from plugin site ''{1}''.", pluginDir.toString(), site));
     261                    Main.warn(tr("Failed to create plugin directory ''{0}''. Cannot cache plugin list from plugin site ''{1}''.", pluginDir.toString(), site));
    262262                }
    263263            }
    … …  
    311311            List<PluginInformation> pis = new PluginListParser().parse(in);
    312312            availablePlugins.addAll(filterDeprecatedPlugins(pis));
    313         } catch(UnsupportedEncodingException e) {
    314             System.err.println(tr("Failed to parse plugin list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    315             e.printStackTrace();
    316         } catch(PluginListParseException e) {
    317             System.err.println(tr("Failed to parse plugin list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
     313        } catch (UnsupportedEncodingException e) {
     314            Main.error(tr("Failed to parse plugin list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
     315            e.printStackTrace();
     316        } catch (PluginListParseException e) {
     317            Main.error(tr("Failed to parse plugin list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    318318            e.printStackTrace();
    319319        }

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

    r6232 r6248  
    370370                try {
    371371                    closeDate = formatter.parse(m.group(2));
    372                 } catch(ParseException ex) {
    373                     System.err.println(tr("Failed to parse date ''{0}'' replied by server.", m.group(2)));
     372                } catch (ParseException ex) {
     373                    Main.error(tr("Failed to parse date ''{0}'' replied by server.", m.group(2)));
    374374                    ex.printStackTrace();
    375375                }

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

    r6084 r6248  
    55import java.io.File;
    66import java.io.FileInputStream;
     7import java.io.IOException;
    78import java.io.InputStream;
    8 import java.io.IOException;
    99import java.net.URL;
    1010import java.text.MessageFormat;
    … …  
    1414import java.util.Comparator;
    1515import java.util.HashMap;
     16import java.util.Locale;
    1617import java.util.jar.JarInputStream;
    1718import java.util.zip.ZipEntry;
    18 import java.util.Locale;
    1919
    2020import javax.swing.JColorChooser;
    … …  
    641641            } else {
    642642                if (!l.getLanguage().equals("en")) {
    643                     System.out.println(tr("Unable to find translation for the locale {0}. Reverting to {1}.",
     643                    Main.info(tr("Unable to find translation for the locale {0}. Reverting to {1}.",
    644644                            l.getDisplayName(), Locale.getDefault().getDisplayName()));
    645645                } else {

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

    r6246 r6248  
    308308            } else {
    309309                if (!suppressWarnings) {
    310                     System.err.println(tr("Failed to locate image ''{0}''", name));
     310                    Main.error(tr("Failed to locate image ''{0}''", name));
    311311                }
    312312                return null;
    … …  
    613613            }
    614614        } catch (Exception e) {
    615             System.err.println(tr("Warning: failed to handle zip file ''{0}''. Exception was: {1}", archive.getName(), e.toString()));
     615            Main.warn(tr("Failed to handle zip file ''{0}''. Exception was: {1}", archive.getName(), e.toString()));
    616616        } finally {
    617617            Utils.close(zipFile);
    … …  
    671671                        return u;
    672672                } catch (SecurityException e) {
    673                     System.out.println(tr(
    674                             "Warning: failed to access directory ''{0}'' for security reasons. Exception was: {1}",
     673                    Main.warn(tr(
     674                            "Failed to access directory ''{0}'' for security reasons. Exception was: {1}",
    675675                            name, e.toString()));
    676676                }
    … …  
    685685                return u;
    686686        } catch (SecurityException e) {
    687             System.out.println(tr(
    688                     "Warning: failed to access directory ''{0}'' for security reasons. Exception was: {1}", dir, e
     687            Main.warn(tr(
     688                    "Failed to access directory ''{0}'' for security reasons. Exception was: {1}", dir, e
    689689                    .toString()));
    690690        }
    … …  
    736736                @Override
    737737                public void startElement(String uri, String localName, String qName, Attributes atts) throws SAXException {
    738                     System.out.println();
    739738                    if (localName.equalsIgnoreCase("img")) {
    740739                        String val = atts.getValue("src");
    … …  
    759758            return r.getResult();
    760759        } catch (Exception e) {
    761             System.out.println("INFO: parsing " + base + fn + " failed:\n" + e);
     760            Main.warn("Parsing " + base + fn + " failed:\n" + e);
    762761            return null;
    763762        }
    764         System.out.println("INFO: parsing " + base + fn + " failed: Unexpected content.");
     763        Main.warn("Parsing " + base + fn + " failed: Unexpected content.");
    765764        return null;
    766765    }

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

    r6070 r6248  
    5151                    // Workaround for KDE (Desktop API is severely flawed)
    5252                    // see http://bugs.sun.com/view_bug.do?bug_id=6486393
    53                     System.err.println("Warning: Desktop class failed. Platform dependent fall back for open url in browser.");
     53                    Main.warn("Desktop class failed. Platform dependent fall back for open url in browser.");
    5454                    displayUrlFallback(uri);
    5555                }
    … …  
    6060        } else {
    6161            try {
    62                 System.err.println("Warning: Desktop class is not supported. Platform dependent fall back for open url in browser.");
     62                Main.warn("Desktop class is not supported. Platform dependent fall back for open url in browser.");
    6363                displayUrlFallback(uri);
    6464            } catch (IOException e) {

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

    r6125 r6248  
    4040            MsetEnabledPreferencesMenu.invoke(Ocom_apple_eawt_Application, new Object[] { Boolean.TRUE });
    4141        } catch (Exception ex) {
    42             // Oops, what now?
    43             // We'll just ignore this for now. The user will still be able to close JOSM
    44             // by closing all its windows.
    45             System.out.println("Failed to register with OSX: " + ex);
     42            // We'll just ignore this for now. The user will still be able to close JOSM by closing all its windows.
     43            Main.warn("Failed to register with OSX: " + ex);
    4644        }
    4745    }
    … …  
    4947    public Object invoke (Object proxy, Method method, Object[] args) throws Throwable {
    5048        Boolean handled = Boolean.TRUE;
    51         //System.out.println("Going to handle method "+method+" (short: "+method.getName()+") with event "+args[0]);
    5249        if (method.getName().equals("handleQuit")) {
    5350            handled = Main.exitJosm(false, 0);
    … …  
    6259                args[0].getClass().getDeclaredMethod("setHandled", new Class[] { boolean.class }).invoke(args[0], new Object[] { handled });
    6360            } catch (Exception ex) {
    64                 System.out.println("Failed to report handled event: " + ex);
     61                Main.warn("Failed to report handled event: " + ex);
    6562            }
    6663        }
    … …  
    6966    @Override
    7067    public void openUrl(String url) throws IOException {
    71         // Ain't that KISS?
    7268        Runtime.getRuntime().exec("open " + url);
    7369    }

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

    r5979 r6248  
    371371        if (potentialShortcut != null) {
    372372            // this always is a logic error in the hook
    373             System.err.println("CONFLICT WITH SYSTEM KEY "+shortText);
     373            Main.error("CONFLICT WITH SYSTEM KEY "+shortText);
    374374            return null;
    375375        }
    … …  
    415415                    if ( findShortcut(k, newmodifier) == null ) {
    416416                        Shortcut newsc = new Shortcut(shortText, longText, requestedKey, m, k, newmodifier, false, false);
    417                         System.out.println(tr("Silent shortcut conflict: ''{0}'' moved by ''{1}'' to ''{2}''.",
     417                        Main.info(tr("Silent shortcut conflict: ''{0}'' moved by ''{1}'' to ''{2}''.",
    418418                            shortText, conflict.getShortText(), newsc.getKeyText()));
    419419                        newsc.saveDefault();

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

    r6229 r6248  
    2424import javax.xml.validation.ValidatorHandler;
    2525
     26import org.openstreetmap.josm.Main;
    2627import org.openstreetmap.josm.io.MirroredInputStream;
    2728import org.xml.sax.Attributes;
    … …  
    296297            } catch (SAXException e) {
    297298                // Exception very unlikely to happen, so no need to translate this
    298                 System.err.println("Cannot disable 'load-external-dtd' feature: "+e.getMessage());
     299                Main.error("Cannot disable 'load-external-dtd' feature: "+e.getMessage());
    299300            }
    300301            reader.parse(new InputSource(in));
Note: See TracChangeset for help on using the changeset viewer.