bug#54754: 28.1; Still requiring typing "yes" when read-short-answer is non-nil

bug#54754: 28.1; Still requiring typing "yes" when read-short-answer is non-nil

[Damien Cassou]

Hi,

when killing an unsaved buffer, Emacs would ask me to type "yes" to
express that I'm sure. Reading the documentation of `read-answer-short',
I thought that setting this variable to t would allow me to type just
"y" but it doesn't: I still have to type "yes".

In the same documentation, I can read that if the variable's value is
`auto' (which is the default but I changed it to t) then
`use-short-answers' is used to decide if typing "y" is enough. I tried
to set `use-short-answers' to t while keeping `read-answer-short' to t
and it solved my problem.

I suggest clarifying the documentation of `read-answer-short' to say
that the value of `use-short-answers' might have an impact even if
`read-answer-short' is t. I also suggest explaining when this is
necessary.

Best.


In GNU Emacs 28.1 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.30, cairo version 1.16.0)
Repository revision: emacs-28.1
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12014000
System Description: Fedora Linux 35 (Workstation Edition)

Configured using:
 'configure
 --prefix=/nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1
 --disable-build-details --with-modules --with-x-toolkit=gtk3 --with-xft
 --with-cairo --with-native-compilation'

Configured features:
CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GSETTINGS HARFBUZZ JPEG JSON
LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NATIVE_COMP NOTIFY
INOTIFY PDUMPER PNG RSVG SECCOMP SOUND THREADS TIFF TOOLKIT_SCROLL_BARS
X11 XDBE XIM XPM GTK3 ZLIB

Important settings:
  value of $EMACSLOADPATH: /nix/store/367h0f0q4avla4xgv4w0ip6w186nkpj5-emacs-packages-deps/share/emacs/site-lisp:
  value of $EMACSNATIVELOADPATH: /nix/store/367h0f0q4avla4xgv4w0ip6w186nkpj5-emacs-packages-deps/share/emacs/native-lisp::
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Info

Minor modes in effect:
  firestarter-mode: t
  editorconfig-mode: t
  all-the-icons-completion-mode: t
  marginalia-mode: t
  vertico-multiform-mode: t
  vertico-indexed-mode: t
  vertico-mode: t
  savehist-mode: t
  envrc-global-mode: t
  envrc-mode: t
  minions-mode: t
  mpdel-mode: t
  global-subword-mode: t
  subword-mode: t
  beginend-global-mode: t
  global-paren-face-mode: t
  electric-pair-mode: t
  which-key-mode: t
  drag-stuff-global-mode: t
  drag-stuff-mode: t
  org-roam-db-autosync-mode: t
  nano-modeline-mode: t
  global-git-commit-mode: t
  runner-run-in-background: t
  magit-auto-revert-mode: t
  shell-dirtrack-mode: t
  recentf-mode: t
  global-undo-tree-mode: t
  undo-tree-mode: t
  lin-global-mode: t
  global-hl-line-mode: t
  global-auto-revert-mode: t
  winner-mode: t
  save-place-mode: t
  global-so-long-mode: t
  auto-compile-on-load-mode: t
  auto-compile-on-save-mode: t
  override-global-mode: t
  global-eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  window-divider-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  buffer-read-only: t
  column-number-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
/home/cassou/.emacs.d/elpa/eslint-disable-rule-20220320.1848/eslint-disable-rule hides /home/cassou/.emacs.d/lib/eslint-disable-rule/eslint-disable-rule
/home/cassou/.emacs.d/elpa/eslint-disable-rule-20220320.1848/eslint-disable-rule-flymake hides /home/cassou/.emacs.d/lib/eslint-disable-rule/eslint-disable-rule-flymake
/home/cassou/.emacs.d/elpa/eslint-disable-rule-20220320.1848/eslint-disable-rule-all hides /home/cassou/.emacs.d/lib/eslint-disable-rule/eslint-disable-rule-all
/home/cassou/.emacs.d/elpa/eslint-disable-rule-20220320.1848/eslint-disable-rule-flycheck hides /home/cassou/.emacs.d/lib/eslint-disable-rule/eslint-disable-rule-flycheck
/home/cassou/.emacs.d/elpa/eslint-disable-rule-20220320.1848/eslint-disable-rule-autoloads hides /home/cassou/.emacs.d/lib/eslint-disable-rule/eslint-disable-rule-autoloads
/home/cassou/.emacs.d/lib/password-store/password-store hides /home/cassou/.nix-profile/share/emacs/site-lisp/password-store
/home/cassou/.nix-profile/share/emacs/site-lisp/site-start hides /nix/store/367h0f0q4avla4xgv4w0ip6w186nkpj5-emacs-packages-deps/share/emacs/site-lisp/site-start
/home/cassou/.nix-profile/share/emacs/site-lisp/site-start hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/site-start
/home/cassou/.emacs.d/lib/marginalia/marginalia hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/marginalia-20220404.804/marginalia
/home/cassou/.emacs.d/lib/marginalia/marginalia-autoloads hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/marginalia-20220404.804/marginalia-autoloads
/home/cassou/.emacs.d/lib/tablist/tablist-filter hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/tablist-20200427.2205/tablist-filter
/home/cassou/.emacs.d/lib/tablist/tablist-autoloads hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/tablist-20200427.2205/tablist-autoloads
/home/cassou/.emacs.d/lib/tablist/tablist hides /nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/tablist-20200427.2205/tablist
/home/cassou/.nix-profile/share/emacs/site-lisp/site-start hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/site-lisp/site-start
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/jsonrpc-1.0.15/jsonrpc hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/jsonrpc
/home/cassou/.emacs.d/lib/transient/lisp/transient hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/transient
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/flymake-1.2.2/flymake hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/progmodes/flymake
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/xref-1.4.1/xref hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/progmodes/xref
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/nadvice-0.3/nadvice hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/emacs-lisp/nadvice
/home/cassou/.emacs.d/lib/hierarchy/hierarchy hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/emacs-lisp/hierarchy
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/let-alist-1.0.6/let-alist hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/emacs-lisp/let-alist
/nix/store/bisf41cv7ix1qq2lv5nzn7cix8ch5853-emacs-packages-deps/share/emacs/site-lisp/elpa/eldoc-1.11.1/eldoc hides /nix/store/yw24pqjn68qa98n0pdrx9gbp5q1ca01l-emacs-gcc-28.1/share/emacs/28.1/lisp/emacs-lisp/eldoc

Features:
(shadow emacsbug misearch multi-isearch consult-xref dumb-jump popup
texinfo texinfo-loaddefs wgrep grep compile macrostep-c cmacexp
macrostep helpful trace edebug backtrace info-look f help-fns elisp-refs
embrace expand-region subword-mode-expansions text-mode-expansions
cc-mode-expansions the-org-mode-expansions python-el-fgallina-expansions
js2-mode-expansions js-mode-expansions er-basic-expansions
expand-region-core expand-region-custom diff-hl log-view vc-dir ewoc
ws-butler firestarter cal-china lunar solar cal-dst cal-bahai cal-islam
cal-hebrew holidays hol-loaddefs cal-move org-duration cal-iso
mixed-pitch disp-table ox-beamer org-protocol sendmail qp sort mail-extr
company-oddmuse company-keywords company-etags company-gtags
company-dabbrev-code company-files company-clang company-capf
company-cmake company-semantic company-template company-bbdb gnus-dired
editorconfig-core editorconfig-core-handle editorconfig-fnmatch vc-mtn
vc-hg vc-bzr vc-src vc-sccs vc-svn vc-cvs vc-rcs embark-consult
consult-vertico consult notmuch notmuch-tree notmuch-jump notmuch-hello
notmuch-show notmuch-print notmuch-crypto notmuch-mua notmuch-message
notmuch-draft notmuch-maildir-fcc notmuch-address notmuch-company
notmuch-parser notmuch-wash coolj notmuch-query goto-addr icalendar
diary-lib diary-loaddefs notmuch-tag notmuch-lib notmuch-version
notmuch-compat mm-view mml-smime smime dig time vertico-directory
mule-util tramp-cmds flyspell ispell editorconfig embark ffap cape
all-the-icons-completion all-the-icons all-the-icons-faces data-material
data-weathericons data-octicons data-fileicons data-faicons
data-alltheicons orderless epkg-marginalia epkg-melpa epkg-gelpa
epkg-utils epkg-list epkg-desc epkg closql marginalia vertico-multiform
vertico-buffer vertico-indexed vertico savehist envrc inheritenv
nix-sandbox ob-verb verb finsit-javascript finsit-js-xref xref-js2 vc
finsit-js-tests js2-imenu-extras finsit-js-tern tern finsit-js-mdn
finsit-js-kill finsit-js-js2 js2-refactor js2r-paredit js2r-conveniences
js2r-conditionals js2r-wrapping js2r-functions js2r-vars
mc-hide-unmatched-lines-mode mc-mark-more thingatpt mc-cycle-cursors
multiple-cursors-core rect js2r-iife js2r-formatting js2r-helpers s
js2-mode etags fileloop xref project finsit-js-import
finsit-js-htmlcanvas finsit-js-flycheck finsit-js-eslint eslintd-fix xdg
finsit-js-core finsit-js-company company-dabbrev company yasnippet
flycheck-package package-lint let-alist finder finder-inf flycheck js
cc-mode cc-fonts cc-guess cc-menus cc-cmds cc-styles cc-align cc-engine
cc-vars cc-defs url-queue minions ivy-mpdel ivy flx delsel ivy-faces
ivy-overlay colir mpdel mpdel-browser libmpdel-directory mpdel-playlist
mpdel-tablist mpdel-song mpdel-core navigel tablist tablist-filter
semantic/wisent/comp semantic/wisent semantic/wisent/wisent
semantic/util-modes semantic/util semantic semantic/tag semantic/lex
semantic/fw mode-local cedet libmpdel tq time-stamp alert log4e
notifications dbus xml finsit-basecamp epa-file libbcel libbcel-actions
libbcel-nav libbcel-structs libbcel-client libbcel-oauth request
libbcel-util finsit-magit finsit-bugref vc-git vc-dispatcher paredit
aggressive-indent nameless display-line-numbers magit-bookmark bookmark
pp finsit-core bug-reference cap-words superword subword beginend
auth-source-pass paren-face elec-pair unify-opening which-key drag-stuff
org-roam-dailies org-roam-migrate org-roam-mode org-roam-capture
org-roam-id org-roam-node org-roam-db org-roam-utils org-roam-compat
org-roam org-capture org-id ox-twbs ox-odt rng-loc rng-uri rng-parse
rng-match rng-dt rng-util rng-pttrn nxml-parse nxml-ns nxml-enc xmltok
nxml-util ox-latex ox-icalendar org-agenda org-refile ox-html table
ox-ascii ox-publish ox org-element avl-tree generator org org-macro
org-footnote org-pcomplete org-list org-faces org-entities noutline
outline org-version ob-python python tramp-sh tramp tramp-loaddefs
trampver tramp-integration files-x tramp-compat parse-time ls-lisp ob-R
ob-dot ob-emacs-lisp ob-shell ob ob-tangle org-src ob-ref ob-lob
ob-table ob-exp ob-comint ob-core ob-eval org-table oc-basic bibtex
iso8601 ol org-keys oc org-compat org-macs org-loaddefs find-func
cal-menu calendar cal-loaddefs emacsql-sqlite url-http url-auth url-gw
nsm advice emacsql emacsql-compiler goggles pulse color ace-link avy
nano-modeline magit-tbdiff magit-extras magit-submodule magit-obsolete
magit-popup magit-blame magit-stash magit-reflog magit-bisect magit-push
magit-pull magit-fetch magit-clone magit-remote magit-commit
magit-sequence magit-notes magit-worktree magit-tag magit-merge
magit-branch magit-reset magit-files magit-refs magit-status magit
package browse-url url url-proxy url-privacy url-expand url-methods
url-history url-cookie url-domsuf url-util mailcap url-handlers
url-parse url-vars magit-repos magit-apply magit-wip magit-log
which-func magit-diff smerge-mode diff-mode git-commit log-edit message
rmc puny dired-imenu imenu runner dired-aux dired-x dired dired-loaddefs
rfc822 mml mml-sec epa derived epg rfc6068 epg-config gnus-util rmail
rmail-loaddefs auth-source password-cache json map text-property-search
time-date mm-decode mm-bodies mm-encode mailabbrev mail-utils gmm-utils
mailheader pcvs-util add-log magit-core magit-autorevert magit-margin
magit-transient magit-process with-editor shell pcomplete comint server
ansi-color magit-mode transient magit-git magit-base magit-section
format-spec crm eieio eieio-core eieio-loaddefs dash recentf tree-widget
wid-edit undo-tree diff queue lin face-remap hl-line info-colors
autorevert filenotify winner ring saveplace so-long jka-compr
modus-operandi-theme modus-themes no-littering auto-compile comp
comp-cstr warnings packed cl-macs use-package use-package-ensure
use-package-delight use-package-diminish use-package-bind-key bind-key
easy-mmode use-package-core rx edmacro kmacro cl-extra help-mode cl-seq
borg subr-x pcase info cl-loaddefs cl-lib autoload radix-tree lisp-mnt
mail-parse rfc2231 rfc2047 rfc2045 mm-util ietf-drums mail-prsvr seq
byte-opt gv bytecomp byte-compile cconv iso-transl tooltip eldoc paren
electric uniquify ediff-hook vc-hooks lisp-float-type elisp-mode mwheel
term/x-win x-win term/common-win x-dnd tool-bar dnd fontset image
regexp-opt fringe tabulated-list replace newcomment text-mode lisp-mode
prog-mode register page tab-bar menu-bar rfn-eshadow isearch easymenu
timer select scroll-bar mouse jit-lock font-lock syntax font-core
term/tty-colors frame minibuffer cl-generic cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese eucjp-ms
cp51932 hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese composite emoji-zwj charscript charprop case-table
epa-hook jka-cmpr-hook help simple abbrev obarray cl-preloaded nadvice
button loaddefs faces cus-face macroexp files window text-properties
overlay sha1 md5 base64 format env code-pages mule custom widget
hashtable-print-readable backquote threads dbusbind inotify
dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit x multi-tty make-network-process
native-compile emacs)

Memory information:
((conses 16 1244344 187224)
 (symbols 48 78110 89)
 (strings 32 332205 55822)
 (string-bytes 1 10943191)
 (vectors 16 122697)
 (vector-slots 8 3033298 366234)
 (floats 8 1652 1377)
 (intervals 56 12168 1926)
 (buffers 992 43))

-- 
Damien Cassou

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill

bug#54754: 28.1; Still requiring typing "yes" when read-short-answer is non-nil

[Lars Ingebrigtsen]

Damien Cassou <damien@cassou.me> writes:

> when killing an unsaved buffer, Emacs would ask me to type "yes" to
> express that I'm sure. Reading the documentation of `read-answer-short',
> I thought that setting this variable to t would allow me to type just
> "y" but it doesn't: I still have to type "yes".
>
> In the same documentation, I can read that if the variable's value is
> `auto' (which is the default but I changed it to t) then
> `use-short-answers' is used to decide if typing "y" is enough. I tried
> to set `use-short-answers' to t while keeping `read-answer-short' to t
> and it solved my problem.
>
> I suggest clarifying the documentation of `read-answer-short' to say
> that the value of `use-short-answers' might have an impact even if
> `read-answer-short' is t. I also suggest explaining when this is
> necessary.

Hm, yes, this is a bit confusing.

But as the documentation for the `read-short-answers' says -- that
variable only affects how the function `read-answer' prompts.
`read-answer' is not involved at all when killing an unsaved buffer --
that's `yes-and-no-p', which has the length of the answer controlled by
the `use-short-answers' variable.

But `use-short-answers' also affects `read-answer', which makes this
doubly confusing.

I've now clarified this in the Emacs 29 doc string, and also added links
from the yes-or-no-p doc string.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no