Re: address@hidden: Speed of all-completions]

[David Kastrup]

Andreas Schwab <address@hidden> writes:

> This is due to this change:
> 
>       (Ftry_completion, Fall_completions, Ftest_completion): Bind
>       case-fold-search to the value of completion-ignore-case when
>       checking completion-regexp-list.
> 
> I've checked in a fix that avoids the overhead of specbind when
> completion-regexp-list is empty.

At the cost of being more expensive when it isn't.  I don't see why
you have to undo the binding all the time.  Just initialize the
binding depth variable to something illegal at the start of the
function.  Then when you need the binding, you check whether it has
already been done and do it if needed.

And at the exit of the function, you check whether the binding had
happened and in that case undo it before returning.

That way you avoid the overhead of the binding when it is not needed,
but pay at most for one binding when it _is_ needed.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum