RFR: Remove sys/ prefix from poll.h/signal.h includes
David Holmes
david.holmes at oracle.com
Wed Apr 12 00:08:33 UTC 2017
Hi Mikael,
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 12/04/2017 8:42 AM, Mikael Vidstedt wrote:
>
> In several places in the JDK sources poll.h and signal.h are included through sys/{poll.h,signal.h}. The posix spec says the files should be included without the sys/ prefix, and musl issues a warning when sys/{poll.h,signal.h} are included.
>
> This change simple removes the sys/ prefix in all the relevant places. I’ve tried my best to verify that this works on all the platforms/toolchains I have access to, and also got help from the SAP guys to verify that it should work on AIX as well.
>
> hotspot: http://cr.openjdk.java.net/~mikael/webrevs/portola/pollsignal/webrev.00/hotspot/webrev/
> jdk: http://cr.openjdk.java.net/~mikael/webrevs/portola/pollsignal/webrev.00/jdk/webrev/
>
> Cheers,
> Mikael
>
More information about the portola-dev
mailing list