Ticket #17035: 17035-progress-monitor-v2.patch

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

@@ -170 +170 @@
             }
             tests = null;
             if (ValidatorPrefHelper.PREF_USE_IGNORE.get()) {
+                getProgressMonitor().setCustomText("");
                 getProgressMonitor().subTask(tr("Updating ignored errors ..."));
                 for (TestError error : errors) {
                     if (canceled) return;

src/org/openstreetmap/josm/data/validation/Test.java

@@ -76 +76 @@
     /** the start time to compute elapsed time when test finishes */
     protected long startTime;
 
+    private boolean showElementCount;
+
     /**
      * Constructor
      * @param name Name of the test
@@ -202 +204 @@
         if (progressMonitor != null) {
             progressMonitor.setTicksCount(selection.size());
         }
+        long cnt = 0;
         for (OsmPrimitive p : selection) {
             if (isCanceled()) {
                 break;
@@ -211 +214 @@
             }
             if (progressMonitor != null) {
                 progressMonitor.worked(1);
+                cnt++;
+                // add frequently changing info to progress monitor so that it
+                // doesn't seem to hang when test takes long
+                if (showElementCount && cnt % 1000 == 0) {
+                    progressMonitor.setExtraText(tr("{0} of {1} elements done", cnt, selection.size()));
+                }
             }
         }
     }
@@ -380 +389 @@
     public void clear() {
         errors.clear();
     }
+
+    protected void setShowElements(boolean b) {
+        showElementCount = b;
+    }
 }

src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java

@@ -1092 +1092 @@
     @Override
     public void startTest(ProgressMonitor progressMonitor) {
         super.startTest(progressMonitor);
+        super.setShowElements(true);
         if (indexData == null) {
             indexData = new IndexData(checks);
         }