Re: [PATCH v4 05/11] replace: introduce --convert-graft-file

> On Sat, Apr 21, 2018 at 5:48 AM, Johannes Schindelin
> <johannes.schindelin@gmx.de> wrote:
> > This option is intended to help with the transition away from the
> > now-deprecated graft file.
> >
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> > diff --git a/builtin/replace.c b/builtin/replace.c
> > @@ -454,6 +455,38 @@ static int create_graft(int argc, const char **argv, int force)
> > +static int convert_graft_file(int force)
> > +{
> > +       const char *graft_file = get_graft_file();
> > +       FILE *fp = fopen_or_warn(graft_file, "r");
> > +       struct strbuf buf = STRBUF_INIT, err = STRBUF_INIT;
> > +       struct argv_array args = ARGV_ARRAY_INIT;
> > +
> > +       if (!fp)
> > +               return -1;
> > +
> > +       while (strbuf_getline(&buf, fp) != EOF) {
> > +               if (*buf.buf == '#')
> > +                       continue;
> > +
> > +               argv_array_split(&args, buf.buf);
> > +               if (args.argc && create_graft(args.argc, args.argv, force))
> > +                       strbuf_addf(&err, "\n\t%s", buf.buf);
> > +               argv_array_clear(&args);
> > +       }
> > +
> > +       strbuf_release(&buf);
> > +       argv_array_clear(&args);
> 
> This argv_array_clear() is redundant, isn't it?