Work around approximate active status

Author: iqqu

src/main/java/com/rarchives/ripme/ripper/DownloadThreadPool.java

@@ -108,6 +108,9 @@ public int getPendingThreadCount() {
         return threadPool.getQueue().size();
     }
 
+    /**
+     * @return The approximate active thread count
+     */
     public int getActiveThreadCount() {
         return threadPool.getActiveCount();
     }

src/main/java/com/rarchives/ripme/ui/MainWindow.java

@@ -1703,6 +1703,8 @@ public void run() {
         }
     }
 
+    private long lastStatusUpdate = 0;
+
     private synchronized void handleEvent(StatusEvent evt) {
         RipStatusMessage msg = evt.msg;
         RipStatusMessage.STATUS status = msg.getStatus();
@@ -1711,7 +1713,15 @@ private synchronized void handleEvent(StatusEvent evt) {
         if (status == RipStatusMessage.STATUS.CHUNK_BYTES) {
             transferRate.addChunk((Long) msg.getObject());
             transferRateValue.setText(transferRate.formatHumanTransferRate());
-            return;
+
+            // Quick hack: ripper.getActiveCount() and dependent values are approximate and the value can be outdated (too large) when it is called when the ripper notifies COMPLETE,
+            // so allow the status info to update too, but throttle it a little.
+            long now = System.currentTimeMillis();
+            boolean allowStatusUpdate = now > lastStatusUpdate + 200;
+            if (!allowStatusUpdate) {
+                return;
+            }
+            lastStatusUpdate = now;
         }
 
         if (evt.ripper.useByteProgessBar()) {
@@ -1735,6 +1745,11 @@ private synchronized void handleEvent(StatusEvent evt) {
             currentlyRippingProgress.setValue(evt.ripper.getCompletionPercentage());
         }
 
+        // Quick hack finish:
+        if (status == RipStatusMessage.STATUS.CHUNK_BYTES) {
+            return;
+        }
+
         switch (status) {
         case LOADING_RESOURCE:
         case DOWNLOAD_STARTED:

src/main/java/com/rarchives/ripme/ui/MinimumWidthLabel.java

@@ -2,13 +2,15 @@
 
 import javax.swing.JLabel;
 import java.awt.Dimension;
+import java.util.Objects;
 
 /**
  * GridBagLayout does not respect minimum size, only preferred size.
  * In order to set a minimum width, we need to override getPreferredSize.
  */
 public class MinimumWidthLabel extends JLabel {
     private String minimumWidthText;
+    private String currentText;
 
     public MinimumWidthLabel(String minimumWidthText, String defaultText) {
         this.minimumWidthText = minimumWidthText;
@@ -30,4 +32,14 @@ public Dimension getPreferredSize() {
     public void setMinimumWidthText(String minimumWidthText) {
         this.minimumWidthText = minimumWidthText;
     }
+
+    @Override
+    public void setText(String text) {
+        // Cache the last text to save cycles,
+        // because this may be called with the same value very often
+        if (!Objects.equals(currentText, text)) {
+            currentText = text;
+            super.setText(text);
+        }
+    }
 }