GNU bug report logs -
#66394
29.1; Make register-read-with-preview more useful
Previous Next
Reported by: Thierry Volpiatto <thievol <at> posteo.net>
Date: Sat, 7 Oct 2023 19:07:01 UTC
Severity: normal
Found in version 29.1
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
Message #141 received at 66394 <at> debbugs.gnu.org (full text, mbox):
> From: Thierry Volpiatto <thievol <at> posteo.net>
> Cc: monnier <at> iro.umontreal.ca, michael_heerdegen <at> web.de,
> stefankangas <at> gmail.com, 66394 <at> debbugs.gnu.org
> Date: Wed, 29 Nov 2023 18:18:40 +0000
>
> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> > The response "a" to the "Point to register:" question is something I
> > needed to type;
>
> Of course we are using now read-from-minibuffer and the test is related
> to read-key not quitting with C-g
>
> > previously the test never asked any questions (and it shouldn't,
> > AFAIU).
>
> Yes, it shouldn't, but now we are using a real minibuffer, is this test
> really needed?
> It is like if you had to make tests in all functions using
> read-from-minibuffer to check if it quit properly with C-g (it does of course).
Can we have a similar issue with read-from-minibuffer? Not with C-g,
but with some other invalid input, like some function key or non-ASCII
character or invalid character codepoint? If this cannot happen, then
perhaps the whole test file should be removed?
Anyway, I trust you to do what is needed here, I just want the test
suite to keep running successfully, and I don't have time to work on
this myself. TIA.
> ALSO: I intentionally leave register-preview-delay defcustom with a FIXME
> comment, do you want to remove it? It is not used anymore now AFAIK.
This should be declared obsolete, with an explanation that it is no
longer used, and a note about that should be added to NEWS, in the
incompatible changes section. Then we can remove the FIXME.
Thanks.
This bug report was last modified 1 year and 209 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.