Index: trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadGpsTask.java
===================================================================
--- trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadGpsTask.java	(revision 7596)
+++ trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadGpsTask.java	(revision 7597)
@@ -68,5 +68,7 @@
     public Future<?> loadUrl(boolean newLayer, String url, ProgressMonitor progressMonitor) {
         CheckParameterUtil.ensureParameterNotNull(url, "url");
-        if (url.matches(PATTERN_TRACE_ID) || url.matches(PATTERN_EXTERNAL_GPX_SCRIPT) || url.matches(PATTERN_EXTERNAL_GPX_FILE)) {
+        if (url.matches(PATTERN_TRACE_ID)
+         || url.matches(PATTERN_EXTERNAL_GPX_SCRIPT)
+         || url.matches(PATTERN_EXTERNAL_GPX_FILE)) {
             downloadTask = new DownloadTask(newLayer,
                     new OsmServerLocationReader(url), progressMonitor);
@@ -101,5 +103,5 @@
 
         public DownloadTask(boolean newLayer, OsmServerReader reader, ProgressMonitor progressMonitor) {
-            super(tr("Downloading GPS data"));
+            super(tr("Downloading GPS data"), progressMonitor, false);
             this.reader = reader;
             this.newLayer = newLayer;
@@ -132,5 +134,6 @@
             String name = newLayerName != null ? newLayerName : tr("Downloaded GPX Data");
 
-            GpxImporterData layers = GpxImporter.loadLayers(rawData, reader.isGpxParsedProperly(), name, tr("Markers from {0}", name));
+            GpxImporterData layers = GpxImporter.loadLayers(rawData, reader.isGpxParsedProperly(), name,
+                    tr("Markers from {0}", name));
 
             GpxLayer gpxLayer = addOrMergeLayer(layers.getGpxLayer(), findGpxMergeLayer());
Index: trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java
===================================================================
--- trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java	(revision 7596)
+++ trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java	(revision 7597)
@@ -72,8 +72,7 @@
 
         public DownloadTask(OsmServerReader reader, ProgressMonitor progressMonitor) {
-            super(tr("Downloading Notes"));
+            super(tr("Downloading Notes"), progressMonitor, false);
             this.reader = reader;
         }
-
 
         @Override
Index: trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java
===================================================================
--- trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java	(revision 7596)
+++ trunk/src/org/openstreetmap/josm/gui/PleaseWaitRunnable.java	(revision 7597)
@@ -32,8 +32,10 @@
     /**
      * Create the runnable object with a given message for the user.
+     * @param title message for the user
      */
     public PleaseWaitRunnable(String title) {
         this(title, false);
     }
+
     /**
      * Create the runnable object with a given message for the user.
@@ -65,4 +67,14 @@
     }
 
+    /**
+     * Create the runnable object with a given message for the user
+     *
+     * @param title message for the user
+     * @param progressMonitor progress monitor
+     * @param ignoreException If true, exception will be propagated to calling code. If false then
+     * exception will be thrown directly in EDT. When this runnable is executed using executor framework
+     * then use false unless you read result of task (because exception will get lost if you don't)
+     * @throws IllegalArgumentException thrown if parent is null
+     */
     public PleaseWaitRunnable(String title, ProgressMonitor progressMonitor, boolean ignoreException) {
         this.title = title;
@@ -175,4 +187,8 @@
     protected abstract void finish();
 
+    /**
+     * Relies the progress monitor.
+     * @return the progress monitor
+     */
     public ProgressMonitor getProgressMonitor() {
         return progressMonitor;
Index: trunk/src/org/openstreetmap/josm/plugins/PluginDownloadTask.java
===================================================================
--- trunk/src/org/openstreetmap/josm/plugins/PluginDownloadTask.java	(revision 7596)
+++ trunk/src/org/openstreetmap/josm/plugins/PluginDownloadTask.java	(revision 7597)
@@ -155,5 +155,5 @@
         File pluginDir = Main.pref.getPluginsDirectory();
         if (!pluginDir.exists() && !pluginDir.mkdirs()) {
-            //lastException = new PluginDownloadException(tr("Failed to create plugin directory ''{0}''", pluginDir.toString()));
+            /*lastException =*/ new PluginDownloadException(tr("Failed to create plugin directory ''{0}''", pluginDir.toString()));
             failed.addAll(toUpdate);
             return;