cd completion does not work in CVS version (unsafe word splitting in _alternative).

cd completion does not work in CVS version (unsafe word splitting in _alternative).

[Andrej Borsenkow]

With fresh CVS ... (actually, it was the reason I fetched CVS - it did not work
in patched pws-12 as well) cd into $cdpath does not work. I tracked it down to:

+_alternative:62> action=( _path_files -W \(/home/bor /home/bor/src /tools/src
/archive/sni/dbase\) -/ )
+_alternative:63> _path_files -J -default- -W (/home/bor /home/bor/src
/tools/src/archive/sni/dbase) -/
+

The relevant lines in _alternative are:

        action=( $=action )
        ${(e)action[1]} "$subopts[@]" "$expl[@]" "${(@e)~action[2,-1]}"

In other words, _every_ action with SP in argument is doomed because it is
splitted in multiple arguments. This is a general problem.

In case of _cd this results from:

    _alternative \
        'local-directories:local directories:_path_files -/' \
        "path-directories: directories in cdpath:_path_files -W
\\(${${(@)cdpath:#.}}\\) -/"

I don't see, why should we exclude `.'. If it is in cdpath and user requested
path-directories - why not?

/andrej

RE: cd completion does not work in CVS version (unsafe word splitting in _alternative).

[Andrej Borsenkow]

>         ${(e)action[1]} "$subopts[@]" "$expl[@]" "${(@e)~action[2,-1]}"

Sorry, this was already edited (but does not work for the same reason). The
correct is, 'course,

${(e)action[1]} "$subopts[@]" "$expl[@]" ${(e)~action[2,-1]}

/andrej