Mark prompt message assembled in place for translation, unfolding each use case for each entry in the %patch_modes hash table. Previously, this script relied on whether $patch_mode was set to run the command patch_update_cmd() or show status and loop the main loop. Now, it uses $cmd to indicate we must run patch_update_cmd() and $patch_mode is used to tell which flavor of the %patch_modes are we on. This is introduced in order to be able to mark and unfold the message prompt knowing in which context we are. The tracking of context was done previously by point %patch_mode_flavour hash table to the correct entry of %patch_modes, focusing only on value of %patch_modes. Now, we are also interested in the key ('staged', 'stash', 'checkout_head', ...). Signed-off-by: Vasco Almeida <vascomalmeida@xxxxxxx> --- git-add--interactive.perl | 54 ++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 46 insertions(+), 8 deletions(-) diff --git a/git-add--interactive.perl b/git-add--interactive.perl index 6bbde2d..c8d5093 100755 --- a/git-add--interactive.perl +++ b/git-add--interactive.perl @@ -93,6 +93,7 @@ sub colored { } # command line options +my $cmd; my $patch_mode; my $patch_mode_revision; @@ -173,7 +174,8 @@ my %patch_modes = ( }, ); -my %patch_mode_flavour = %{$patch_modes{stage}}; +$patch_mode = 'stage'; +my %patch_mode_flavour = %{$patch_modes{$patch_mode}}; sub run_cmd_pipe { if ($^O eq 'MSWin32') { @@ -1311,6 +1313,44 @@ sub display_hunks { return $i; } +my %patch_update_prompt_modes = ( + stage => { + mode => __("Stage mode change [y,n,q,a,d,/%s,?]? "), + deletion => __("Stage deletion [y,n,q,a,d,/%s,?]? "), + hunk => __("Stage this hunk [y,n,q,a,d,/%s,?]? "), + }, + stash => { + mode => __("Stash mode change [y,n,q,a,d,/%s,?]? "), + deletion => __("Stash deletion [y,n,q,a,d,/%s,?]? "), + hunk => __("Stash this hunk [y,n,q,a,d,/%s,?]? "), + }, + reset_head => { + mode => __("Unstage mode change [y,n,q,a,d,/%s,?]? "), + deletion => __("Unstage deletion [y,n,q,a,d,/%s,?]? "), + hunk => __("Unstage this hunk [y,n,q,a,d,/%s,?]? "), + }, + reset_nothead => { + mode => __("Apply mode change to index [y,n,q,a,d,/%s,?]? "), + deletion => __("Apply deletion to index [y,n,q,a,d,/%s,?]? "), + hunk => __("Apply this hunk to index [y,n,q,a,d,/%s,?]? "), + }, + checkout_index => { + mode => __("Discard mode change from worktree [y,n,q,a,d,/%s,?]? "), + deletion => __("Discard deletion from worktree [y,n,q,a,d,/%s,?]? "), + hunk => __("Discard this hunk from worktree [y,n,q,a,d,/%s,?]? "), + }, + checkout_head => { + mode => __("Discard mode change from index and worktree [y,n,q,a,d,/%s,?]? "), + deletion => __("Discard deletion from index and worktree [y,n,q,a,d,/%s,?]? "), + hunk => __("Discard this hunk from index and worktree [y,n,q,a,d,/%s,?]? "), + }, + checkout_nothead => { + mode => __("Apply mode change to index and worktree [y,n,q,a,d,/%s,?]? "), + deletion => __("Apply deletion to index and worktree [y,n,q,a,d,/%s,?]? "), + hunk => __("Apply this hunk to index and worktree [y,n,q,a,d,/%s,?]? "), + }, +); + sub patch_update_file { my $quit = 0; my ($ix, $num); @@ -1383,12 +1423,9 @@ sub patch_update_file { for (@{$hunk[$ix]{DISPLAY}}) { print; } - print colored $prompt_color, $patch_mode_flavour{VERB}, - ($hunk[$ix]{TYPE} eq 'mode' ? ' mode change' : - $hunk[$ix]{TYPE} eq 'deletion' ? ' deletion' : - ' this hunk'), - $patch_mode_flavour{TARGET}, - " [y,n,q,a,d,/$other,?]? "; + print colored $prompt_color, + sprintf($patch_update_prompt_modes{$patch_mode}{$hunk[$ix]{TYPE}}, $other); + my $line = prompt_single_character; last unless defined $line; if ($line) { @@ -1644,6 +1681,7 @@ sub process_args { die sprintf(__("invalid argument %s, expecting --"), $arg) unless $arg eq "--"; %patch_mode_flavour = %{$patch_modes{$patch_mode}}; + $cmd = 1; } elsif ($arg ne "--") { die sprintf(__("invalid argument %s, expecting --"), $arg); @@ -1680,7 +1718,7 @@ sub main_loop { process_args(); refresh(); -if ($patch_mode) { +if ($cmd) { patch_update_cmd(); } else { -- 2.7.4