[PATCH bash ordered autocomplete] Order the autocompletion in bash

[J Rt]

What do you think about this? Should this be merged in master or not?
I am slightly reluctant to start 'messing up' with IFS in all the
codebase, I would be afraid to introduce bugs etc.

On Wed, Apr 8, 2020 at 2:05 PM J Rt <jean.rblt at gmail.com> wrote:
>
> No wories :) .
>
> I guess that putting local on IFS everywhere may solve the problem,
> but I had been playing a bit around without managing to get it to
> work. Guess I was missing an IFS somewhere.
>
>
> On Tue, Apr 7, 2020 at 3:49 PM Reed Wade <reedwade at misterbanal.net> wrote:
> >
> > Hello back !
> >
> > Sorry for big delay, Migadu got problems with imap access cause of
> > recent events.
> >
> > > - the version I sent last time works.
> >
> > cool !
> >
> > > - the version with the local var taken away and the "" protection works:
> >
> > cool !
> >
> > > _append_to_compreply () {
> > >     local IFS=" "
> > >     for word in "$(_sort_entries_string "$1")"; do
> > >         COMPREPLY+=($word)
> > >     done
> > > }
> > >
> > > - this does not work (with uncommented IFS it works as shown on the
> > > previous point), so I think the IFS var is needed. :
> > >
> > > bla bla bla
> >
> > Aha yup. I just discovered some days ago bash actualy use something
> > called "dynamic scopes". In short, if a var is define outside of the
> > scope, bash update this variable value. Except if "local" is used ofc.
> >
> > Is it possible to add "local IFS" where needed ? This will make the
> > _append_to_compreply IFS useless right ?