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

[Warner Losh]

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...

Warner