Subject: Re: [PATCH] setup_revisions(): do not access outside argv

From: Miles Bader <[email-masked]>
Date: Thu, 21 May 2009 11:38:25 +0900

Junio C Hamano <[email-masked]> writes:
> If a function takes (int ac, char **av), then people should be able to
> depend on the usual convention of
>
>  (1) for any i < ac, av[i] is not NULL; and
>  (2) av[ac] is NULL.

Hmm, isn't potentially useful to be able to pass a sub-range (of a
longer argv vector) to an ac/av function?  In such a case, av[ac] may
not be NULL.

-Miles

-- 
97% of everything is grunge