RFR(S) 8186897: semaphore_posix.hpp should not be included on OSX
David Holmes
david.holmes at oracle.com
Tue Aug 29 12:30:42 UTC 2017
Looks good!
Thanks,
David
On 29/08/2017 9:42 PM, Mikael Gerdin wrote:
> Hi,
> Please review this small fix to not include the semaphore_posix header
> in os_posix.cpp on OSX.
> If the transitive includes of os_posix.cpp are changed such that it
> includes semaphore.hpp a name clash will otherwise occur since all
> platform specific semaphore headers define the SemaphoreImpl typedef.
>
> Webrev: http://cr.openjdk.java.net/~mgerdin/8186897/webrev.0
> Bug: https://bugs.openjdk.java.net/browse/JDK-8186897
> Testing: JPRT buildonly
>
> Thanks
> /Mikael
More information about the hotspot-runtime-dev
mailing list