RFR(S): 8180003: Remove sys/ prefix from poll.h and signal.h includes

David Holmes david.holmes at oracle.com
Tue May 9 23:19:23 UTC 2017


Hi Mikael,

To repeat myself from:

http://mail.openjdk.java.net/pipermail/portola-dev/2017-April/000025.html

Changes look okay. I agree with the rationale.

Looking at actual implementations, linux and mac OS are trivially fine
(poll.h just includes sys/poll.h). Solaris is non-trivially fine -
poll.h does more than what sys/poll.h does, but nothing that affects our 
sources.

Thanks,
David

:)

On 10/05/2017 7:29 AM, Mikael Vidstedt wrote:
>
> Please review this small change which removes the sys/ prefix from a bunch of includes of poll.h and signal.h.
>
> hotspot: http://cr.openjdk.java.net/~mikael/webrevs/8180003/webrev.00/hotspot/webrev/
> jdk: http://cr.openjdk.java.net/~mikael/webrevs/8180003/webrev.00/jdk/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8180003/webrev.00/jdk/webrev/>
>
> Using the sys/ prefix works on many platforms, but the posix spec makes it clear that the poll.h and signal.h header files should be included without the prefix.
>
> I have verified that this change works on all the Oracle supported platforms, but I could use some help verifying it on AIX.
>
> Cheers,
> Mikael
>


More information about the nio-dev mailing list