RFR (trivial): 8235833: PosixPlatform.cpp should not include sysctl.h

[Yasumasa Suenaga]

Thanks Alexey! and I wait Reviewer(s).

Yasumasa


On 2019/12/13 1:42, Alexey Semenyuk wrote:
> Looks good.
> 
> - Alexey
> 
> On 12/12/2019 8:23 AM, Yasumasa Suenaga wrote:
>> Hi Andrew,
>>
>> I can see same error whether --disable-precompiled-headers or not.
>> So I filed it to JBS and uploaded webrev.
>> Could you review it?
>>
>>   JBS: https://bugs.openjdk.java.net/browse/JDK-8235833
>>   webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8235833/webrev.00/
>>
>> It works fine on Fedora 31.
>> I've pushed it to submit repo. I believe it passes all tests.
>>
>>
>> Thanks,
>>
>> Yasumasa
>>
>>
>> On 2019/12/12 19:25, Andrew Haley wrote:
>>> On 12/12/19 9:31 AM, Yasumasa Suenaga wrote:
>>>> AFAICS sysctl.h provides `sysctl()` and some macros, but they are not used in PosixPlatform.cpp .
>>>> In fact, I did not see any error when I remove `#include <sys/sysctl.h>` from it.
>>>
>>> You probably won't see an error unless you configure with
>>> --disable-precompiled-headers. Please try that, and we can then
>>> consider removing that #include.
>>>
>