RFR: JDK-8001619: Remove usage of _ALLBSD_SOURCE in bsd files

Staffan Larsen staffan.larsen at oracle.com
Fri Oct 26 05:03:50 PDT 2012


Oh... and new webrev at: http://cr.openjdk.java.net/~sla/8001619/webrev.01/


On 26 okt 2012, at 13:58, Staffan Larsen <staffan.larsen at oracle.com> wrote:

> 
> On 26 okt 2012, at 13:35, David Holmes <David.Holmes at oracle.com> wrote:
> 
>> On 26/10/2012 8:47 PM, Staffan Larsen wrote:
>>> This is an attempt at cleaning up the source in src/os/bsd by removing the usage of _ALLBSD_SOURCE in these files. Since the define is always set when compiling these files it is not necessary to use it in the source files. The usage was originally added during the porting to make it easier to see changes against the base linux version.
>>> 
>>> http://cr.openjdk.java.net/~sla/8001619/webrev.00/
>> 
>> Generally looks okay (though now I see some of the actual BSD code it makes me a little nervous!). A couple of minor things:
> 
> Thanks.
> 
>> 
>> os_bsd.cpp:
>> 
>> 977 }
>> 978 #elif
>> 
>> 3957   return true;
>> 3958 #elif
>> 3959   return false;
>> 
>> Shouldn't the elif's have become an else?
>> 
>> Ditto for os_bsd_x86.cpp:
>> 
>> 850 #elif
> 
> You are right. I've fixed this.
> 
>> Also you can probably do the same thing for bsd_zero: /src/os_cpu/bsd_zero/vm/os_bsd_zero.cpp. Just check with the zero folk.
> 
> I didn't touch the zero files since I wasn't sure who maintains it. I'm not sure how to build it so I didn't want to do changes.
> 
> /Staffan
> 
> 
>> 
>> David
>> -----
>> 
>>> Thanks,
>>> /Staffan
> 



More information about the hotspot-dev mailing list