Changeset 11995 in josm for trunk/src/org/openstreetmap/josm

Timestamp:

2017-04-24T21:16:45+02:00 (7 years ago)

Author:

Don-vip

Message:

fix #14690 - Loading commandline file blocks JOSM (regression from r11986)

File:

• trunk/src/org/openstreetmap/josm/Main.java (modified) (2 diffs)

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

@@ -830 +830 @@
      * Handle command line instructions after GUI has been initialized.
      * @param args program arguments
-     */
-    protected static void postConstructorProcessCmdLine(ProgramArguments args) {
+     * @return the list of submitted tasks
+     */
+    protected static List<Future<?>> postConstructorProcessCmdLine(ProgramArguments args) {
         List<Future<?>> tasks = new ArrayList<>();
         List<File> fileList = new ArrayList<>();
@@ -843 +844 @@
             tasks.addAll(DownloadParamType.paramType(s).downloadGps(s));
         }
-        // Make sure all download tasks complete before handling selection arguments
-        for (Future<?> task : tasks) {
-            try {
-                task.get();
-            } catch (InterruptedException | ExecutionException e) {
-                error(e);
-            }
-        }
-        for (String s : args.get(Option.SELECTION)) {
-            SearchAction.search(s, SearchAction.SearchMode.add);
-        }
+        final Collection<String> selectionArguments = args.get(Option.SELECTION);
+        if (!selectionArguments.isEmpty()) {
+            tasks.add(Main.worker.submit(() -> {
+                for (String s : selectionArguments) {
+                    SearchAction.search(s, SearchAction.SearchMode.add);
+                }
+            }));
+        }
+        return tasks;
     }