add -p: prompt for single characters

Use Term::ReadKey, if available and enabled with interactive.singlekey,
to let the user answer add -p's prompts by pressing a single key.  We're
not doing the same in the main 'add -i' interface because file selection
etc. may expect several characters.

Two commands take an argument: 'g' can easily cope since it'll just
offer a choice of chunks.  '/' now (unconditionally, even without
readkey) offers a chance to enter a regex if none was given.

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Thomas Rast 2009-02-05 09:28:26 +01:00 committed by Junio C Hamano
parent 68c02d7c46
commit ca6ac7f135
2 changed files with 48 additions and 4 deletions

View file

@ -1000,6 +1000,13 @@ instaweb.port::
The port number to bind the gitweb httpd to. See The port number to bind the gitweb httpd to. See
linkgit:git-instaweb[1]. linkgit:git-instaweb[1].
interactive.singlekey::
In interactive programs, allow the user to provide one-letter
input with a single key (i.e., without hitting enter).
Currently this is used only by the `\--patch` mode of
linkgit:git-add[1]. Note that this setting is silently
ignored if portable keystroke input is not available.
log.date:: log.date::
Set default date-time mode for the log command. Setting log.date Set default date-time mode for the log command. Setting log.date
value is similar to using 'git-log'\'s --date option. The value is one of the value is similar to using 'git-log'\'s --date option. The value is one of the

View file

@ -33,6 +33,14 @@
my $normal_color = $repo->get_color("", "reset"); my $normal_color = $repo->get_color("", "reset");
my $use_readkey = 0;
if ($repo->config_bool("interactive.singlekey")) {
eval {
use Term::ReadKey;
$use_readkey = 1;
};
}
sub colored { sub colored {
my $color = shift; my $color = shift;
my $string = join("", @_); my $string = join("", @_);
@ -758,11 +766,32 @@ sub diff_applies {
return close $fh; return close $fh;
} }
sub _restore_terminal_and_die {
ReadMode 'restore';
print "\n";
exit 1;
}
sub prompt_single_character {
if ($use_readkey) {
local $SIG{TERM} = \&_restore_terminal_and_die;
local $SIG{INT} = \&_restore_terminal_and_die;
ReadMode 'cbreak';
my $key = ReadKey 0;
ReadMode 'restore';
print "$key" if defined $key;
print "\n";
return $key;
} else {
return <STDIN>;
}
}
sub prompt_yesno { sub prompt_yesno {
my ($prompt) = @_; my ($prompt) = @_;
while (1) { while (1) {
print colored $prompt_color, $prompt; print colored $prompt_color, $prompt;
my $line = <STDIN>; my $line = prompt_single_character;
return 0 if $line =~ /^n/i; return 0 if $line =~ /^n/i;
return 1 if $line =~ /^y/i; return 1 if $line =~ /^y/i;
} }
@ -893,7 +922,7 @@ sub patch_update_file {
print @{$mode->{DISPLAY}}; print @{$mode->{DISPLAY}};
print colored $prompt_color, print colored $prompt_color,
"Stage mode change [y/n/a/d/?]? "; "Stage mode change [y/n/a/d/?]? ";
my $line = <STDIN>; my $line = prompt_single_character;
if ($line =~ /^y/i) { if ($line =~ /^y/i) {
$mode->{USE} = 1; $mode->{USE} = 1;
last; last;
@ -966,7 +995,7 @@ sub patch_update_file {
print; print;
} }
print colored $prompt_color, "Stage this hunk [y,n,a,d,/$other,?]? "; print colored $prompt_color, "Stage this hunk [y,n,a,d,/$other,?]? ";
my $line = <STDIN>; my $line = prompt_single_character;
if ($line) { if ($line) {
if ($line =~ /^y/i) { if ($line =~ /^y/i) {
$hunk[$ix]{USE} = 1; $hunk[$ix]{USE} = 1;
@ -1018,9 +1047,17 @@ sub patch_update_file {
next; next;
} }
elsif ($line =~ m|^/(.*)|) { elsif ($line =~ m|^/(.*)|) {
my $regex = $1;
if ($1 eq "") {
print colored $prompt_color, "search for regex? ";
$regex = <STDIN>;
if (defined $regex) {
chomp $regex;
}
}
my $search_string; my $search_string;
eval { eval {
$search_string = qr{$1}m; $search_string = qr{$regex}m;
}; };
if ($@) { if ($@) {
my ($err,$exp) = ($@, $1); my ($err,$exp) = ($@, $1);