I can't tell if this is intentional or not, but something doesn't smell
right here:

    def shutdown(self, tid=None):
        self._available.acquire()
        try:
            if tid:
                cleanup_list = [tid]
            else:
                cleanup_list = self._active.keys()
            for tid in cleanup_list:                 # tid used in for loop
                self._cleanup(tid)
            if not tid:                              # tid tested for None
                for _, cnx in self._dormant.iteritems():
                    cnx.close()
        finally:
            self._available.release()

I'm not sure what the code is trying to do, but it seems like you want
to use a different loop variable name (?)

-- 
Dave Abrahams
BoostPro Computing
http://www.boostpro.com


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Trac 
Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/trac-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to