RE: [Proxool-developer] Re: [Proxool-user] warning messages

[Bill H. <bi...@lo...>]

Michael,

On Thu, 2004-04-22 at 20:40, Michael Hoefer wrote:

> Locally I fixed it in perhaps a better way by checking if the hook is
> alive:
> 
> 	if (shutdownHook != null && !shutdownHook.isAlive() ) {
>         ShutdownHook.remove(shutdownHook);
>       }

And there we come across another fix that has been made since 0.8.3.
That ShutdownHook code is JDK 1.3 and isn't supported by JDK 1.2. That
was an oversight. CVS does the same using reflection to register a
shutdown hook only if the JDK supports it.

On Thu, 2004-04-22 at 20:43, Michael Hoefer wrote: 

 .The best solution is just to refactor the meat of the shutdown method
> into another method that is then invoked from the shutdownhook.
> Something like removeConnections(String finalizer, int delay)....

I agree. I'll look into it. I need to check whether that means we can
lose the test for an IllegalStateException. 

- Bill