Request for review - change two include header files according to POSIX.1-2008

Alan Bateman Alan.Bateman at oracle.com
Wed Oct 12 14:16:16 UTC 2011


David Holmes wrote:
> :
>
> We'll need to verify the changes for <sys/fcntl.h> in the 
> genUnixConstants code. There tends to be a reason (often historical 
> and possibly no longer applicable) for using the sys variants (though 
> sometimes it was just that the non-sys version didn't exist at some 
> point in time).
This one is just an oversight, it can be changed to fcntl.h. There is 
another one in genSolarisConstants.c that can also be changed but that 
one is probably not an issue for AIX as that source file is Solaris 
specific.

I've created a bug to track this:
7100054: (porting) Native code should include fcntl.h and unistd.h 
rather than sys/fcntl.h and sys/unistd.h

-Alan.



More information about the core-libs-dev mailing list