[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] rm: Add number of arguments to interactive prompt

From: Pádraig Brady
Subject: Re: [PATCH] rm: Add number of arguments to interactive prompt
Date: Mon, 22 Jul 2013 03:02:35 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 07/19/2013 07:12 PM, Rasmus Villemoes wrote:
> Pádraig Brady <address@hidden> writes:
>> I've adjusted to correctly pluralize and remove the redudant "all".
>> thanks,
> Not to start painting a bikeshed,

No worries, we're good at that here :)

> but: I'm not so sure "all" is
> redundant. If prompted with "Remove 50 arguments?", I'd like to choose
> the option "yes, and if I provided more than 50 arguments (how would I
> know, the shell did most of the work), please also remove those"; the
> yesno() library function does not provide that option :-)

I see your point.
However it's unlikely that someone would be confused that
rm was removing only some of the supplied arguments.
Also it mentions "arguments" rather than files,
also suggesting this is a sanity check referring to all arguments.

> If "all" must go, "the" could take its place. With the proper
> pluralization, that would probably also make a nicer prompt in the
> n_files==1 case.

So it's not worth another commit to change further.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]