Your fix looks good. On 18/06/2011 20:39, Alan Bateman wrote:
I need a reviewer for a small fix to a test that I caught looping on my syst.em. The test creates a server thread that emulates a non responding LDAP server. Unfortunately the thread goes into a loop reading from the connection once it is closed. I'll bet the original author only tested this on Solaris where it would have completed as intended due to now-disabled and legacy interruptible I/O support. While I was there I eliminated the hard coded port that would cause it to fail if that port were already in use. The test could be improved in other ways too but for now the webrev with the proposed changes is here: http://cr.openjdk.java.net/~alanb/7056489/webrev/ Thanks, Alan.
