[BusyBox] warning in uniq_main() !?

Steven Scholz steven.scholz at imc-berlin.de
Wed Aug 13 10:55:36 UTC 2003


Philip Blundell wrote:

> On Wed, 2003-08-13 at 11:33, Steven Scholz wrote:
> 
>>-       unsigned long dups, skip_fields, skip_chars, i;
>>-       const char *s0, *e0, *s1, *e1, *input_filename;
>>+       unsigned long dups=0, skip_fields, skip_chars, i;
>>+       const char *s0, *e0=NULL, *s1, *e1, *input_filename;
> 
> 
> Adding an initialiser to a local variable will probably increase the
> code size.  Since that value is never actually used, any extra
> instructions that are added this way would just be wasting space.

Hmm. So adding unessesary warnings would be better than adding one or two bytes?

Steven




More information about the busybox mailing list