Re: _call_program (and possibly other hooks) or opt_args quoting prob lem.

[Sven Wischnowsky <wischnow@xxxxxxxxx>]

Bart Schaefer wrote:

> ...
> 
> On May 21, 10:14am, Sven Wischnowsky wrote:
> } 
> } I'm not sure if we should commit this patch, because, as was already
> } pointed out, ${(e)...} on a string as it is reported now should give
> } one the needed expansion. With the patch that would become ${(Qe)...}.
> } 
> } Any opinions?
> 
> Having looked at Andrej's example with _files, I think you should commit
> the patch.  It would appear that the number of cases where we'd need to
> insert uses of ${(e)...} without the patch, exceeds the number of cases
> where we'd need to insert ${(Qe)...} with it.
> 
> However, that opinion is based on very superficial analysis, so if anyone
> thinks otherwise ...

I had the same feeling anyway, I'm committing it now.


Bye
  Sven

-- 
Sven Wischnowsky                          wischnow@xxxxxxxxx