Skip to content

Commit 086dee6

Browse files
committed
built-in add -p: implement hunk editing
Just like `git add --edit` allows the user to edit the diff before it is being applied to the index, this feature allows the user to edit the diff *hunk*. Naturally, it gets a bit more complicated here because the result has to play well with the remaining hunks of the overall diff. Therefore, we have to do a loop in which we let the user edit the hunk, then test whether the result would work, and if not, drop the edits and let the user decide whether to try editing the hunk again. Note: in contrast to the Perl version, we use the same diff "coalescing" (i.e. merging overlapping hunks into a single one) also for the check after editing, and we introduce a new flag for that purpose that asks the `reassemble_patch()` function to pretend that all hunks were selected for use. This allows us to continue to run `git apply` *without* the `--allow-overlap` option (unlike the Perl version), and it also fixes two known breakages in `t3701-add-interactive.sh` (which we cannot mark as resolved so far because the Perl script version is still the default and continues to have those breakages). Signed-off-by: Johannes Schindelin <[email protected]>
1 parent a10a6fe commit 086dee6

File tree

3 files changed

+344
-17
lines changed

3 files changed

+344
-17
lines changed

add-interactive.c

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,13 @@ int init_add_i_state(struct repository *r, struct add_i_state *s)
4949

5050
strlcpy(s->fraginfo_color,
5151
diff_get_color(s->use_color, DIFF_FRAGINFO), COLOR_MAXLEN);
52+
strlcpy(s->context_color,
53+
diff_get_color(s->use_color, DIFF_CONTEXT), COLOR_MAXLEN);
54+
strlcpy(s->file_old_color,
55+
diff_get_color(s->use_color, DIFF_FILE_OLD), COLOR_MAXLEN);
56+
strlcpy(s->file_new_color,
57+
diff_get_color(s->use_color, DIFF_FILE_NEW), COLOR_MAXLEN);
58+
5259

5360
return 0;
5461
}

add-interactive.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ struct add_i_state {
1212
char error_color[COLOR_MAXLEN];
1313
char reset_color[COLOR_MAXLEN];
1414
char fraginfo_color[COLOR_MAXLEN];
15+
char context_color[COLOR_MAXLEN];
16+
char file_old_color[COLOR_MAXLEN];
17+
char file_new_color[COLOR_MAXLEN];
1518
};
1619

1720
int init_add_i_state(struct repository *r, struct add_i_state *s);

0 commit comments

Comments
 (0)