> -----Original Message----- > From: Geoffrey Talvola [mailto:[EMAIL PROTECTED]] > Stuart Donaldson [mailto:[EMAIL PROTECTED]] 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. -Stuart- ------------------------------------------------------- This sf.net email is sponsored by: With Great Power, Comes Great Responsibility Learn to use your power at OSDN's High Performance Computing Channel http://hpc.devchannel.org/ _______________________________________________ Webware-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/webware-devel
