Re: Awkward and confusing question in git --send-email

From

Junio C Hamano <gitster@pobox.com>

Date

Apr 24, 2016, 19:31 UTC

Message-ID

<xmqqlh42n6qn.fsf@gitster.mtv.corp.google.com>

In-Reply-To

<20160423182235.GA20615@jocasta.intra>

John Darrington <john@darrington.wattle.id.au> writes:

> Git --send-email poses the question:
>
>  "Who should the emails be sent to (if any)?"
>
> Apart from sounding clumsy (not to mention incorrect grammar), it took me a while to work out what this meant.
>
> I suggest this be changed to :
>
> "To whom should the email be sent (if anyone) ?"

Sounds good.

-- >8 --
Subject: send-email: fix grammo in the prompt that asks e-mail recipients

Reported-by: John Darrington <john@darrington.wattle.id.au>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 git-send-email.perl | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/git-send-email.perl b/git-send-email.perl
index c45b22a..1ef94f6 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -827,9 +827,10 @@ sub file_declares_8bit_cte {
 # But it's a no-op to run sanitize_address on an already sanitized address.
 $sender = sanitize_address($sender);
 
+my $to_whom = "To whom should the emails be sent (if anyone)?";
 my $prompting = 0;
 if (!@initial_to && !defined $to_cmd) {
-	my $to = ask("Who should the emails be sent to (if any)? ",
+	my $to = ask("$to_whom ",
 		     default => "",
 		     valid_re => qr/\@.*\./, confirm_only => 1);
 	push @initial_to, parse_address_line($to) if defined $to; # sanitized/validated later
@@ -924,7 +925,7 @@ sub validate_address {
 			cleanup_compose_files();
 			exit(0);
 		}
-		$address = ask("Who should the email be sent to (if any)? ",
+		$address = ask("$to_whom ",
 			default => "",
 			valid_re => qr/\@.*\./, confirm_only => 1);
 	}