Re: [Bug-readline] directory_completion_hook interacts with tilde expansion

[Chet Ramey]

On 3/25/10 2:29 PM, "Stefan H. Holek · Jarn" wrote:
> Not so fast!
> 
> I have a custom dequoting function that performs backslash-dequoting,
> and I *must* install it as a directory_completion_hook as well, or I'd
> be unable to complete paths like 'funny\ dir/foo'.
> 
> The culprit appears to be this: While the filename_dequoting_function
> results are assigned to 'user_dirname', opendir is called with
> 'dirname', effectively receiving a still quoted directory name.
> 
> The rl_filename_completion_function again:
> 
>       if (rl_directory_completion_hook &&
> (*rl_directory_completion_hook) (&dirname))
>         {
>           free (users_dirname);
>           users_dirname = savestring (dirname);
>         }
>       else if (rl_completion_found_quote && rl_filename_dequoting_function)
>         {
>           temp = (*rl_filename_dequoting_function) (users_dirname,
> rl_completion_quote_character);
>           free (users_dirname);
>           users_dirname = temp;
>         }
>       directory = opendir (dirname); /* <-- STILL QUOTED! */
> 
> I fear you are going to tell me it's by design again, but I fail to see
> how this is useful behavior - yet. ;-)

Yeah, that looks like a problem.  I'll take a look.  Thanks for the
report.

Chet

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