bug-bash
[Top][All Lists]
Advanced

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

Re: [PATCH] Add nofirstword completion option


From: Chet Ramey
Subject: Re: [PATCH] Add nofirstword completion option
Date: Tue, 12 Jun 2018 14:54:55 -0400
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.8.0

On 6/12/18 1:28 PM, Luca Boccassi wrote:
> On Fri, 2018-06-08 at 11:03 -0400, Chet Ramey wrote:
>> On 6/8/18 10:40 AM, Luca Boccassi wrote:
>>
>>> Any chance you had a sec to look at the diff? Would love some
>>> feedback!
>>
>> I'm going to try and look at it this weekend. It's been a busy couple
>> of
>> months.
> 
> Hi Chet,
> 
> I've seen that the change is now in the devel branch, thank you so
> much!
> I also like way more the new option name, definitely clearer :-)
> 
> One difference I noticed with the previous version is that when trying
> to complete a command when the line starts with an empty space (eg: to
> avoid the history) the initial word programmable completion doesn't
> happen.
> 
> It seems to hit the following branch:
> 
>        else if (s >= e && n[0] == '\0' && text[0] == '\0' && start > 0)
>          {
>            foundcs = 0; /* empty command name following assignments */
>            in_command_position = was_assignment;
>          }
> 
> Given was_assignment is 0, then the programmable completion is later
> not attempted.
> 
> Since that branch matches an empty command name, should it perhaps call
> the EMPTYCMD completion?

It's not an empty command line, since it starts with point after column 0.
It should perform command completion, though.

Try the attached patch, or wait for the next devel branch push.

Chet

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU    address@hidden    http://tiswww.cwru.edu/~chet/

Attachment: command-completion.patch
Description: Text document

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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