> -----Original Message-----
> From: Geoffrey Talvola [mailto:gtalvola@...]
> Stuart Donaldson [mailto:stuartd@...] wrote:
> Would it be better to make your modifications to
> NewThreadedAppServer rather
> than to ThreadedAppServer? It may save time down the road if
> NewThreadedAppServer becomes the standard.
I am actually working primarily with NewThreadedAppServer since I have only
heard positive comments towards using it to replace ThreadedAppServer. My
comments apply to both however.
I have tried my above suggestion, dealing with shutDown() and it appears
this should ONLY be called from the _closeThread thread. Any references to
it elsewhere could initiate race conditions and end up with more than one
call to shutDown().
The existing run() function along with the Monitor quit function both
reference shutDown directly. While the existing run() function reference is
probably stale code (reference the KeyboardInterrupt discussion) the Monitor
use of shutDown will probably be a race condition.