Changeset 6248 in josm for trunk/src/org/openstreetmap/josm/io/imagery/HTMLGrabber.java

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)

File:

• trunk/src/org/openstreetmap/josm/io/imagery/HTMLGrabber.java (modified) (2 diffs)

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

@@ -30 +30 @@
         String urlstring = url.toExternalForm();
 
-        System.out.println("Grabbing HTML " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
+        Main.info("Grabbing HTML " + (attempt > 1? "(attempt " + attempt + ") ":"") + url);
 
         ArrayList<String> cmdParams = new ArrayList<String>();
         StringTokenizer st = new StringTokenizer(MessageFormat.format(PROP_BROWSER.get(), urlstring));
-        while( st.hasMoreTokens() ) {
+        while (st.hasMoreTokens()) {
             cmdParams.add(st.nextToken());
         }
@@ -43 +43 @@
         try {
             browser = builder.start();
-        } catch(IOException ioe) {
+        } catch (IOException ioe) {
             throw new IOException( "Could not start browser. Please check that the executable path is correct.\n" + ioe.getMessage() );
         }