lib/python/Products/, line 110, method _cook_check

         try:    mtime=os.stat(self.filename)[8]
         except: mtime=0

I've just spent an hour or so tracking down an awkward bug in some 
unit-tests. The true error was being hidden by this bare except.

I'd change it, and commit the change, except that I don't know what it 
is supposed to be catching. Any hints?

Actually, I'd rather just remove the try: except: and let os.stat fail 
if it is going to fail. Any objections?

(of course there will be... even bare excepts were put there for some 

Steve Alexander

Zope-Dev maillist  -  [EMAIL PROTECTED]
**  No cross posts or HTML encoding!  **
(Related lists - )

Reply via email to