Review request for 6829503

David Holmes - Sun Microsystems David.Holmes at Sun.COM
Sun Apr 19 22:50:20 UTC 2009


Hi Alan,

Alan Bateman said the following on 04/20/09 00:00:
> As it happens, there was a bug in that code (6526376) that caused 
> NPE to be thrown so jdk7 b10 is the first build where 
> IllegalStateException is possible.

Ah I see. Seems to me that 6526376 took the easy fix and ignored the 
underlying issue of whether this should be an exceptional circumstance 
or not. :(

> I don't think we want to force Mandy into fixing issues that aren't of 
> her doing so submitting a bug and letting her move on is probably the 
> best thing to do.

Agreed.

I think there's probably a way to keep the list in java code (for 
resource management) and also add the fd to list used by the VM. Adding 
a new "VM shutdown hook" to be executed by the VMThread at the 
termination safepoint would probably be the best place to deal with this.

Cheers,
David



More information about the core-libs-dev mailing list