On May 9, 2006, at 7:18 AM, Oleg Broytmann wrote:

On Fri, May 05, 2006 at 01:43:04PM -0700, David Creemer wrote:
def threadSafeMethod(lock):
    def decorator(fn):
        def _wrapper(self, *args, **kwargs):
            try:
                try:
                    lock.acquire()
                    return_value = fn(self, *args, **kwargs)
                except:
print "********\nWould Have Missed The Error \n*************"
            finally:
                lock.release()
            return return_value

   Why "print" instead of just single "finally"?

        def _wrapper(self, *args, **kwargs):
            lock.acquire()
            try:
                return_value = fn(self, *args, **kwargs)
            finally:
                lock.release()



Ignore the "print" part -- it was there just to prove that the exception is getting thrown.

Your change is perfect, as far as I'm concerned. I'm currently running exactly that code w/good results on my production server.

-- David



Oleg.
--
Oleg Broytmann http://phd.pp.ru/ [EMAIL PROTECTED]
           Programmers don't die, they just GOSUB without RETURN.


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel? cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
sqlobject-discuss mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

---
David Creemer
http://www.zachary.com





-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
sqlobject-discuss mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to