Re: [PATCH] git-send-email: default value for "From:" field.
- From
- Quy Tonthat <qtonthat@gmail.com>
- Date
- Jan 19, 2007, 01:14 UTC
- Message-ID
- <45B01B82.9050707@gmail.com>
- In-Reply-To
- <45AFDAEA.607@freescale.com>
Timur Tabi wrote:
Show 25 quoted lines
> Quy Tonthat wrote:
>> If user hits enter at the prompt for
>> "Who should the emails appear to be from?",
>> the value for "From:" field was emptied instead of GIT_COMMITER_IDENT.
>>
>> Signed-off-by: Quy Tonthat <qtonthat@gmail.com>
>> ---
>> It seems the original code assumes readline to accept
>> an extra argument for default value. I don't remember I ever encountered
>> that feature from readline. Is there anything like that out there ?
>
> I was thinking the same thing. Almost every call to readline() in
> git-send-email is like that, so could your patch below could be expanded
> to include the other instances?
>
> $_ = $term->readline("Who should the emails be sent to? ",
> "");
>
> $_ = $term->readline("What subject should the emails start with? ",
> $initial_subject);
>
> $_= $term->readline("Message-ID to be used as In-Reply-To for
> the first email? ",
> $initial_reply_to);
> It is correct to expand the fix to other instances, but that won't practically change anything at run time (All those "defaults" are empty when passed to readline). I don't normally change somebody else's codes just for the sake of correctness or styles. Quy