Author: j16sdiz
Date: 2008-12-16 04:58:32 +0000 (Tue, 16 Dec 2008)
New Revision: 24361

Modified:
   trunk/plugins/XMLSpider/XMLSpider.java
Log:
remove from runningFetch at end of onSuccess() / onFailure()

Modified: trunk/plugins/XMLSpider/XMLSpider.java
===================================================================
--- trunk/plugins/XMLSpider/XMLSpider.java      2008-12-15 17:52:20 UTC (rev 
24360)
+++ trunk/plugins/XMLSpider/XMLSpider.java      2008-12-16 04:58:32 UTC (rev 
24361)
@@ -413,10 +413,10 @@
                        }
                } finally {
                        synchronized (this) {
-                               runningFetch.remove(page);
                                page.lastChange = System.currentTimeMillis();
                                db.store(page);
                                db.commit();
+                               runningFetch.remove(page);
                        }
                        startSomeRequests();
                }
@@ -425,11 +425,10 @@
        public void onFailure(FetchException fe, ClientGetter state, Page page) 
{
                Logger.minor(this, "Failed: " + page + " : " + state, fe);
 
+               try {
                synchronized (this) {
                        if (stopped)
                                return;
-
-                       runningFetch.remove(page);
                        
                        if (fe.newURI != null) {
                                // redirect, mark as succeeded
@@ -453,6 +452,9 @@
                }
                
                startSomeRequests();
+               } finally {
+                       runningFetch.remove(page);
+               }
        }
 
        /**

_______________________________________________
cvs mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs

Reply via email to