Subject: bug#56535: 28.1; `no' and `no-ding' values for isearch-wrap-pause option (since Emacs 28.1) not working as documented

From: Juri Linkov <[email-masked]>
Date: Sun, 17 Jul 2022 22:01:09 +0300

>>> Please try the following patch after setting isearch-wrap-pause to the
>>> new value 'auto-repeat':
>>>
>> Regret the delayed response.
>>
>> Yes, I do believe this would be a good idea.
>>
>> Let me try this patch atop emacs-28.1 and get back to you.
>
> Thanks Juri. Your patch works.

No problem with the delay.  Thanks for confirming that it works.

Now we have to find a better name to describe what it really does.
Probably it should have the prefix "no-" since it makes no pause
like the existing two values 'no' and 'no-ding':

  :type '(choice (const :tag "Pause before wrapping" t)
                 (const :tag "No pause before wrapping" no)
                 (const :tag "No pause and no flashing" no-ding)
                 (const :tag "Disable wrapping" nil))

This raises the question: should it ding and flash on wrapping?
Or two new separate values are needed: one to ding when wrapping
on typing a character, and another not to ding?  To ding, or not to ding?