bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#47205: 27.1.91; [bug] unexpected input event interrupted expand-file


From: Michael Albinus
Subject: bug#47205: 27.1.91; [bug] unexpected input event interrupted expand-file
Date: Thu, 18 Mar 2021 11:07:39 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)

Eli Zaretskii <eliz@gnu.org> writes:

>> It’s from the issue https://github.com/bbatsov/helm-projectile/issues/154, 
>> 
>> “[bug] `helm-projectile-find-file' empty candidate when there is "*.iso/" 
>> folder”
>> 
>> Through the traceback I attached in first mail, the dbus event cause a 
>> unexpected return value in line 
>> 
>> https://github.com/emacs-helm/helm/blob/26d4e9e4d38a143d7dfb2fe206eacbc326944532/helm.el#L4649,
>> 
>> Then it’s function will show nothing.
>
> Is this because helm uses while-no-input or throw-on-input?  If so,
> does it help to bind while-no-input-ignore-events to a list that
> includes this kind of dbus events?

This doesn't help (yet), because dbus-event isn't handled via
while-no-input-ignore-events. The appended patch extends this for
dbus-event and file-notify. The following adapted test works then:

--8<---------------cut here---------------start------------->8---
# emacs --batch --eval "(message \"%s\" (catch 'input (let 
((while-no-input-ignore-events (cons 'dbus-event while-no-input-ignore-events)) 
(throw-on-input 'input)) (expand-file-name \"a.iso/b\" \"/\"))))"
--8<---------------cut here---------------end--------------->8---

Is it OK to push this to the master branch (plus doc adaptions)?

Best regards, Michael.

Attachment: txtlKmmDMzDeI.txt
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]