From: Phillip Wood <phillip.wood123@gmail•com>
To: Patrick Steinhardt <ps@pks•im>, git@vger•kernel.org
Cc: "D. Ben Knoble" <ben.knoble@gmail•com>,
"Junio C Hamano" <gitster@pobox•com>,
"Sergey Organov" <sorganov@gmail•com>,
"Jean-Noël AVILA" <jn.avila@free•fr>,
"Martin von Zweigbergk" <martinvonz@gmail•com>,
"Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail•com>,
"Elijah Newren" <newren@gmail•com>,
"Karthik Nayak" <karthik.188@gmail•com>
Subject: Re: [PATCH v6 07/11] add-patch: split out `struct interactive_options`
Date: Thu, 20 Nov 2025 15:05:17 +0000 [thread overview]
Message-ID: <3872545c-6738-4852-ab2c-8c08525ff795@gmail.com> (raw)
In-Reply-To: <20251027-b4-pks-history-builtin-v6-7-407dd3f57ad3@pks.im>
On 27/10/2025 11:33, Patrick Steinhardt wrote:
> The `struct add_p_opt` is reused both by our infra for "git add -p" and
> "git add -i". Users of `run_add_i()` for example are expected to pass
> `struct add_p_opt`. This is somewhat confusing and raises the question
> of which options apply to what part of the stack.
To some extent the config setting are intertwined because run_add_i() is
a superset of run_add_p(). Callers of run_add_i() needs to initalize an
instance of "struct add_p_opt" because they are indirectly calling
run_add_p().
> But things are even more confusing than that: while callers are expected
> to pass in `struct add_p_opt`, these options ultimately get used to
> initialize a `struct add_i_state` that is used by both subsystems. So we
> are basically going full circle here.
It is certainly confusing that we have to initalize a "struct
add_i_state" in run_add_p(). struct add_p_opt is only consumed in
add-patch.c, the reason it apperas in add-interactive.c is that
run_add_i() needs to pass it along to run_add_p().
> Refactor the code and split out a new `struct interactive_options` that
> hosts common options used by both. These options are then applied to a
> `struct interactive_config` that hosts common configuration.
I'm a little skeptical about renaming "sturct add_p_opt" as it only
holds members that are relavent to run_add_p(). Also if we're trying to
draw clear boundaries between the two subsystems hosting "struct
interactive_options" and "struct interactive_config" in add-patch.c
rather than add-interactive.c is potentially confusing.
> This refactoring doesn't yet fully detangle the two subsystems from one
> another, as we still end up calling `init_add_i_state()` in the "git add
> -p" subsystem. This will be fixed in a subsequent commit.
I think the ultimate aim of not having to initalize a "struct
add_i_state" in run_add_p() is a good idea. I'm not sure though that
having to pass a "struct interactive_options" to run_add_p() is any less
confusing than having to pass a "struct add_p_opt" to run_add_i().
Thanks
Phillip
> Signed-off-by: Patrick Steinhardt <ps@pks•im>
> ---
> add-interactive.c | 174 +++++++++++------------------------------------------
> add-interactive.h | 23 +------
> add-patch.c | 170 +++++++++++++++++++++++++++++++++++++++++++--------
> add-patch.h | 36 ++++++++++-
> builtin/add.c | 22 +++----
> builtin/checkout.c | 4 +-
> builtin/commit.c | 16 ++---
> builtin/reset.c | 16 ++---
> builtin/stash.c | 46 +++++++-------
> commit.h | 2 +-
> 10 files changed, 270 insertions(+), 239 deletions(-)
>
> diff --git a/add-interactive.c b/add-interactive.c
> index 68fc09547dd..05d2e7eefe3 100644
> --- a/add-interactive.c
> +++ b/add-interactive.c
> @@ -3,7 +3,6 @@
> #include "git-compat-util.h"
> #include "add-interactive.h"
> #include "color.h"
> -#include "config.h"
> #include "diffcore.h"
> #include "gettext.h"
> #include "hash.h"
> @@ -20,119 +19,18 @@
> #include "prompt.h"
> #include "tree.h"
>
> -static void init_color(struct repository *r, enum git_colorbool use_color,
> - const char *section_and_slot, char *dst,
> - const char *default_color)
> -{
> - char *key = xstrfmt("color.%s", section_and_slot);
> - const char *value;
> -
> - if (!want_color(use_color))
> - dst[0] = '\0';
> - else if (repo_config_get_value(r, key, &value) ||
> - color_parse(value, dst))
> - strlcpy(dst, default_color, COLOR_MAXLEN);
> -
> - free(key);
> -}
> -
> -static enum git_colorbool check_color_config(struct repository *r, const char *var)
> -{
> - const char *value;
> - enum git_colorbool ret;
> -
> - if (repo_config_get_value(r, var, &value))
> - ret = GIT_COLOR_UNKNOWN;
> - else
> - ret = git_config_colorbool(var, value);
> -
> - /*
> - * Do not rely on want_color() to fall back to color.ui for us. It uses
> - * the value parsed by git_color_config(), which may not have been
> - * called by the main command.
> - */
> - if (ret == GIT_COLOR_UNKNOWN &&
> - !repo_config_get_value(r, "color.ui", &value))
> - ret = git_config_colorbool("color.ui", value);
> -
> - return ret;
> -}
> -
> void init_add_i_state(struct add_i_state *s, struct repository *r,
> - struct add_p_opt *add_p_opt)
> + struct interactive_options *opts)
> {
> s->r = r;
> - s->context = -1;
> - s->interhunkcontext = -1;
> -
> - s->use_color_interactive = check_color_config(r, "color.interactive");
> -
> - init_color(r, s->use_color_interactive, "interactive.header",
> - s->header_color, GIT_COLOR_BOLD);
> - init_color(r, s->use_color_interactive, "interactive.help",
> - s->help_color, GIT_COLOR_BOLD_RED);
> - init_color(r, s->use_color_interactive, "interactive.prompt",
> - s->prompt_color, GIT_COLOR_BOLD_BLUE);
> - init_color(r, s->use_color_interactive, "interactive.error",
> - s->error_color, GIT_COLOR_BOLD_RED);
> - strlcpy(s->reset_color_interactive,
> - want_color(s->use_color_interactive) ? GIT_COLOR_RESET : "", COLOR_MAXLEN);
> -
> - s->use_color_diff = check_color_config(r, "color.diff");
> -
> - init_color(r, s->use_color_diff, "diff.frag", s->fraginfo_color,
> - diff_get_color(s->use_color_diff, DIFF_FRAGINFO));
> - init_color(r, s->use_color_diff, "diff.context", s->context_color,
> - "fall back");
> - if (!strcmp(s->context_color, "fall back"))
> - init_color(r, s->use_color_diff, "diff.plain",
> - s->context_color,
> - diff_get_color(s->use_color_diff, DIFF_CONTEXT));
> - init_color(r, s->use_color_diff, "diff.old", s->file_old_color,
> - diff_get_color(s->use_color_diff, DIFF_FILE_OLD));
> - init_color(r, s->use_color_diff, "diff.new", s->file_new_color,
> - diff_get_color(s->use_color_diff, DIFF_FILE_NEW));
> - strlcpy(s->reset_color_diff,
> - want_color(s->use_color_diff) ? GIT_COLOR_RESET : "", COLOR_MAXLEN);
> -
> - FREE_AND_NULL(s->interactive_diff_filter);
> - repo_config_get_string(r, "interactive.difffilter",
> - &s->interactive_diff_filter);
> -
> - FREE_AND_NULL(s->interactive_diff_algorithm);
> - repo_config_get_string(r, "diff.algorithm",
> - &s->interactive_diff_algorithm);
> -
> - if (!repo_config_get_int(r, "diff.context", &s->context))
> - if (s->context < 0)
> - die(_("%s cannot be negative"), "diff.context");
> - if (!repo_config_get_int(r, "diff.interHunkContext", &s->interhunkcontext))
> - if (s->interhunkcontext < 0)
> - die(_("%s cannot be negative"), "diff.interHunkContext");
> -
> - repo_config_get_bool(r, "interactive.singlekey", &s->use_single_key);
> - if (s->use_single_key)
> - setbuf(stdin, NULL);
> -
> - if (add_p_opt->context != -1) {
> - if (add_p_opt->context < 0)
> - die(_("%s cannot be negative"), "--unified");
> - s->context = add_p_opt->context;
> - }
> - if (add_p_opt->interhunkcontext != -1) {
> - if (add_p_opt->interhunkcontext < 0)
> - die(_("%s cannot be negative"), "--inter-hunk-context");
> - s->interhunkcontext = add_p_opt->interhunkcontext;
> - }
> + interactive_config_init(&s->cfg, r, opts);
> }
>
> void clear_add_i_state(struct add_i_state *s)
> {
> - FREE_AND_NULL(s->interactive_diff_filter);
> - FREE_AND_NULL(s->interactive_diff_algorithm);
> + interactive_config_clear(&s->cfg);
> memset(s, 0, sizeof(*s));
> - s->use_color_interactive = GIT_COLOR_UNKNOWN;
> - s->use_color_diff = GIT_COLOR_UNKNOWN;
> + interactive_config_clear(&s->cfg);
> }
>
> /*
> @@ -286,7 +184,7 @@ static void list(struct add_i_state *s, struct string_list *list, int *selected,
> return;
>
> if (opts->header)
> - color_fprintf_ln(stdout, s->header_color,
> + color_fprintf_ln(stdout, s->cfg.header_color,
> "%s", opts->header);
>
> for (i = 0; i < list->nr; i++) {
> @@ -354,7 +252,7 @@ static ssize_t list_and_choose(struct add_i_state *s,
>
> list(s, &items->items, items->selected, &opts->list_opts);
>
> - color_fprintf(stdout, s->prompt_color, "%s", opts->prompt);
> + color_fprintf(stdout, s->cfg.prompt_color, "%s", opts->prompt);
> fputs(singleton ? "> " : ">> ", stdout);
> fflush(stdout);
>
> @@ -432,7 +330,7 @@ static ssize_t list_and_choose(struct add_i_state *s,
>
> if (from < 0 || from >= items->items.nr ||
> (singleton && from + 1 != to)) {
> - color_fprintf_ln(stderr, s->error_color,
> + color_fprintf_ln(stderr, s->cfg.error_color,
> _("Huh (%s)?"), p);
> break;
> } else if (singleton) {
> @@ -992,7 +890,7 @@ static int run_patch(struct add_i_state *s, const struct pathspec *ps,
> free(files->items.items[i].string);
> } else if (item->index.unmerged ||
> item->worktree.unmerged) {
> - color_fprintf_ln(stderr, s->error_color,
> + color_fprintf_ln(stderr, s->cfg.error_color,
> _("ignoring unmerged: %s"),
> files->items.items[i].string);
> free(item);
> @@ -1014,9 +912,9 @@ static int run_patch(struct add_i_state *s, const struct pathspec *ps,
> opts->prompt = N_("Patch update");
> count = list_and_choose(s, files, opts);
> if (count > 0) {
> - struct add_p_opt add_p_opt = {
> - .context = s->context,
> - .interhunkcontext = s->interhunkcontext,
> + struct interactive_options opts = {
> + .context = s->cfg.context,
> + .interhunkcontext = s->cfg.interhunkcontext,
> };
> struct strvec args = STRVEC_INIT;
> struct pathspec ps_selected = { 0 };
> @@ -1028,7 +926,7 @@ static int run_patch(struct add_i_state *s, const struct pathspec *ps,
> parse_pathspec(&ps_selected,
> PATHSPEC_ALL_MAGIC & ~PATHSPEC_LITERAL,
> PATHSPEC_LITERAL_PATH, "", args.v);
> - res = run_add_p(s->r, ADD_P_ADD, &add_p_opt, NULL, &ps_selected);
> + res = run_add_p(s->r, ADD_P_ADD, &opts, NULL, &ps_selected);
> strvec_clear(&args);
> clear_pathspec(&ps_selected);
> }
> @@ -1064,10 +962,10 @@ static int run_diff(struct add_i_state *s, const struct pathspec *ps,
> struct child_process cmd = CHILD_PROCESS_INIT;
>
> strvec_pushl(&cmd.args, "git", "diff", "-p", "--cached", NULL);
> - if (s->context != -1)
> - strvec_pushf(&cmd.args, "--unified=%i", s->context);
> - if (s->interhunkcontext != -1)
> - strvec_pushf(&cmd.args, "--inter-hunk-context=%i", s->interhunkcontext);
> + if (s->cfg.context != -1)
> + strvec_pushf(&cmd.args, "--unified=%i", s->cfg.context);
> + if (s->cfg.interhunkcontext != -1)
> + strvec_pushf(&cmd.args, "--inter-hunk-context=%i", s->cfg.interhunkcontext);
> strvec_pushl(&cmd.args, oid_to_hex(!is_initial ? &oid :
> s->r->hash_algo->empty_tree), "--", NULL);
> for (i = 0; i < files->items.nr; i++)
> @@ -1085,17 +983,17 @@ static int run_help(struct add_i_state *s, const struct pathspec *ps UNUSED,
> struct prefix_item_list *files UNUSED,
> struct list_and_choose_options *opts UNUSED)
> {
> - color_fprintf_ln(stdout, s->help_color, "status - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "status - %s",
> _("show paths with changes"));
> - color_fprintf_ln(stdout, s->help_color, "update - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "update - %s",
> _("add working tree state to the staged set of changes"));
> - color_fprintf_ln(stdout, s->help_color, "revert - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "revert - %s",
> _("revert staged set of changes back to the HEAD version"));
> - color_fprintf_ln(stdout, s->help_color, "patch - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "patch - %s",
> _("pick hunks and update selectively"));
> - color_fprintf_ln(stdout, s->help_color, "diff - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "diff - %s",
> _("view diff between HEAD and index"));
> - color_fprintf_ln(stdout, s->help_color, "add untracked - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "add untracked - %s",
> _("add contents of untracked files to the staged set of changes"));
>
> return 0;
> @@ -1103,21 +1001,21 @@ static int run_help(struct add_i_state *s, const struct pathspec *ps UNUSED,
>
> static void choose_prompt_help(struct add_i_state *s)
> {
> - color_fprintf_ln(stdout, s->help_color, "%s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "%s",
> _("Prompt help:"));
> - color_fprintf_ln(stdout, s->help_color, "1 - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "1 - %s",
> _("select a single item"));
> - color_fprintf_ln(stdout, s->help_color, "3-5 - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "3-5 - %s",
> _("select a range of items"));
> - color_fprintf_ln(stdout, s->help_color, "2-3,6-9 - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "2-3,6-9 - %s",
> _("select multiple ranges"));
> - color_fprintf_ln(stdout, s->help_color, "foo - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "foo - %s",
> _("select item based on unique prefix"));
> - color_fprintf_ln(stdout, s->help_color, "-... - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "-... - %s",
> _("unselect specified items"));
> - color_fprintf_ln(stdout, s->help_color, "* - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, "* - %s",
> _("choose all items"));
> - color_fprintf_ln(stdout, s->help_color, " - %s",
> + color_fprintf_ln(stdout, s->cfg.help_color, " - %s",
> _("(empty) finish selecting"));
> }
>
> @@ -1152,7 +1050,7 @@ static void print_command_item(int i, int selected UNUSED,
>
> static void command_prompt_help(struct add_i_state *s)
> {
> - const char *help_color = s->help_color;
> + const char *help_color = s->cfg.help_color;
> color_fprintf_ln(stdout, help_color, "%s", _("Prompt help:"));
> color_fprintf_ln(stdout, help_color, "1 - %s",
> _("select a numbered item"));
> @@ -1163,7 +1061,7 @@ static void command_prompt_help(struct add_i_state *s)
> }
>
> int run_add_i(struct repository *r, const struct pathspec *ps,
> - struct add_p_opt *add_p_opt)
> + struct interactive_options *interactive_opts)
> {
> struct add_i_state s = { NULL };
> struct print_command_item_data data = { "[", "]" };
> @@ -1206,15 +1104,15 @@ int run_add_i(struct repository *r, const struct pathspec *ps,
> ->util = util;
> }
>
> - init_add_i_state(&s, r, add_p_opt);
> + init_add_i_state(&s, r, interactive_opts);
>
> /*
> * When color was asked for, use the prompt color for
> * highlighting, otherwise use square brackets.
> */
> - if (want_color(s.use_color_interactive)) {
> - data.color = s.prompt_color;
> - data.reset = s.reset_color_interactive;
> + if (want_color(s.cfg.use_color_interactive)) {
> + data.color = s.cfg.prompt_color;
> + data.reset = s.cfg.reset_color_interactive;
> }
> print_file_item_data.color = data.color;
> print_file_item_data.reset = data.reset;
> diff --git a/add-interactive.h b/add-interactive.h
> index 2e3d1d871d2..eefa2edc7c1 100644
> --- a/add-interactive.h
> +++ b/add-interactive.h
> @@ -2,37 +2,20 @@
> #define ADD_INTERACTIVE_H
>
> #include "add-patch.h"
> -#include "color.h"
>
> struct pathspec;
> struct repository;
>
> struct add_i_state {
> struct repository *r;
> - enum git_colorbool use_color_interactive;
> - enum git_colorbool use_color_diff;
> - char header_color[COLOR_MAXLEN];
> - char help_color[COLOR_MAXLEN];
> - char prompt_color[COLOR_MAXLEN];
> - char error_color[COLOR_MAXLEN];
> - char reset_color_interactive[COLOR_MAXLEN];
> -
> - char fraginfo_color[COLOR_MAXLEN];
> - char context_color[COLOR_MAXLEN];
> - char file_old_color[COLOR_MAXLEN];
> - char file_new_color[COLOR_MAXLEN];
> - char reset_color_diff[COLOR_MAXLEN];
> -
> - int use_single_key;
> - char *interactive_diff_filter, *interactive_diff_algorithm;
> - int context, interhunkcontext;
> + struct interactive_config cfg;
> };
>
> void init_add_i_state(struct add_i_state *s, struct repository *r,
> - struct add_p_opt *add_p_opt);
> + struct interactive_options *opts);
> void clear_add_i_state(struct add_i_state *s);
>
> int run_add_i(struct repository *r, const struct pathspec *ps,
> - struct add_p_opt *add_p_opt);
> + struct interactive_options *opts);
>
> #endif
> diff --git a/add-patch.c b/add-patch.c
> index 3594dd22534..5c6969927ac 100644
> --- a/add-patch.c
> +++ b/add-patch.c
> @@ -5,6 +5,8 @@
> #include "add-interactive.h"
> #include "add-patch.h"
> #include "advice.h"
> +#include "config.h"
> +#include "diff.h"
> #include "editor.h"
> #include "environment.h"
> #include "gettext.h"
> @@ -279,6 +281,122 @@ struct add_p_state {
> const char *revision;
> };
>
> +static void init_color(struct repository *r,
> + enum git_colorbool use_color,
> + const char *section_and_slot, char *dst,
> + const char *default_color)
> +{
> + char *key = xstrfmt("color.%s", section_and_slot);
> + const char *value;
> +
> + if (!want_color(use_color))
> + dst[0] = '\0';
> + else if (repo_config_get_value(r, key, &value) ||
> + color_parse(value, dst))
> + strlcpy(dst, default_color, COLOR_MAXLEN);
> +
> + free(key);
> +}
> +
> +static enum git_colorbool check_color_config(struct repository *r, const char *var)
> +{
> + const char *value;
> + enum git_colorbool ret;
> +
> + if (repo_config_get_value(r, var, &value))
> + ret = GIT_COLOR_UNKNOWN;
> + else
> + ret = git_config_colorbool(var, value);
> +
> + /*
> + * Do not rely on want_color() to fall back to color.ui for us. It uses
> + * the value parsed by git_color_config(), which may not have been
> + * called by the main command.
> + */
> + if (ret == GIT_COLOR_UNKNOWN &&
> + !repo_config_get_value(r, "color.ui", &value))
> + ret = git_config_colorbool("color.ui", value);
> +
> + return ret;
> +}
> +
> +void interactive_config_init(struct interactive_config *cfg,
> + struct repository *r,
> + struct interactive_options *opts)
> +{
> + cfg->context = -1;
> + cfg->interhunkcontext = -1;
> +
> + cfg->use_color_interactive = check_color_config(r, "color.interactive");
> +
> + init_color(r, cfg->use_color_interactive, "interactive.header",
> + cfg->header_color, GIT_COLOR_BOLD);
> + init_color(r, cfg->use_color_interactive, "interactive.help",
> + cfg->help_color, GIT_COLOR_BOLD_RED);
> + init_color(r, cfg->use_color_interactive, "interactive.prompt",
> + cfg->prompt_color, GIT_COLOR_BOLD_BLUE);
> + init_color(r, cfg->use_color_interactive, "interactive.error",
> + cfg->error_color, GIT_COLOR_BOLD_RED);
> + strlcpy(cfg->reset_color_interactive,
> + want_color(cfg->use_color_interactive) ? GIT_COLOR_RESET : "", COLOR_MAXLEN);
> +
> + cfg->use_color_diff = check_color_config(r, "color.diff");
> +
> + init_color(r, cfg->use_color_diff, "diff.frag", cfg->fraginfo_color,
> + diff_get_color(cfg->use_color_diff, DIFF_FRAGINFO));
> + init_color(r, cfg->use_color_diff, "diff.context", cfg->context_color,
> + "fall back");
> + if (!strcmp(cfg->context_color, "fall back"))
> + init_color(r, cfg->use_color_diff, "diff.plain",
> + cfg->context_color,
> + diff_get_color(cfg->use_color_diff, DIFF_CONTEXT));
> + init_color(r, cfg->use_color_diff, "diff.old", cfg->file_old_color,
> + diff_get_color(cfg->use_color_diff, DIFF_FILE_OLD));
> + init_color(r, cfg->use_color_diff, "diff.new", cfg->file_new_color,
> + diff_get_color(cfg->use_color_diff, DIFF_FILE_NEW));
> + strlcpy(cfg->reset_color_diff,
> + want_color(cfg->use_color_diff) ? GIT_COLOR_RESET : "", COLOR_MAXLEN);
> +
> + FREE_AND_NULL(cfg->interactive_diff_filter);
> + repo_config_get_string(r, "interactive.difffilter",
> + &cfg->interactive_diff_filter);
> +
> + FREE_AND_NULL(cfg->interactive_diff_algorithm);
> + repo_config_get_string(r, "diff.algorithm",
> + &cfg->interactive_diff_algorithm);
> +
> + if (!repo_config_get_int(r, "diff.context", &cfg->context))
> + if (cfg->context < 0)
> + die(_("%s cannot be negative"), "diff.context");
> + if (!repo_config_get_int(r, "diff.interHunkContext", &cfg->interhunkcontext))
> + if (cfg->interhunkcontext < 0)
> + die(_("%s cannot be negative"), "diff.interHunkContext");
> +
> + repo_config_get_bool(r, "interactive.singlekey", &cfg->use_single_key);
> + if (cfg->use_single_key)
> + setbuf(stdin, NULL);
> +
> + if (opts->context != -1) {
> + if (opts->context < 0)
> + die(_("%s cannot be negative"), "--unified");
> + cfg->context = opts->context;
> + }
> + if (opts->interhunkcontext != -1) {
> + if (opts->interhunkcontext < 0)
> + die(_("%s cannot be negative"), "--inter-hunk-context");
> + cfg->interhunkcontext = opts->interhunkcontext;
> + }
> +}
> +
> +void interactive_config_clear(struct interactive_config *cfg)
> +{
> + FREE_AND_NULL(cfg->interactive_diff_filter);
> + FREE_AND_NULL(cfg->interactive_diff_algorithm);
> + memset(cfg, 0, sizeof(*cfg));
> + cfg->use_color_interactive = GIT_COLOR_UNKNOWN;
> + cfg->use_color_diff = GIT_COLOR_UNKNOWN;
> +}
> +
> static void add_p_state_clear(struct add_p_state *s)
> {
> size_t i;
> @@ -299,9 +417,9 @@ static void err(struct add_p_state *s, const char *fmt, ...)
> va_list args;
>
> va_start(args, fmt);
> - fputs(s->s.error_color, stdout);
> + fputs(s->s.cfg.error_color, stdout);
> vprintf(fmt, args);
> - puts(s->s.reset_color_interactive);
> + puts(s->s.cfg.reset_color_interactive);
> va_end(args);
> }
>
> @@ -424,12 +542,12 @@ static int parse_diff(struct add_p_state *s, const struct pathspec *ps)
> int res;
>
> strvec_pushv(&args, s->mode->diff_cmd);
> - if (s->s.context != -1)
> - strvec_pushf(&args, "--unified=%i", s->s.context);
> - if (s->s.interhunkcontext != -1)
> - strvec_pushf(&args, "--inter-hunk-context=%i", s->s.interhunkcontext);
> - if (s->s.interactive_diff_algorithm)
> - strvec_pushf(&args, "--diff-algorithm=%s", s->s.interactive_diff_algorithm);
> + if (s->s.cfg.context != -1)
> + strvec_pushf(&args, "--unified=%i", s->s.cfg.context);
> + if (s->s.cfg.interhunkcontext != -1)
> + strvec_pushf(&args, "--inter-hunk-context=%i", s->s.cfg.interhunkcontext);
> + if (s->s.cfg.interactive_diff_algorithm)
> + strvec_pushf(&args, "--diff-algorithm=%s", s->s.cfg.interactive_diff_algorithm);
> if (s->revision) {
> struct object_id oid;
> strvec_push(&args,
> @@ -458,9 +576,9 @@ static int parse_diff(struct add_p_state *s, const struct pathspec *ps)
> }
> strbuf_complete_line(plain);
>
> - if (want_color_fd(1, s->s.use_color_diff)) {
> + if (want_color_fd(1, s->s.cfg.use_color_diff)) {
> struct child_process colored_cp = CHILD_PROCESS_INIT;
> - const char *diff_filter = s->s.interactive_diff_filter;
> + const char *diff_filter = s->s.cfg.interactive_diff_filter;
>
> setup_child_process(s, &colored_cp, NULL);
> xsnprintf((char *)args.v[color_arg_index], 8, "--color");
> @@ -693,7 +811,7 @@ static void render_hunk(struct add_p_state *s, struct hunk *hunk,
> hunk->colored_end - hunk->colored_start);
> return;
> } else {
> - strbuf_addstr(out, s->s.fraginfo_color);
> + strbuf_addstr(out, s->s.cfg.fraginfo_color);
> p = s->colored.buf + header->colored_extra_start;
> len = header->colored_extra_end
> - header->colored_extra_start;
> @@ -715,7 +833,7 @@ static void render_hunk(struct add_p_state *s, struct hunk *hunk,
> if (len)
> strbuf_add(out, p, len);
> else if (colored)
> - strbuf_addf(out, "%s\n", s->s.reset_color_diff);
> + strbuf_addf(out, "%s\n", s->s.cfg.reset_color_diff);
> else
> strbuf_addch(out, '\n');
> }
> @@ -1104,12 +1222,12 @@ static void recolor_hunk(struct add_p_state *s, struct hunk *hunk)
>
> strbuf_addstr(&s->colored,
> plain[current] == '-' ?
> - s->s.file_old_color :
> + s->s.cfg.file_old_color :
> plain[current] == '+' ?
> - s->s.file_new_color :
> - s->s.context_color);
> + s->s.cfg.file_new_color :
> + s->s.cfg.context_color);
> strbuf_add(&s->colored, plain + current, eol - current);
> - strbuf_addstr(&s->colored, s->s.reset_color_diff);
> + strbuf_addstr(&s->colored, s->s.cfg.reset_color_diff);
> if (next > eol)
> strbuf_add(&s->colored, plain + eol, next - eol);
> current = next;
> @@ -1238,7 +1356,7 @@ static int run_apply_check(struct add_p_state *s,
>
> static int read_single_character(struct add_p_state *s)
> {
> - if (s->s.use_single_key) {
> + if (s->s.cfg.use_single_key) {
> int res = read_key_without_echo(&s->answer);
> printf("%s\n", res == EOF ? "" : s->answer.buf);
> return res;
> @@ -1252,7 +1370,7 @@ static int read_single_character(struct add_p_state *s)
> static int prompt_yesno(struct add_p_state *s, const char *prompt)
> {
> for (;;) {
> - color_fprintf(stdout, s->s.prompt_color, "%s", _(prompt));
> + color_fprintf(stdout, s->s.cfg.prompt_color, "%s", _(prompt));
> fflush(stdout);
> if (read_single_character(s) == EOF)
> return -1;
> @@ -1560,15 +1678,15 @@ static int patch_update_file(struct add_p_state *s,
> else
> prompt_mode_type = PROMPT_HUNK;
>
> - printf("%s(%"PRIuMAX"/%"PRIuMAX") ", s->s.prompt_color,
> + printf("%s(%"PRIuMAX"/%"PRIuMAX") ", s->s.cfg.prompt_color,
> (uintmax_t)hunk_index + 1,
> (uintmax_t)(file_diff->hunk_nr
> ? file_diff->hunk_nr
> : 1));
> printf(_(s->mode->prompt_mode[prompt_mode_type]),
> s->buf.buf);
> - if (*s->s.reset_color_interactive)
> - fputs(s->s.reset_color_interactive, stdout);
> + if (*s->s.cfg.reset_color_interactive)
> + fputs(s->s.cfg.reset_color_interactive, stdout);
> fflush(stdout);
> if (read_single_character(s) == EOF)
> break;
> @@ -1730,7 +1848,7 @@ static int patch_update_file(struct add_p_state *s,
> err(s, _("Sorry, cannot split this hunk"));
> } else if (!split_hunk(s, file_diff,
> hunk - file_diff->hunk)) {
> - color_fprintf_ln(stdout, s->s.header_color,
> + color_fprintf_ln(stdout, s->s.cfg.header_color,
> _("Split into %d hunks."),
> (int)splittable_into);
> rendered_hunk_index = -1;
> @@ -1748,7 +1866,7 @@ static int patch_update_file(struct add_p_state *s,
> } else if (s->answer.buf[0] == '?') {
> const char *p = _(help_patch_remainder), *eol = p;
>
> - color_fprintf(stdout, s->s.help_color, "%s",
> + color_fprintf(stdout, s->s.cfg.help_color, "%s",
> _(s->mode->help_patch_text));
>
> /*
> @@ -1766,7 +1884,7 @@ static int patch_update_file(struct add_p_state *s,
> if (*p != '?' && !strchr(s->buf.buf, *p))
> continue;
>
> - color_fprintf_ln(stdout, s->s.help_color,
> + color_fprintf_ln(stdout, s->s.cfg.help_color,
> "%.*s", (int)(eol - p), p);
> }
> } else {
> @@ -1807,7 +1925,7 @@ static int patch_update_file(struct add_p_state *s,
> }
>
> int run_add_p(struct repository *r, enum add_p_mode mode,
> - struct add_p_opt *o, const char *revision,
> + struct interactive_options *opts, const char *revision,
> const struct pathspec *ps)
> {
> struct add_p_state s = {
> @@ -1815,7 +1933,7 @@ int run_add_p(struct repository *r, enum add_p_mode mode,
> };
> size_t i, binary_count = 0;
>
> - init_add_i_state(&s.s, r, o);
> + init_add_i_state(&s.s, r, opts);
>
> if (mode == ADD_P_STASH)
> s.mode = &patch_mode_stash;
> diff --git a/add-patch.h b/add-patch.h
> index 4394c741076..a4a05d9d145 100644
> --- a/add-patch.h
> +++ b/add-patch.h
> @@ -1,15 +1,45 @@
> #ifndef ADD_PATCH_H
> #define ADD_PATCH_H
>
> +#include "color.h"
> +
> struct pathspec;
> struct repository;
>
> -struct add_p_opt {
> +struct interactive_options {
> int context;
> int interhunkcontext;
> };
>
> -#define ADD_P_OPT_INIT { .context = -1, .interhunkcontext = -1 }
> +#define INTERACTIVE_OPTIONS_INIT { \
> + .context = -1, \
> + .interhunkcontext = -1, \
> +}
> +
> +struct interactive_config {
> + enum git_colorbool use_color_interactive;
> + enum git_colorbool use_color_diff;
> + char header_color[COLOR_MAXLEN];
> + char help_color[COLOR_MAXLEN];
> + char prompt_color[COLOR_MAXLEN];
> + char error_color[COLOR_MAXLEN];
> + char reset_color_interactive[COLOR_MAXLEN];
> +
> + char fraginfo_color[COLOR_MAXLEN];
> + char context_color[COLOR_MAXLEN];
> + char file_old_color[COLOR_MAXLEN];
> + char file_new_color[COLOR_MAXLEN];
> + char reset_color_diff[COLOR_MAXLEN];
> +
> + int use_single_key;
> + char *interactive_diff_filter, *interactive_diff_algorithm;
> + int context, interhunkcontext;
> +};
> +
> +void interactive_config_init(struct interactive_config *cfg,
> + struct repository *r,
> + struct interactive_options *opts);
> +void interactive_config_clear(struct interactive_config *cfg);
>
> enum add_p_mode {
> ADD_P_ADD,
> @@ -20,7 +50,7 @@ enum add_p_mode {
> };
>
> int run_add_p(struct repository *r, enum add_p_mode mode,
> - struct add_p_opt *o, const char *revision,
> + struct interactive_options *opts, const char *revision,
> const struct pathspec *ps);
>
> #endif
> diff --git a/builtin/add.c b/builtin/add.c
> index 32709794b38..6f1e2130528 100644
> --- a/builtin/add.c
> +++ b/builtin/add.c
> @@ -31,7 +31,7 @@ static const char * const builtin_add_usage[] = {
> NULL
> };
> static int patch_interactive, add_interactive, edit_interactive;
> -static struct add_p_opt add_p_opt = ADD_P_OPT_INIT;
> +static struct interactive_options interactive_opts = INTERACTIVE_OPTIONS_INIT;
> static int take_worktree_changes;
> static int add_renormalize;
> static int pathspec_file_nul;
> @@ -160,7 +160,7 @@ static int refresh(struct repository *repo, int verbose, const struct pathspec *
> int interactive_add(struct repository *repo,
> const char **argv,
> const char *prefix,
> - int patch, struct add_p_opt *add_p_opt)
> + int patch, struct interactive_options *interactive_opts)
> {
> struct pathspec pathspec;
> int ret;
> @@ -172,9 +172,9 @@ int interactive_add(struct repository *repo,
> prefix, argv);
>
> if (patch)
> - ret = !!run_add_p(repo, ADD_P_ADD, add_p_opt, NULL, &pathspec);
> + ret = !!run_add_p(repo, ADD_P_ADD, interactive_opts, NULL, &pathspec);
> else
> - ret = !!run_add_i(repo, &pathspec, add_p_opt);
> + ret = !!run_add_i(repo, &pathspec, interactive_opts);
>
> clear_pathspec(&pathspec);
> return ret;
> @@ -256,8 +256,8 @@ static struct option builtin_add_options[] = {
> OPT_GROUP(""),
> OPT_BOOL('i', "interactive", &add_interactive, N_("interactive picking")),
> OPT_BOOL('p', "patch", &patch_interactive, N_("select hunks interactively")),
> - OPT_DIFF_UNIFIED(&add_p_opt.context),
> - OPT_DIFF_INTERHUNK_CONTEXT(&add_p_opt.interhunkcontext),
> + OPT_DIFF_UNIFIED(&interactive_opts.context),
> + OPT_DIFF_INTERHUNK_CONTEXT(&interactive_opts.interhunkcontext),
> OPT_BOOL('e', "edit", &edit_interactive, N_("edit current diff and apply")),
> OPT__FORCE(&ignored_too, N_("allow adding otherwise ignored files"), 0),
> OPT_BOOL('u', "update", &take_worktree_changes, N_("update tracked files")),
> @@ -400,9 +400,9 @@ int cmd_add(int argc,
> prepare_repo_settings(repo);
> repo->settings.command_requires_full_index = 0;
>
> - if (add_p_opt.context < -1)
> + if (interactive_opts.context < -1)
> die(_("'%s' cannot be negative"), "--unified");
> - if (add_p_opt.interhunkcontext < -1)
> + if (interactive_opts.interhunkcontext < -1)
> die(_("'%s' cannot be negative"), "--inter-hunk-context");
>
> if (patch_interactive)
> @@ -412,11 +412,11 @@ int cmd_add(int argc,
> die(_("options '%s' and '%s' cannot be used together"), "--dry-run", "--interactive/--patch");
> if (pathspec_from_file)
> die(_("options '%s' and '%s' cannot be used together"), "--pathspec-from-file", "--interactive/--patch");
> - exit(interactive_add(repo, argv + 1, prefix, patch_interactive, &add_p_opt));
> + exit(interactive_add(repo, argv + 1, prefix, patch_interactive, &interactive_opts));
> } else {
> - if (add_p_opt.context != -1)
> + if (interactive_opts.context != -1)
> die(_("the option '%s' requires '%s'"), "--unified", "--interactive/--patch");
> - if (add_p_opt.interhunkcontext != -1)
> + if (interactive_opts.interhunkcontext != -1)
> die(_("the option '%s' requires '%s'"), "--inter-hunk-context", "--interactive/--patch");
> }
>
> diff --git a/builtin/checkout.c b/builtin/checkout.c
> index f9453473fe2..d230b1f8995 100644
> --- a/builtin/checkout.c
> +++ b/builtin/checkout.c
> @@ -546,7 +546,7 @@ static int checkout_paths(const struct checkout_opts *opts,
>
> if (opts->patch_mode) {
> enum add_p_mode patch_mode;
> - struct add_p_opt add_p_opt = {
> + struct interactive_options interactive_opts = {
> .context = opts->patch_context,
> .interhunkcontext = opts->patch_interhunk_context,
> };
> @@ -575,7 +575,7 @@ static int checkout_paths(const struct checkout_opts *opts,
> else
> BUG("either flag must have been set, worktree=%d, index=%d",
> opts->checkout_worktree, opts->checkout_index);
> - return !!run_add_p(the_repository, patch_mode, &add_p_opt,
> + return !!run_add_p(the_repository, patch_mode, &interactive_opts,
> rev, &opts->pathspec);
> }
>
> diff --git a/builtin/commit.c b/builtin/commit.c
> index 0243f17d53c..640495cc57e 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -123,7 +123,7 @@ static const char *edit_message, *use_message;
> static char *fixup_message, *fixup_commit, *squash_message;
> static const char *fixup_prefix;
> static int all, also, interactive, patch_interactive, only, amend, signoff;
> -static struct add_p_opt add_p_opt = ADD_P_OPT_INIT;
> +static struct interactive_options interactive_opts = INTERACTIVE_OPTIONS_INIT;
> static int edit_flag = -1; /* unspecified */
> static int quiet, verbose, no_verify, allow_empty, dry_run, renew_authorship;
> static int config_commit_verbose = -1; /* unspecified */
> @@ -356,9 +356,9 @@ static const char *prepare_index(const char **argv, const char *prefix,
> const char *ret;
> char *path = NULL;
>
> - if (add_p_opt.context < -1)
> + if (interactive_opts.context < -1)
> die(_("'%s' cannot be negative"), "--unified");
> - if (add_p_opt.interhunkcontext < -1)
> + if (interactive_opts.interhunkcontext < -1)
> die(_("'%s' cannot be negative"), "--inter-hunk-context");
>
> if (is_status)
> @@ -407,7 +407,7 @@ static const char *prepare_index(const char **argv, const char *prefix,
> old_index_env = xstrdup_or_null(getenv(INDEX_ENVIRONMENT));
> setenv(INDEX_ENVIRONMENT, the_repository->index_file, 1);
>
> - if (interactive_add(the_repository, argv, prefix, patch_interactive, &add_p_opt) != 0)
> + if (interactive_add(the_repository, argv, prefix, patch_interactive, &interactive_opts) != 0)
> die(_("interactive add failed"));
>
> the_repository->index_file = old_repo_index_file;
> @@ -432,9 +432,9 @@ static const char *prepare_index(const char **argv, const char *prefix,
> ret = get_lock_file_path(&index_lock);
> goto out;
> } else {
> - if (add_p_opt.context != -1)
> + if (interactive_opts.context != -1)
> die(_("the option '%s' requires '%s'"), "--unified", "--interactive/--patch");
> - if (add_p_opt.interhunkcontext != -1)
> + if (interactive_opts.interhunkcontext != -1)
> die(_("the option '%s' requires '%s'"), "--inter-hunk-context", "--interactive/--patch");
> }
>
> @@ -1742,8 +1742,8 @@ int cmd_commit(int argc,
> OPT_BOOL('i', "include", &also, N_("add specified files to index for commit")),
> OPT_BOOL(0, "interactive", &interactive, N_("interactively add files")),
> OPT_BOOL('p', "patch", &patch_interactive, N_("interactively add changes")),
> - OPT_DIFF_UNIFIED(&add_p_opt.context),
> - OPT_DIFF_INTERHUNK_CONTEXT(&add_p_opt.interhunkcontext),
> + OPT_DIFF_UNIFIED(&interactive_opts.context),
> + OPT_DIFF_INTERHUNK_CONTEXT(&interactive_opts.interhunkcontext),
> OPT_BOOL('o', "only", &only, N_("commit only specified files")),
> OPT_BOOL('n', "no-verify", &no_verify, N_("bypass pre-commit and commit-msg hooks")),
> OPT_BOOL(0, "dry-run", &dry_run, N_("show what would be committed")),
> diff --git a/builtin/reset.c b/builtin/reset.c
> index ed35802af15..088449e1209 100644
> --- a/builtin/reset.c
> +++ b/builtin/reset.c
> @@ -346,7 +346,7 @@ int cmd_reset(int argc,
> struct object_id oid;
> struct pathspec pathspec;
> int intent_to_add = 0;
> - struct add_p_opt add_p_opt = ADD_P_OPT_INIT;
> + struct interactive_options interactive_opts = INTERACTIVE_OPTIONS_INIT;
> const struct option options[] = {
> OPT__QUIET(&quiet, N_("be quiet, only report errors")),
> OPT_BOOL(0, "no-refresh", &no_refresh,
> @@ -371,8 +371,8 @@ int cmd_reset(int argc,
> PARSE_OPT_OPTARG,
> option_parse_recurse_submodules_worktree_updater),
> OPT_BOOL('p', "patch", &patch_mode, N_("select hunks interactively")),
> - OPT_DIFF_UNIFIED(&add_p_opt.context),
> - OPT_DIFF_INTERHUNK_CONTEXT(&add_p_opt.interhunkcontext),
> + OPT_DIFF_UNIFIED(&interactive_opts.context),
> + OPT_DIFF_INTERHUNK_CONTEXT(&interactive_opts.interhunkcontext),
> OPT_BOOL('N', "intent-to-add", &intent_to_add,
> N_("record only the fact that removed paths will be added later")),
> OPT_PATHSPEC_FROM_FILE(&pathspec_from_file),
> @@ -423,9 +423,9 @@ int cmd_reset(int argc,
> oidcpy(&oid, &tree->object.oid);
> }
>
> - if (add_p_opt.context < -1)
> + if (interactive_opts.context < -1)
> die(_("'%s' cannot be negative"), "--unified");
> - if (add_p_opt.interhunkcontext < -1)
> + if (interactive_opts.interhunkcontext < -1)
> die(_("'%s' cannot be negative"), "--inter-hunk-context");
>
> prepare_repo_settings(the_repository);
> @@ -436,12 +436,12 @@ int cmd_reset(int argc,
> die(_("options '%s' and '%s' cannot be used together"), "--patch", "--{hard,mixed,soft}");
> trace2_cmd_mode("patch-interactive");
> update_ref_status = !!run_add_p(the_repository, ADD_P_RESET,
> - &add_p_opt, rev, &pathspec);
> + &interactive_opts, rev, &pathspec);
> goto cleanup;
> } else {
> - if (add_p_opt.context != -1)
> + if (interactive_opts.context != -1)
> die(_("the option '%s' requires '%s'"), "--unified", "--patch");
> - if (add_p_opt.interhunkcontext != -1)
> + if (interactive_opts.interhunkcontext != -1)
> die(_("the option '%s' requires '%s'"), "--inter-hunk-context", "--patch");
> }
>
> diff --git a/builtin/stash.c b/builtin/stash.c
> index 948eba06fbc..3b509052338 100644
> --- a/builtin/stash.c
> +++ b/builtin/stash.c
> @@ -1306,7 +1306,7 @@ static int stash_staged(struct stash_info *info, struct strbuf *out_patch,
>
> static int stash_patch(struct stash_info *info, const struct pathspec *ps,
> struct strbuf *out_patch, int quiet,
> - struct add_p_opt *add_p_opt)
> + struct interactive_options *interactive_opts)
> {
> int ret = 0;
> struct child_process cp_read_tree = CHILD_PROCESS_INIT;
> @@ -1331,7 +1331,7 @@ static int stash_patch(struct stash_info *info, const struct pathspec *ps,
> old_index_env = xstrdup_or_null(getenv(INDEX_ENVIRONMENT));
> setenv(INDEX_ENVIRONMENT, the_repository->index_file, 1);
>
> - ret = !!run_add_p(the_repository, ADD_P_STASH, add_p_opt, NULL, ps);
> + ret = !!run_add_p(the_repository, ADD_P_STASH, interactive_opts, NULL, ps);
>
> the_repository->index_file = old_repo_index_file;
> if (old_index_env && *old_index_env)
> @@ -1427,7 +1427,8 @@ static int stash_working_tree(struct stash_info *info, const struct pathspec *ps
> }
>
> static int do_create_stash(const struct pathspec *ps, struct strbuf *stash_msg_buf,
> - int include_untracked, int patch_mode, struct add_p_opt *add_p_opt,
> + int include_untracked, int patch_mode,
> + struct interactive_options *interactive_opts,
> int only_staged, struct stash_info *info, struct strbuf *patch,
> int quiet)
> {
> @@ -1509,7 +1510,7 @@ static int do_create_stash(const struct pathspec *ps, struct strbuf *stash_msg_b
> untracked_commit_option = 1;
> }
> if (patch_mode) {
> - ret = stash_patch(info, ps, patch, quiet, add_p_opt);
> + ret = stash_patch(info, ps, patch, quiet, interactive_opts);
> if (ret < 0) {
> if (!quiet)
> fprintf_ln(stderr, _("Cannot save the current "
> @@ -1595,7 +1596,8 @@ static int create_stash(int argc, const char **argv, const char *prefix UNUSED,
> }
>
> static int do_push_stash(const struct pathspec *ps, const char *stash_msg, int quiet,
> - int keep_index, int patch_mode, struct add_p_opt *add_p_opt,
> + int keep_index, int patch_mode,
> + struct interactive_options *interactive_opts,
> int include_untracked, int only_staged)
> {
> int ret = 0;
> @@ -1667,7 +1669,7 @@ static int do_push_stash(const struct pathspec *ps, const char *stash_msg, int q
> if (stash_msg)
> strbuf_addstr(&stash_msg_buf, stash_msg);
> if (do_create_stash(ps, &stash_msg_buf, include_untracked, patch_mode,
> - add_p_opt, only_staged, &info, &patch, quiet)) {
> + interactive_opts, only_staged, &info, &patch, quiet)) {
> ret = -1;
> goto done;
> }
> @@ -1841,7 +1843,7 @@ static int push_stash(int argc, const char **argv, const char *prefix,
> const char *stash_msg = NULL;
> char *pathspec_from_file = NULL;
> struct pathspec ps;
> - struct add_p_opt add_p_opt = ADD_P_OPT_INIT;
> + struct interactive_options interactive_opts = INTERACTIVE_OPTIONS_INIT;
> struct option options[] = {
> OPT_BOOL('k', "keep-index", &keep_index,
> N_("keep index")),
> @@ -1849,8 +1851,8 @@ static int push_stash(int argc, const char **argv, const char *prefix,
> N_("stash staged changes only")),
> OPT_BOOL('p', "patch", &patch_mode,
> N_("stash in patch mode")),
> - OPT_DIFF_UNIFIED(&add_p_opt.context),
> - OPT_DIFF_INTERHUNK_CONTEXT(&add_p_opt.interhunkcontext),
> + OPT_DIFF_UNIFIED(&interactive_opts.context),
> + OPT_DIFF_INTERHUNK_CONTEXT(&interactive_opts.interhunkcontext),
> OPT__QUIET(&quiet, N_("quiet mode")),
> OPT_BOOL('u', "include-untracked", &include_untracked,
> N_("include untracked files in stash")),
> @@ -1907,19 +1909,19 @@ static int push_stash(int argc, const char **argv, const char *prefix,
> }
>
> if (!patch_mode) {
> - if (add_p_opt.context != -1)
> + if (interactive_opts.context != -1)
> die(_("the option '%s' requires '%s'"), "--unified", "--patch");
> - if (add_p_opt.interhunkcontext != -1)
> + if (interactive_opts.interhunkcontext != -1)
> die(_("the option '%s' requires '%s'"), "--inter-hunk-context", "--patch");
> }
>
> - if (add_p_opt.context < -1)
> + if (interactive_opts.context < -1)
> die(_("'%s' cannot be negative"), "--unified");
> - if (add_p_opt.interhunkcontext < -1)
> + if (interactive_opts.interhunkcontext < -1)
> die(_("'%s' cannot be negative"), "--inter-hunk-context");
>
> ret = do_push_stash(&ps, stash_msg, quiet, keep_index, patch_mode,
> - &add_p_opt, include_untracked, only_staged);
> + &interactive_opts, include_untracked, only_staged);
>
> clear_pathspec(&ps);
> free(pathspec_from_file);
> @@ -1944,7 +1946,7 @@ static int save_stash(int argc, const char **argv, const char *prefix,
> const char *stash_msg = NULL;
> struct pathspec ps;
> struct strbuf stash_msg_buf = STRBUF_INIT;
> - struct add_p_opt add_p_opt = ADD_P_OPT_INIT;
> + struct interactive_options interactive_opts = INTERACTIVE_OPTIONS_INIT;
> struct option options[] = {
> OPT_BOOL('k', "keep-index", &keep_index,
> N_("keep index")),
> @@ -1952,8 +1954,8 @@ static int save_stash(int argc, const char **argv, const char *prefix,
> N_("stash staged changes only")),
> OPT_BOOL('p', "patch", &patch_mode,
> N_("stash in patch mode")),
> - OPT_DIFF_UNIFIED(&add_p_opt.context),
> - OPT_DIFF_INTERHUNK_CONTEXT(&add_p_opt.interhunkcontext),
> + OPT_DIFF_UNIFIED(&interactive_opts.context),
> + OPT_DIFF_INTERHUNK_CONTEXT(&interactive_opts.interhunkcontext),
> OPT__QUIET(&quiet, N_("quiet mode")),
> OPT_BOOL('u', "include-untracked", &include_untracked,
> N_("include untracked files in stash")),
> @@ -1973,20 +1975,20 @@ static int save_stash(int argc, const char **argv, const char *prefix,
>
> memset(&ps, 0, sizeof(ps));
>
> - if (add_p_opt.context < -1)
> + if (interactive_opts.context < -1)
> die(_("'%s' cannot be negative"), "--unified");
> - if (add_p_opt.interhunkcontext < -1)
> + if (interactive_opts.interhunkcontext < -1)
> die(_("'%s' cannot be negative"), "--inter-hunk-context");
>
> if (!patch_mode) {
> - if (add_p_opt.context != -1)
> + if (interactive_opts.context != -1)
> die(_("the option '%s' requires '%s'"), "--unified", "--patch");
> - if (add_p_opt.interhunkcontext != -1)
> + if (interactive_opts.interhunkcontext != -1)
> die(_("the option '%s' requires '%s'"), "--inter-hunk-context", "--patch");
> }
>
> ret = do_push_stash(&ps, stash_msg, quiet, keep_index,
> - patch_mode, &add_p_opt, include_untracked,
> + patch_mode, &interactive_opts, include_untracked,
> only_staged);
>
> strbuf_release(&stash_msg_buf);
> diff --git a/commit.h b/commit.h
> index 1d6e0c7518b..7b6e59d6c19 100644
> --- a/commit.h
> +++ b/commit.h
> @@ -258,7 +258,7 @@ int for_each_commit_graft(each_commit_graft_fn, void *);
> int interactive_add(struct repository *repo,
> const char **argv,
> const char *prefix,
> - int patch, struct add_p_opt *add_p_opt);
> + int patch, struct interactive_options *opts);
>
> struct commit_extra_header {
> struct commit_extra_header *next;
>
next prev parent reply other threads:[~2025-11-20 15:05 UTC|newest]
Thread overview: 362+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-19 10:55 [PATCH RFC 00/11] Introduce git-history(1) command for easy history editing Patrick Steinhardt
2025-08-19 10:55 ` [PATCH RFC 01/11] sequencer: optionally skip printing commit summary Patrick Steinhardt
2025-08-19 10:55 ` [PATCH RFC 02/11] sequencer: add option to rewind HEAD after picking commits Patrick Steinhardt
2025-08-19 10:55 ` [PATCH RFC 03/11] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 04/11] builtin: add new "history" command Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 05/11] builtin/history: implement "drop" subcommand Patrick Steinhardt
2025-08-20 20:39 ` Ben Knoble
2025-08-22 12:21 ` Patrick Steinhardt
2025-08-23 16:15 ` Jean-Noël AVILA
2025-08-24 16:02 ` Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 06/11] builtin/history: implement "reorder" subcommand Patrick Steinhardt
2025-08-23 16:24 ` Jean-Noël AVILA
2025-08-24 17:25 ` Kristoffer Haugsbakk
2025-08-24 17:34 ` Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 07/11] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 08/11] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 09/11] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 10/11] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-08-20 21:15 ` D. Ben Knoble
2025-08-22 12:21 ` Patrick Steinhardt
2025-08-19 10:56 ` [PATCH RFC 11/11] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-08-20 21:27 ` D. Ben Knoble
2025-08-22 12:22 ` Patrick Steinhardt
2025-08-22 18:08 ` Junio C Hamano
2025-08-24 16:03 ` Patrick Steinhardt
2025-08-23 16:37 ` Jean-Noël AVILA
2025-08-24 16:02 ` Patrick Steinhardt
2025-08-19 21:28 ` [PATCH RFC 00/11] Introduce git-history(1) command for easy history editing D. Ben Knoble
2025-08-20 6:54 ` Patrick Steinhardt
2025-08-20 16:55 ` Ben Knoble
2025-08-20 17:36 ` Junio C Hamano
2025-08-20 17:49 ` Ben Knoble
2025-08-22 12:21 ` Patrick Steinhardt
2025-08-22 17:58 ` Junio C Hamano
2025-08-21 16:26 ` Sergey Organov
2025-08-21 17:21 ` Ben Knoble
2025-08-21 18:15 ` Sergey Organov
2025-08-24 1:25 ` Martin von Zweigbergk
2025-08-24 16:03 ` Patrick Steinhardt
2025-09-17 20:12 ` SZEDER Gábor
2025-12-03 18:18 ` Matthias Beyer
2025-12-10 9:58 ` Phillip Wood
2025-12-10 10:37 ` Matthias Beyer
2025-12-10 11:34 ` Phillip Wood
2025-12-10 14:18 ` Junio C Hamano
2025-12-19 12:22 ` Patrick Steinhardt
2025-12-19 13:58 ` SZEDER Gábor
2025-12-19 14:09 ` Patrick Steinhardt
2025-12-19 16:30 ` Elijah Newren
2025-12-20 16:51 ` Elijah Newren
2026-01-06 15:39 ` Patrick Steinhardt
2025-12-22 10:46 ` Phillip Wood
2026-01-06 15:40 ` Patrick Steinhardt
2025-12-22 13:47 ` D. Ben Knoble
2025-12-10 16:49 ` Martin von Zweigbergk
2025-12-10 18:27 ` Elijah Newren
2025-12-10 18:45 ` Martin von Zweigbergk
2025-12-10 19:55 ` Elijah Newren
2025-12-15 23:50 ` Kristoffer Haugsbakk
2025-08-24 17:31 ` Kristoffer Haugsbakk
2025-08-24 17:38 ` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 00/16] " Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 01/16] sequencer: optionally skip printing commit summary Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 02/16] sequencer: add option to rewind HEAD after picking commits Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 03/16] sequencer: introduce new history editing mode Patrick Steinhardt
2025-08-26 12:55 ` D. Ben Knoble
2025-09-03 12:19 ` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 04/16] sequencer: stop using `the_repository` in `sequencer_remove_state()` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 05/16] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-08-25 16:38 ` Junio C Hamano
2025-09-03 12:19 ` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 06/16] builtin: add new "history" command Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 07/16] builtin/history: introduce subcommands to manage interrupted rewrites Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 08/16] builtin/history: implement "drop" subcommand Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 09/16] builtin/history: implement "reorder" subcommand Patrick Steinhardt
2025-08-26 13:03 ` D. Ben Knoble
2025-09-03 12:19 ` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 10/16] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-08-25 16:41 ` Junio C Hamano
2025-08-24 17:42 ` [PATCH RFC v2 11/16] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 12/16] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-08-25 16:43 ` Junio C Hamano
2025-08-24 17:42 ` [PATCH RFC v2 13/16] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 14/16] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-08-24 17:42 ` [PATCH RFC v2 15/16] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-08-24 18:03 ` Kristoffer Haugsbakk
2025-09-03 12:20 ` Patrick Steinhardt
2025-08-26 13:14 ` D. Ben Knoble
2025-09-03 12:20 ` Patrick Steinhardt
2025-09-03 21:55 ` D. Ben Knoble
2025-09-04 12:57 ` Patrick Steinhardt
2025-09-12 18:26 ` D. Ben Knoble
2025-09-15 9:32 ` Patrick Steinhardt
2025-09-15 13:04 ` Ben Knoble
2025-08-24 17:42 ` [PATCH RFC v2 16/16] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-08-24 18:08 ` Kristoffer Haugsbakk
2025-09-03 12:20 ` Patrick Steinhardt
2025-09-03 23:39 ` [PATCH RFC v2 00/16] Introduce git-history(1) command for easy history editing D. Ben Knoble
2025-09-04 13:05 ` Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 00/18] " Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 01/18] sequencer: optionally skip printing commit summary Patrick Steinhardt
2025-09-10 14:01 ` Phillip Wood
2025-09-15 9:32 ` Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 02/18] sequencer: add option to rewind HEAD after picking commits Patrick Steinhardt
2025-09-10 14:04 ` Phillip Wood
2025-09-15 9:32 ` Patrick Steinhardt
2025-09-15 14:10 ` Phillip Wood
2025-09-04 14:27 ` [PATCH RFC v3 03/18] sequencer: introduce new history editing mode Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 04/18] sequencer: stop using `the_repository` in `sequencer_remove_state()` Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 05/18] sequencer: wire up "rewritten-hook" for REPLAY_HISTORY_EDIT Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 06/18] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 07/18] builtin: add new "history" command Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 08/18] builtin/history: introduce subcommands to manage interrupted rewrites Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 09/18] builtin/history: implement "drop" subcommand Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 10/18] builtin/history: implement "reorder" subcommand Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 11/18] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 12/18] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 13/18] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 14/18] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 15/18] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 16/18] sequencer: allow callers to provide mappings for the old commit Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 17/18] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-09-10 14:04 ` Phillip Wood
2025-09-15 9:32 ` Patrick Steinhardt
2025-09-04 14:27 ` [PATCH RFC v3 18/18] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-09-10 14:05 ` Phillip Wood
2025-09-15 9:32 ` Patrick Steinhardt
2025-09-15 14:10 ` Phillip Wood
2025-09-16 8:09 ` Patrick Steinhardt
2025-09-16 8:42 ` Phillip Wood
2025-09-05 10:29 ` [PATCH RFC v3 00/18] Introduce git-history(1) command for easy history editing Kristoffer Haugsbakk
2025-09-05 11:29 ` Patrick Steinhardt
2025-09-07 6:46 ` Elijah Newren
2025-09-10 14:05 ` Phillip Wood
2025-09-10 14:08 ` Phillip Wood
2025-09-15 9:33 ` Patrick Steinhardt
2025-09-16 11:23 ` Oswald Buddenhagen
2025-09-10 20:05 ` Junio C Hamano
2025-09-15 9:32 ` Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 00/12] " Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 01/12] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-10-14 8:49 ` Karthik Nayak
2025-10-21 11:43 ` Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 02/12] replay: extract logic to pick commits Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 03/12] replay: stop using `the_repository` Patrick Steinhardt
2025-10-14 8:53 ` Karthik Nayak
2025-10-01 15:57 ` [PATCH v4 04/12] replay: parse commits before dereferencing them Patrick Steinhardt
2025-10-14 8:57 ` Karthik Nayak
2025-10-01 15:57 ` [PATCH v4 05/12] builtin: add new "history" command Patrick Steinhardt
2025-10-02 9:26 ` Kristoffer Haugsbakk
2025-10-14 9:07 ` Karthik Nayak
2025-10-21 11:43 ` Patrick Steinhardt
2025-10-22 3:32 ` Junio C Hamano
2025-10-22 12:12 ` Karthik Nayak
2025-10-01 15:57 ` [PATCH v4 06/12] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-10-14 11:04 ` Karthik Nayak
2025-10-21 11:43 ` Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 07/12] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 08/12] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-10-02 9:25 ` Kristoffer Haugsbakk
2025-10-14 12:35 ` Karthik Nayak
2025-10-21 11:44 ` Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 09/12] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 10/12] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-10-02 9:28 ` Kristoffer Haugsbakk
2025-10-02 10:24 ` Patrick Steinhardt
2025-10-14 13:08 ` Karthik Nayak
2025-10-01 15:57 ` [PATCH v4 11/12] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-10-01 15:57 ` [PATCH v4 12/12] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-10-14 13:38 ` Karthik Nayak
2025-10-21 11:44 ` Patrick Steinhardt
2025-10-21 21:19 ` D. Ben Knoble
2025-10-27 9:58 ` Patrick Steinhardt
2025-10-14 13:41 ` [PATCH v4 00/12] Introduce git-history(1) command for easy history editing Karthik Nayak
2025-10-14 16:47 ` Junio C Hamano
2025-10-21 14:15 ` [PATCH v5 " Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 01/12] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-10-21 20:38 ` Junio C Hamano
2025-10-21 14:15 ` [PATCH v5 02/12] replay: extract logic to pick commits Patrick Steinhardt
2025-10-21 20:41 ` Junio C Hamano
2025-10-21 14:15 ` [PATCH v5 03/12] replay: stop using `the_repository` Patrick Steinhardt
2025-10-21 20:48 ` Junio C Hamano
2025-10-21 20:52 ` Junio C Hamano
2025-10-21 14:15 ` [PATCH v5 04/12] replay: parse commits before dereferencing them Patrick Steinhardt
2025-10-21 20:57 ` Junio C Hamano
2025-10-27 9:57 ` Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 05/12] builtin: add new "history" command Patrick Steinhardt
2025-10-21 21:15 ` Junio C Hamano
2025-10-27 9:57 ` Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 06/12] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-10-21 21:34 ` Junio C Hamano
2025-10-21 21:43 ` D. Ben Knoble
2025-10-27 9:58 ` Patrick Steinhardt
2025-10-27 9:58 ` Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 07/12] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 08/12] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 09/12] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 10/12] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-10-21 14:15 ` [PATCH v5 11/12] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-10-21 14:16 ` [PATCH v5 12/12] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-10-21 18:53 ` [PATCH v5 00/12] Introduce git-history(1) command for easy history editing Junio C Hamano
2025-10-27 11:33 ` [PATCH v6 00/11] " Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 01/11] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 02/11] replay: extract logic to pick commits Patrick Steinhardt
2025-11-17 16:27 ` Phillip Wood
2025-11-20 7:01 ` Elijah Newren
2025-10-27 11:33 ` [PATCH v6 03/11] replay: stop using `the_repository` Patrick Steinhardt
2025-11-20 7:01 ` Elijah Newren
2025-12-02 18:47 ` Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 04/11] builtin: add new "history" command Patrick Steinhardt
2025-11-17 16:28 ` Phillip Wood
2025-12-02 18:48 ` Patrick Steinhardt
2025-11-20 7:02 ` Elijah Newren
2025-12-02 18:48 ` Patrick Steinhardt
2025-12-02 22:44 ` D. Ben Knoble
2025-12-03 10:48 ` Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 05/11] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-11-17 16:27 ` Phillip Wood
2025-12-02 18:50 ` Patrick Steinhardt
2025-12-10 9:52 ` Phillip Wood
2025-11-20 7:03 ` Elijah Newren
2025-12-02 18:50 ` Patrick Steinhardt
2025-11-25 8:31 ` SZEDER Gábor
2025-12-02 18:50 ` Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 06/11] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-11-20 7:03 ` Elijah Newren
2025-10-27 11:33 ` [PATCH v6 07/11] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-11-20 7:03 ` Elijah Newren
2025-11-20 15:05 ` Phillip Wood [this message]
2025-12-02 18:48 ` Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 08/11] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-11-20 7:03 ` Elijah Newren
2025-11-20 15:05 ` Phillip Wood
2025-10-27 11:33 ` [PATCH v6 09/11] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-11-20 7:04 ` Elijah Newren
2025-11-20 15:05 ` Phillip Wood
2025-12-02 18:49 ` Patrick Steinhardt
2025-10-27 11:33 ` [PATCH v6 10/11] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-11-20 7:04 ` Elijah Newren
2025-10-27 11:33 ` [PATCH v6 11/11] builtin/history: implement "split" subcommand Patrick Steinhardt
2025-11-20 7:05 ` Elijah Newren
2025-12-02 18:49 ` Patrick Steinhardt
2025-11-21 14:31 ` Phillip Wood
2025-12-02 18:51 ` Patrick Steinhardt
2025-12-10 9:51 ` Phillip Wood
2025-12-19 13:00 ` Patrick Steinhardt
2025-11-12 19:13 ` [PATCH v6 00/11] Introduce git-history(1) command for easy history editing Sergey Organov
2025-11-20 7:07 ` Elijah Newren
2025-11-20 20:28 ` Junio C Hamano
2025-11-20 20:40 ` Elijah Newren
2025-11-20 20:49 ` Junio C Hamano
2025-11-20 22:02 ` Elijah Newren
2025-11-21 14:31 ` Phillip Wood
2025-11-21 16:01 ` Junio C Hamano
2025-11-23 2:54 ` Elijah Newren
2025-12-02 18:49 ` Patrick Steinhardt
2025-12-05 8:49 ` Elijah Newren
2025-12-09 7:53 ` Patrick Steinhardt
2025-12-09 17:43 ` Martin von Zweigbergk
2025-12-10 11:32 ` Phillip Wood
2025-12-10 6:55 ` Elijah Newren
2025-12-09 18:29 ` Kristoffer Haugsbakk
2025-12-12 22:00 ` Working on top of mega merges D. Ben Knoble
2025-11-23 2:30 ` [PATCH v6 00/11] Introduce git-history(1) command for easy history editing Elijah Newren
2025-11-24 16:31 ` Phillip Wood
2025-11-25 3:39 ` Elijah Newren
2025-12-03 10:48 ` [PATCH v7 00/12] " Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 01/12] wt-status: provide function to expose status for trees Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 02/12] replay: extract logic to pick commits Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 03/12] replay: stop using `the_repository` Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 04/12] builtin: add new "history" command Patrick Steinhardt
2025-12-22 17:11 ` Kristoffer Haugsbakk
2026-01-06 15:41 ` Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 05/12] builtin/history: implement "reword" subcommand Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 06/12] add-patch: split out header from "add-interactive.h" Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 07/12] add-patch: split out `struct interactive_options` Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 08/12] add-patch: remove dependency on "add-interactive" subsystem Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 09/12] add-patch: add support for in-memory index patching Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 10/12] add-patch: allow disabling editing of hunks Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 11/12] cache-tree: allow writing in-memory index as tree Patrick Steinhardt
2025-12-03 10:48 ` [PATCH v7 12/12] builtin/history: implement "split" subcommand Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 0/7] Introduce git-history(1) command for easy history editing Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 1/7] builtin/replay: extract core logic to replay revisions Patrick Steinhardt
2026-01-07 17:53 ` D. Ben Knoble
2026-01-09 7:37 ` Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 2/7] builtin/replay: move core logic into "libgit.a" Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 3/7] replay: small set of cleanups Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 4/7] replay: yield the object ID of the final rewritten commit Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 5/7] wt-status: provide function to expose status for trees Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 6/7] builtin: add new "history" command Patrick Steinhardt
2026-01-07 10:10 ` [PATCH v8 7/7] builtin/history: implement "reword" subcommand Patrick Steinhardt
2026-01-07 18:01 ` D. Ben Knoble
2026-01-09 7:37 ` Patrick Steinhardt
2026-01-09 23:24 ` D. Ben Knoble
2026-01-10 1:20 ` Elijah Newren
2026-01-12 13:03 ` Patrick Steinhardt
2026-01-07 17:39 ` [PATCH v8 0/7] Introduce git-history(1) command for easy history editing D. Ben Knoble
2026-01-09 7:34 ` Patrick Steinhardt
2026-01-09 23:27 ` D. Ben Knoble
2026-01-09 8:35 ` [PATCH v9 " Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 1/7] builtin/replay: extract core logic to replay revisions Patrick Steinhardt
2026-01-10 1:14 ` Elijah Newren
2026-01-12 13:02 ` Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 2/7] builtin/replay: move core logic into "libgit.a" Patrick Steinhardt
2026-01-10 1:16 ` Elijah Newren
2026-01-12 13:02 ` Patrick Steinhardt
2026-01-13 6:00 ` Elijah Newren
2026-01-13 7:31 ` Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 3/7] replay: small set of cleanups Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 4/7] replay: yield the object ID of the final rewritten commit Patrick Steinhardt
2026-01-10 1:17 ` Elijah Newren
2026-01-12 13:03 ` Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 5/7] wt-status: provide function to expose status for trees Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 6/7] builtin: add new "history" command Patrick Steinhardt
2026-01-10 1:17 ` Elijah Newren
2026-01-12 13:02 ` Patrick Steinhardt
2026-01-09 8:35 ` [PATCH v9 7/7] builtin/history: implement "reword" subcommand Patrick Steinhardt
2026-01-10 1:26 ` [PATCH v9 0/7] Introduce git-history(1) command for easy history editing Elijah Newren
2026-01-10 17:14 ` SZEDER Gábor
2026-01-12 13:03 ` Patrick Steinhardt
2026-01-11 5:58 ` [PATCH 0/2] Some Elijah Newren
2026-01-11 5:58 ` [PATCH v9 8/7] SQUASH ME: Fixups Elijah Newren
2026-01-11 5:58 ` [PATCH v9 9/7] history: fix detached HEAD handling Elijah Newren
2026-01-11 6:15 ` [PATCH 0/2] Some Elijah Newren
2026-01-12 13:02 ` Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 0/8] Introduce git-history(1) command for easy history editing Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 1/8] builtin/replay: extract core logic to replay revisions Patrick Steinhardt
2026-01-12 15:08 ` Junio C Hamano
2026-01-12 15:37 ` Patrick Steinhardt
2026-01-13 6:00 ` Elijah Newren
2026-01-13 7:31 ` Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 2/8] builtin/replay: move core logic into "libgit.a" Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 3/8] replay: small set of cleanups Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 4/8] replay: support empty commit ranges Patrick Steinhardt
2026-01-13 6:00 ` Elijah Newren
2026-01-13 7:30 ` Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 5/8] replay: support updating detached HEAD Patrick Steinhardt
2026-01-13 6:00 ` Elijah Newren
2026-01-13 7:30 ` Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 6/8] wt-status: provide function to expose status for trees Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 7/8] builtin: add new "history" command Patrick Steinhardt
2026-01-12 14:15 ` [PATCH v10 8/8] builtin/history: implement "reword" subcommand Patrick Steinhardt
2026-01-13 6:01 ` Elijah Newren
2026-01-13 6:01 ` [PATCH v10 0/8] Introduce git-history(1) command for easy history editing Elijah Newren
2026-01-13 9:54 ` [PATCH v11 " Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 1/8] builtin/replay: extract core logic to replay revisions Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 2/8] builtin/replay: move core logic into "libgit.a" Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 3/8] replay: small set of cleanups Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 4/8] replay: support empty commit ranges Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 5/8] replay: support updating detached HEAD Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 6/8] wt-status: provide function to expose status for trees Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 7/8] builtin: add new "history" command Patrick Steinhardt
2026-01-13 9:54 ` [PATCH v11 8/8] builtin/history: implement "reword" subcommand Patrick Steinhardt
2026-01-16 16:28 ` SZEDER Gábor
2026-01-17 22:56 ` Elijah Newren
2026-01-18 17:50 ` Junio C Hamano
2026-02-03 0:01 ` Junio C Hamano
2026-02-05 8:19 ` Patrick Steinhardt
2026-01-16 6:43 ` [PATCH v11 0/8] Introduce git-history(1) command for easy history editing Elijah Newren
2026-01-16 15:21 ` Junio C Hamano
2026-01-16 16:32 ` SZEDER Gábor
2026-01-21 19:02 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3872545c-6738-4852-ab2c-8c08525ff795@gmail.com \
--to=phillip.wood123@gmail$(echo .)com \
--cc=ben.knoble@gmail$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=jn.avila@free$(echo .)fr \
--cc=karthik.188@gmail$(echo .)com \
--cc=kristofferhaugsbakk@fastmail$(echo .)com \
--cc=martinvonz@gmail$(echo .)com \
--cc=newren@gmail$(echo .)com \
--cc=phillip.wood@dunelm$(echo .)org.uk \
--cc=ps@pks$(echo .)im \
--cc=sorganov@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox