GNU bug report logs - #61479
28.2; icomplete-in-buffer --- Error in post-command-hook: args-out-of-range

Previous Next

Package: emacs;

Reported by: Fernando de Morais <fernandodemorais.jf <at> gmail.com>

Date: Mon, 13 Feb 2023 14:59:02 UTC

Severity: normal

Found in version 28.2

Fixed in version 29.0.60

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Fernando de Morais <fernandodemorais.jf <at> gmail.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 61479 <at> debbugs.gnu.org
Subject: bug#61479: 28.2; icomplete-in-buffer --- Error in post-command-hook: args-out-of-range
Date: Sat, 18 Feb 2023 10:24:55 -0300
Hi Juri,

Juri Linkov <juri <at> linkov.net> writes:

>>> Does it also select a completion candidate?
>>
>> Well, when I hit C-j, it selects the candidate, but then:
>>
>>   - The buffer is modified in a way that only the selected candidate is
>>     visible;
>>   - The same is true for multiple C-M-i when in the candidate list;
>>   - I need to use `undo' to go back to the previous state of the buffer.
>
> Here is a short patch that completely fixes this problem.
> But probably it's too late for the release branch
> since icomplete never worked correctly in the buffer.
>
> diff --git a/lisp/icomplete.el b/lisp/icomplete.el
> index f7a91599f3b..aadc2e32787 100644
> --- a/lisp/icomplete.el
> +++ b/lisp/icomplete.el
> @@ -215,7 +215,11 @@ icomplete-force-complete-and-exit
>         ;; calculated, This causes the first cached completion to
>         ;; be taken (i.e. the one that the user sees highlighted)
>         completion-all-sorted-completions)
> -      (minibuffer-force-complete-and-exit)
> +      (if (window-minibuffer-p)
> +          (minibuffer-force-complete-and-exit)
> +        (minibuffer-force-complete (icomplete--field-beg)
> +                                   (icomplete--field-end)
> +                                   'dont-cycle))
>      ;; Otherwise take the faster route...
>      (minibuffer-complete-and-exit)))

This patch fixes the problem when we hit C-j to select a candidate, but
the problem persists if we type multiple C-M-i when in the candidate
list, unfortunately.

At least, now, `icomplete' in buffer is very usable.  😀

Thank you!

-- 
Regards,
Fernando de Morais.




This bug report was last modified 2 years and 162 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.