I'm cleaning up minor changes in my tree, and I noticed I did this:

diff -u -r1.4 TreatFileJob.cc
--- TreatFileJob.cc     2001/12/11 14:37:45     1.4
+++ TreatFileJob.cc     2002/04/15 02:25:02
@@ -73,7 +73,8 @@
         first=new FileInfo(*fi);

       TreatCurrent(d,fi);
-      return PRF_LATER;
+      if(session->IsOpen())
+        return PRF_LATER;
    }
    int res=session->Done();
    if(res==FA::IN_PROGRESS)

Is that return needed at all?  Can a session ever be in progress but not
open?  I added that conditional to streamline "find", but everything
seems to work fine without that return at all.  (I find the difference
between "open"/"closed" and "in progress" slightly confusing.)

-- 
Glenn Maynard

Reply via email to