RFR [8055421]: (fs) bad error handling in java.base/unix/native/libnio/fs/UnixNativeDispatcher.c

Ivan Gerasimov ivan.gerasimov at oracle.com
Mon Aug 25 14:50:31 UTC 2014


On 25.08.2014 17:17, Alan Bateman wrote:
> On 25/08/2014 14:13, Ivan Gerasimov wrote:
>>
>> Here's the comment I've added to UnixNativeDispatcher.fclose:
>>
>>    /* NOTE: fclose() wrapper is only used with read-only streams.
>>      * If it ever is used with write streams, it might be better to add
>>      * RESTARTABLE(fflush(fp)) before closing, to make sure the stream
>>      * is completely written even if fclose() failed.
>>      */
> This seems okay but I can't tell if you are adding this to the .java 
> or .c file.
>
This is in .c file.
Here's the webrev for the sake of clarity:
http://cr.openjdk.java.net/~igerasim/8055421/3/webrev/

Sincerely yours,
Ivan


More information about the nio-dev mailing list