Re: [PATCH v8 1/4] worktree: remove extra members from struct add_opts

2018-04-23 Thread Eric Sunshine
On Mon, Apr 23, 2018 at 3:38 PM, Thomas Gummerer wrote: > There are two members of 'struct add_opts', which are only used inside > the 'add()' function, but being part of 'struct add_opts' they are > needlessly also passed to the 'add_worktree' function. > > Make them local to the 'add()' function

[PATCH v8 1/4] worktree: remove extra members from struct add_opts

2018-04-23 Thread Thomas Gummerer
There are two members of 'struct add_opts', which are only used inside the 'add()' function, but being part of 'struct add_opts' they are needlessly also passed to the 'add_worktree' function. Make them local to the 'add()' function to make it clearer where they are used. Signed-off-by: Thomas Gu