RFR 9: 8074818: Resolve disabled warnings for libjava
Martin Buchholz
martinrb at google.com
Wed May 27 06:59:29 UTC 2015
On Tue, May 26, 2015 at 7:52 PM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
> Hi,
>
> Sadly, but not entirely unexpectedly there is an anomaly in the include
> files:
> It seems that Windows does not define O_SYNC and O_DSYNC.
> To make up for the absence
> jdk/src/java.base/share/native/libjava/io_util.h
> conditionally defines them. There is no problem if the system include
> files appear
> first, but in the other order, fcntl.h tries to re-define it.
> In the recommended order, there is no issue.
>
We should work hard to remove order dependencies in include files.
I see that io_util.h includes <fcntl.h>, but only on BSD. Why not include
it wherever it is available, (which may be all supported platforms!) before
trying to define O_SYNC and D_SYNC?
More information about the build-dev
mailing list