Re: undo-auto--undoable-change infloop

[Madhu]

*  Stefan Monnier <jwvzge74ay0.fsf-monnier+emacs@gnu.org>
Wrote on Fri, 07 Oct 2022 16:50:54 -0400

>> After this commit the last line of respool has to look like
>>
>> (timer-activate $current-search-timer nil (cancel-timer-internal
>>   $current-search-timer))
>>
>> Which uses an internal function,
>
> Just call `cancel-timer` instead of `cancel-timer-internal` and the
> problem is solved.

(CANCEL-TIMER returns NIL)

Earlier it was possible to avoid unnecessary consing, which is now
imposed on the user by the new API.  The new API may still be usable
but is gratuitously inefficient: I think the public/private API purity
fixation which motivated it is misguided here.