Vasco Almeida <vascomalmeida@xxxxxxx> writes:
> sub help_patch_cmd {
> - my $verb = lc $patch_mode_flavour{VERB};
> - my $target = $patch_mode_flavour{TARGET};
> - print colored $help_color, <<EOF ;
> -y - $verb this hunk$target
> -n - do not $verb this hunk$target
> -q - quit; do not $verb this hunk or any of the remaining ones
> -a - $verb this hunk and all later hunks in the file
> -d - do not $verb this hunk or any of the later hunks in the file
> -g - select a hunk to go to
> + if ($patch_mode eq 'stage') {
> + print colored $help_color, __(
> +"y - stage this hunk
> +n - do not stage this hunk
> +q - quit; do not stage this hunk or any of the remaining ones
> +a - stage this hunk and all later hunks in the file
> +d - do not stage this hunk or any of the later hunks in the file");
> + } elsif ($patch_mode eq 'stash') {
> + ...
> +"\n";
> }
The same "shouldn't this be done with a table not code?" comment
applies to this as well, though this is not as bad as the previous
one.