Re: [RFA] Add libiberty/argv.c:countargv

[Ian Lance Taylor]

On Tue, Sep 20, 2011 at 11:23 AM, Doug Evans <d...@google.com> wrote:
>
> 2011-09-20  Doug Evans  <d...@google.com>
>
>        include/
>        * libiberty.h (countargv): Declare.
>
>        libiberty/
>        * argv.c (countargv): New function.


> +  for (argc = 0; argv[argc] != NULL; argc++);

Please write the semicolon on the next line.  I think it is too easy
to get confused when written that way.

  for (argc = 0; argv[argc] != NULL; argc++)
    ;

This is OK with that change.

Thanks.

Ian