Changeset 31721 for main/trunk/gli/src/org/greenstone/gatherer/download/DownloadJob.java

Timestamp:

2017-06-02T17:37:25+12:00 (7 years ago)

Author:

ak19

Message:

Removed unnecessary comments. Emacs tabbing.

File:

• main/trunk/gli/src/org/greenstone/gatherer/download/DownloadJob.java (modified) (5 diffs)

main/trunk/gli/src/org/greenstone/gatherer/download/DownloadJob.java

@@ -212 +212 @@
         previous_state = getState();
         if (getState() == RUNNING) {
-        //setState(STOPPED);
-        stopDownload(); // cancels any running SafeProcess
+        stopDownload(); // cancels any running SafeProcess, will set the current state to STOPPED when the time is right
         } else {
-        //previous_state = getState();
         setState(RUNNING);
         mummy.resumeThread();
@@ -225 +223 @@
         if(getState() == RUNNING) {
         previous_state = getState();
-        //setState(STOPPED); // do we need to do anything else to stop this? YES, we do:
-        stopDownload(); // cancels any running SafeProcess
+        stopDownload(); // cancels any running SafeProcess, will set the current state to STOPPED when the time is right
         }
         mummy.deleteDownloadJob(this);
@@ -731 +728 @@
 
     SafeProcess.log("@@@@ DONE callDownload()");
-    
-    /*
-    // Regardless of whether state==STOPPED or ends up being COMPLETE, the process is at an end now.
-    // Notify the DownloadScrollPane which is waiting on this job to complete that we are ready 
-    synchronized(this) {
-        System.err.println("**************** Notifying download scrollpane");
-        this.notify();
-    }
-    */  
 
     // refresh the workspace tree
@@ -758 +746 @@
         boolean hadToSendInterrupt = prcs.cancelRunningProcess(); // returns false if it was already terminating/terminated, true if interrupt sent
 
-
-        /*
-          // if process terminating naturally but waiting for process' worker threads to join(),
-          // shall we just remove the progress bar display for this download?
-          // If so, do this section in place of the 2 calls to progress.enableCancelJob(boolean) below
-        if(!hadToSendInterrupt && SwingUtilities.isEventDispatchThread()) {
-        if(getState() == DownloadJob.RUNNING) {
-            progress.mirrorComplete();
-            previous_state = getState();
-            setState(DownloadJob.COMPLETE);     
-        }
-        mummy.deleteCurrentDownloadJob(this); // why wait for the cleanup which can't be interrupted anyway?
-        }
-        */
     } else {
         System.err.println("@@@@ No process to interrupt");
@@ -807 +781 @@
         mummy.deleteCurrentDownloadJob(this); 
         }
-    } /*else {
-        // If we've got to here and the state isn't STOPPED then the
-        // job is complete.
-        System.err.println("**************** NOT Notifying download scrollpane");
-        if(getState() == DownloadJob.RUNNING) {
-        progress.mirrorComplete();
-        previous_state = getState();
-        setState(DownloadJob.COMPLETE);     
-        }
-    }
-    
-    // Regardless of whether state==STOPPED or ends up being COMPLETE, the process is at an end now.
-    // Notify the DownloadScrollPane which is waiting on this job to complete that we are ready 
-    synchronized(this) {
-        System.err.println("**************** Notifying download scrollpane");
-        this.notify();
-        }*/
-
+    }
     }