Conrad Irwin <conrad.irwin@xxxxxxxxx> writes:
> diff --git a/builtin/add.c b/builtin/add.c
> index d39a6ab..f02524b 100644
> --- a/builtin/add.c
> +++ b/builtin/add.c
> @@ -241,7 +241,7 @@ int run_add_interactive(const char *revision, const char *patch_mode,
> return status;
> }
>
> -int interactive_add(int argc, const char **argv, const char *prefix)
> +int interactive_add(int argc, const char **argv, const char *prefix, int patch)
> {
> const char **pathspec = NULL;
>
> @@ -252,7 +252,7 @@ int interactive_add(int argc, const char **argv, const char *prefix)
> }
>
> return run_add_interactive(NULL,
> - patch_interactive ? "--patch" : NULL,
> + patch ? "--patch" : NULL,
> pathspec);
> }
This removes one reason for patch_interactive to be a file level global
variable. Perhaps we should plan to move builtin_add_options[] and the
file level global variables in builtin/add.c to cmd_add() and turn them
into on-stack variables.
It obviously is a totally separate topic, though.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html