Re: PATCH: Re: completion in braces

[Sven Wischnowsky <wischnow@xxxxxxxxx>]

Apologies for not replying to the original post earlier. I meant to
answer it, even had a look at the code, couldn't find a good solution
and then got drowned by real work again.

Oliver Kiddle wrote:

> Following patch seems to fix this, at least for the new completion
> system. But I'm no expert on what's going on in the completion code so
> I'd be happier if someone else such as Sven could check this and perhaps
> get it to work for the compctl system too.

The problem with the patch is that it fixes it only for the (few)
cases where the completion code uses the compquote builtin. A better
solution would be to add the change at the place where the completion
or the unambiguous string is inserted into the line (in cline_str()
and instmatch() in compresult.c). But that's not trivial (finding the
commas that have to be quoted, ignoring those that don't have to be
quoted). It was at that place where I didn't find a good solution...
(from the code's point of view). And it's still the same, so no patch
yet.


Bye
  Sven

-- 
Sven Wischnowsky                          wischnow@xxxxxxxxx