Changeset 17838 for gli

Timestamp:

2008-11-12T16:09:49+13:00 (15 years ago)

Author:

ak19

Message:

Job is no longer set to null in deleteDownloadJob. That's only done in the run method of this DownloadScrollPane thread

File:

• gli/trunk/src/org/greenstone/gatherer/download/DownloadScrollPane.java (modified) (2 diffs)

gli/trunk/src/org/greenstone/gatherer/download/DownloadScrollPane.java

@@ -91 +91 @@
         } catch (Exception e) {
         e.printStackTrace();
-        }
-        
-        setJobToNull();//job = null;
-        
+        }       
     }
     // Get rid of this download's dedicated pane
     finishedDownloadJob(delete_me);
     }
-
-    synchronized protected void setJobToNull() { job = null; }  
 
     /** To be called when a download job has terminated naturally or was prematurely stopped
@@ -215 +210 @@
             DebugStream.println("DownloadJob " + jobDisplayString + " Begun.");
             System.err.println("DownloadJob " + job.port + " " + job.toString() + " Begun.");
-            if(job != null) {
-                job.callDownload();
-            }
+            job.callDownload();
             finishedDownloadJob(delete_me); // Job is done, get rid of the separate display panel for this download
             System.err.println("DownloadJob " + jobDisplayString + " complete.");