Is there a better way to implement this:

[Michael Yanowitz]

Hello:

   I wrote the code below (much irrelevant code removed).
This doesn't quite work. What I wanted it to do was
 a) Execute function ftimed, which takes a function and a timeout
    in seconds.
 b) This will also execute function abort() as a thread.
    This function just runs for the specified
    number of seconds and returns.
    However, before it returns, throws an exception.
c)  If test() is still running when abort() is
    finished, ftimed() should catch the exception and
    return.

It is catching the exception, however it continues running the function.
Why does it continue and not return?

What am I missing, or is there a better way to
implement this (having ftimed() return when the
abort-timer time is exceeded?


import time, thread, sys

thread_finished = "MAX RUN TIME EXCEEDED!"

def abort (seconds):
     start_time = time.time()
     while ((time.time() - start_time) < seconds):
        time.sleep(0.01)
     print "script run time exceeded max_run_time of", seconds, "seconds."
     raise thread_finished
     return


def test():
    i = 0
    while (True):
       time.sleep(1)
       print "HELLO", i
       i+=1


def ftimed (func, seconds):
    thread.start_new_thread (abort, (seconds,))

    try:
        func()
    except thread_finished:
        print  "Timeout"
        return

ftimed (test, 30)
print "Script finished"

  It presently generates the following output:
$ python ./testthread.py
HELLO 0
HELLO 1
HELLO 2
HELLO 3
HELLO 4
HELLO 5
HELLO 6
HELLO 7
HELLO 8
HELLO 9
HELLO 10
HELLO 11
HELLO 12
HELLO 13
HELLO 14
HELLO 15
HELLO 16
HELLO 17
HELLO 18
HELLO 19
HELLO 20
HELLO 21
HELLO 22
HELLO 23
HELLO 24
HELLO 25
HELLO 26
HELLO 27
HELLO 28
HELLO 29
script run time exceeded max_run_time of 30 seconds.
Unhandled exception in thread started by <function abort at 0x009CEF30>
Traceback (most recent call last):
  File "./testthread.py", line 10, in abort
    raise thread_finished
MAX RUN TIME EXCEEDED!
HELLO 30
HELLO 31
HELLO 32
....



Thanks in advance:
Michael Yanowitz