summaryrefslogtreecommitdiff
path: root/t/t4034/ada/pre
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2017-03-02 04:48:22 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2017-03-02 10:10:38 -0800
commitc852bd54bd87fdcdc825f5d45c26aa745be13ba6 (patch)
tree13ddb37b3d06c1d5da9263847180df9d2a0d477e /t/t4034/ada/pre
parentGit 2.12 (diff)
downloadtgif-c852bd54bd87fdcdc825f5d45c26aa745be13ba6.tar.xz
add--interactive: fix missing file prompt for patch mode with "-i"
When invoked as "git add -i", each menu interactive menu option prompts the user to select a list of files. This includes the "patch" option, which gets the list before starting the hunk-selection loop. As "git add -p", it behaves differently, and jumps straight to the hunk selection loop. Since 0539d5e6d (i18n: add--interactive: mark patch prompt for translation, 2016-12-14), the "add -i" case mistakenly jumps to straight to the hunk-selection loop. Prior to that commit the distinction between the two cases was managed by the $patch_mode variable. That commit used $patch_mode for something else, and moved the old meaning to the "$cmd" variable. But it forgot to update the $patch_mode check inside patch_update_cmd() which controls the file-list behavior. The simplest fix would be to change that line to check $cmd. But while we're here, let's use a less obscure name for this flag: $patch_mode_only, a boolean which tells whether we are in full-interactive mode or only in patch-mode. Reported-by: Henrik Grubbström <grubba@grubba.org> Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4034/ada/pre')
0 files changed, 0 insertions, 0 deletions