Re: [PATCH] add: Use struct argv_array in run_add_interactive()

On Sun, Mar 16, 2014 at 7:42 AM, Thomas Rast t...@thomasrast.ch wrote: Fabian Ruch baf...@gmail.com writes: run_add_interactive() in builtin/add.c manually computes array bounds and allocates a static args array to build the add--interactive command line, which is error-prone. Use the

Re: [PATCH] add: Use struct argv_array in run_add_interactive()

Fabian Ruch baf...@gmail.com writes: run_add_interactive() in builtin/add.c manually computes array bounds and allocates a static args array to build the add--interactive command line, which is error-prone. Use the argv-array helper functions instead. Signed-off-by: Fabian Ruch

[PATCH] add: Use struct argv_array in run_add_interactive()

run_add_interactive() in builtin/add.c manually computes array bounds and allocates a static args array to build the add--interactive command line, which is error-prone. Use the argv-array helper functions instead. Signed-off-by: Fabian Ruch baf...@gmail.com --- builtin/add.c | 21

Re: [PATCH] add: Use struct argv_array in run_add_interactive()

On 03/15/2014 12:14 PM, Fabian Ruch wrote: run_add_interactive() in builtin/add.c manually computes array bounds and allocates a static args array to build the add--interactive command line, which is error-prone. Use the argv-array helper functions instead. Signed-off-by: Fabian Ruch