Subject: bug#30073: 27.0.50; dired-do-delete ignores customization for short answers

From: Tino Calancha <[email-masked]>
Date: Sun, 14 Jan 2018 20:01:50 +0900

Juri Linkov <[email-masked]> writes:

>>> What is worse, it doesn't work at all - setting dired-deletion-confirmer
>>> to y-or-n-p has no effect on the question “Recursively delete ...? ”.
>>> It still expects “[yes, no, all, quit, help]” answers, not short ones
>>> like “y/n/!/q/?”.
>>
>> If you want to use y-or-n-p there, you will have to replace
>> dired--yes-no-all-quit-help with your own function.
>
> Thanks for the idea.  Here is the first version of its implementation:
Thank you for the patch.  I like it.
Is OK if we add docstring to those vars?

-(defvar dired-recursive-deletion-confirmer 'dired--yes-no-all-quit-help) ;; or 'dired--y-n-!-q-?
+(defvar dired-recursive-deletion-confirmer 'dired--yes-no-all-quit-help ; or 'dired--y-n-!-q-?
+  "Function for asking user confirmation to delete directories recursively.
+Set it to `dired--y-n-!-q-?' if you prefer reply shortly.")
 
-(defvar dired-deletion-confirmer 'yes-or-no-p) ; or y-or-n-p?
+(defvar dired-deletion-confirmer 'yes-or-no-p ; or y-or-n-p?
+  "Function for asking user confirmation to delete files.
+Set it to `y-or-n-p' if you prefer reply shorty.")