Duy Nguyen <pclouds@xxxxxxxxx> writes: > Looking good. I would add some comment, lest ';' feel lonely. But it's > really personal taste. ... which matches mine. Thanks for the update (which I'll squash in). > > -- 8< -- > diff --git a/builtin/worktree.c b/builtin/worktree.c > index 5ebdcce793..bc75676bf3 100644 > --- a/builtin/worktree.c > +++ b/builtin/worktree.c > @@ -310,7 +310,7 @@ static int add_worktree(const char *path, const char *refname, > strbuf_reset(&sb); > strbuf_addf(&sb, "%s/locked", sb_repo.buf); > if (!ret && opts->keep_locked) > - ; > + ; /* --lock wants to keep "locked" file */ > else > unlink_or_warn(sb.buf); > argv_array_clear(&child_env); > -- 8< --