[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: option abbreviation exceptions
From: |
Jim Meyering |
Subject: |
Re: option abbreviation exceptions |
Date: |
Wed, 14 Jan 2009 17:42:43 +0100 |
Eric Blake <address@hidden> wrote:
> According to Jim Meyering on 1/10/2009 2:32 PM:
>>>> "[" feels like it should be the exception.
>>>> I see treating --v like --version as a feature. a typing saver.
>>>> But with "[", it's good to minimize the number of strings that
>>>> make it act differently.
>>>>
>>>
>>> Or, for echo and [, we could skip parse_long_options altogether, and
>>> hardcode this instead:
>>>
>>> argc == 2 && (STREQ (argv[1], "--help") || STREQ (argv[1], "--version"))
>>
>> One of those approaches would be good.
>> Whichever you prefer.
>
> How about this patch for test/[, echo, and printf? Note that true (and
> false) already did this exact approach. And for all other applications,
> it makes sense to use parse_long_options (and allow abbreviations).
>
> I went ahead and pushed this to the next branch, to make it easier to
> test. Since that branch is subject to rebasing, we can remove (or rework)
> this patch as needed before making it official.
Thanks! That change looks fine.
I thought about NEWS, but it's not warranted,
since these are all shell built-in commands,
and I suspect that only command-line users would
ever use abbreviated --version and --help options with them.