Re: [RFC PATCH 02/14] strvec: introduce `strvec_init_alloc()`

> > It is not satisifying that strvec_init() does *not* become a thin
> > wrapper around this that says "my initial allocation is for zero
> > elements", but that cannot be done easily as a strvec that begins as
> > an empty one has a small optimization to avoid one-slot allocation
> > only to store NULL.  So, ... OK.
>
> Actually, we should do the same optimization if a caller explicitly
> asks
>
> 	strvec_init_alloc(&array, 0);
>
> So perhaps we could do this if we wanted to encapsulate the tricky
> bits in a single place for maintainability.

>  void strvec_init(struct strvec *array)
>  {
> -	struct strvec blank = STRVEC_INIT;
> -	memcpy(array, &blank, sizeof(*array));
> +	strvec_init(array, 0);