[9] RFR of 8080589: (fs) FileChannel.force should use fcntl(F_FULLFSYNC) instead of fsync on OS X

[Brian Burkhalter]

On May 19, 2015, at 1:03 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> On 19/05/2015 19:33, Brian Burkhalter wrote:
>> :
>> 
>> Perhaps this is a safer approach.
> Is F_FULLSYNC not guaranteed to be defined on all supported OS X versions? Just wondering if  #if defined(F_FULLSYNC) is really needed. It otherwise looks okay as it doesn't compile in the current implementation that choose fsync or fdatasync.
> 
> -Alan

I was inferring from the online information that Robert sent and some other scattered information. You have a point though: if we build on a system for which F_FULLSYNC is defined and some earlier system where it is not tries to run the code there will be an error. On that basis it makes sense to remove it unless some kind of dynamic check could be performed.

Brian