svn commit: r235267 - in head/usr.bin/sort: . nls

[Colin Percival]

On 05/11/12 13:08, Warner Losh wrote:
> On May 11, 2012, at 9:02 AM, Colin Percival wrote:
>> On 05/11/12 05:48, Konstantin Belousov wrote:
>>> On Fri, May 11, 2012 at 12:37:16PM +0000, Gabor Kovesdan wrote:
>>>> +bool byte_sort = false; + +static wchar_t **wmonths = NULL; +static
>>>> unsigned char **cmonths = NULL;
>>>
>>> Such initializations are useless. You only increase the size of the binary 
>>> on the disk as the consequence.
>>
>> I just tested this hypothesis, and found no change in binary size using
>> either clang or gcc46.  Presumably they're smart enough to ignore explicit
>> (and unnecessary) initializations of statics to zero.
> 
> How did you test this?  size(1) or ls(1)?  If ls, then you may be running into the page rounding of the .text and .data sections...

size(1), ls(1)-before-strip(1), and ls(1)-after-strip(1).

-- 
Colin Percival
Security Officer, FreeBSD | freebsd.org | The power to serve
Founder / author, Tarsnap | tarsnap.com | Online backups for the truly paranoid