Re: [PATCH] Add nofirstword completion option

[Luca Boccassi]

On Tue, 2018-06-12 at 14:54 -0400, Chet Ramey wrote:
> 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.

Ah, I see.

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

Can confirm it works like a charm, thanks again!

In case it can be somewhat useful, once 5.0-beta with the new option is
out I'll ask QA at $work to run regressions tests on it and report back
in case any issues are found. They have a lot of tests for the command
line interface (not just related to completion), so perhaps it could
help a little with testing the new version.

-- 
Kind regards,
Luca Boccassi

signature.asc
Description: This is a digitally signed message part