On Mon, Mar 26, 2018 at 2:04 PM Stefan Beller <sbeller@xxxxxxxxxx> wrote: > On Fri, Mar 23, 2018 at 8:55 AM Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> wrote: > > > > The argument name "optional" may mislead the reader to think this > > option could be NULL. But it can't be. While at there, document a bit > > more about struct set_gitdir_args. > > > > Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> > Reviewed-by: Stefan Beller <sbeller@xxxxxxxxxx> Uh, not so fast. Is it worthwhile to align the parameter name in the c file as well?