[9] RFR of 8139133: Changing the modification time on a unix domain socket file fails

Alan Bateman Alan.Bateman at oracle.com
Thu Dec 10 10:35:11 UTC 2015



On 09/12/2015 01:03, Brian Burkhalter wrote:
>
> UnixNativeDispatcher modified accordingly.
This looks good now. To be consistent then close0 can be private and the 
declaration moved to below close (you'll see this convention is used in 
this source file). No need for an updated webrev on this.


> :
>
>> The test is still a concern, esp. the sleep.
>
> Excised the sleep calls. The existence of the socket file is 
> determined instead by creating it in a new subdirectory of the test 
> directory and monitoring the subdirectory for new entries using a 
> WatchService. Don’t know whether this is the best way to handle it but 
> it works.
>
Looks better but I worry about the reliability of tests like this. I 
don't object to you pushing this, you just might want to fix up some of 
the odd line breaks to make it looks a bit nicer before pushing.

-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20151210/6d8bc462/attachment.html>


More information about the nio-dev mailing list