From: "T.V Raman" <raman@google.com>
To: monnier@iro.umontreal.ca
Cc: juri@linkov.net, raman@google.com, emacs-devel@gnu.org
Subject: Re: Q: How to programatically terminate repeat-mode
Date: Mon, 3 Oct 2022 07:20:45 -0700 [thread overview]
Message-ID: <25402.61373.427526.906328@retriever.mtv.corp.google.com> (raw)
In-Reply-To: <jwvr0zpcc4t.fsf-monnier+emacs@gnu.org>
yes, API-wise, repeat-mode providing a programmatic means of turning
off a repeat would be best; also I dont think there is a way for a
user to have multiple repeats active, which makes things simpler.
Stefan Monnier writes:
> Juri Linkov [2022-10-03 09:56:32] wrote:
>
> >> Replacing the earlier (push 7 unread-command-events)
> >> with
> >> (when repeat-exit-timer
> >> (timer-set-idle-time repeat-exit-timer 0))
> >>
> >> in my disable-repeat function that I called from the process-sentinel of
> >> the media player -- with check for exit as the guard condition has no
> >> effect and the repeat remains in effect.
> >
> > Maybe you are starting multiple timers? Currently repeat-mode cancels
> > all previously started timers, but unfortunately cancel-timer for
> > repeat-exit-timer doesn't run its timer function that should exit
> > the repeat map. I propose to add a new optional argument RUN-FUNCTION
> > to cancel-timer, that will run the timer function after canceling the timer.
> > This means copying this part of timer-event-handler to cancel-timer:
>
> I don't see any need to add some weird `run-function` arg to
> `cancel-timer`, here.
>
> Why can't the repeat-map code offer a function to ... "programatically
> terminate repeat-mode"? Since it set up the timer, it can know what
> function the timer will run and with what args and can thus run that
> same code if it wants to.
>
>
> Stefan
--
Thanks,
--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1 🦮
--
Thanks,
--Raman(I Search, I Find, I Misplace, I Research)
♉ Id: kg:/m/0285kf1 🦮
next prev parent reply other threads:[~2022-10-03 14:20 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-09 17:42 Q: How to programatically terminate repeat-mode T.V Raman
2022-09-10 19:18 ` Juri Linkov
2022-09-10 19:41 ` T.V Raman
2022-09-10 19:44 ` T.V Raman
2022-10-02 16:20 ` T.V Raman
2022-10-02 17:19 ` T.V Raman
2022-10-02 18:03 ` Juri Linkov
2022-10-03 0:28 ` T.V Raman
2022-10-03 6:56 ` Juri Linkov
2022-10-03 12:48 ` Stefan Monnier
2022-10-03 14:20 ` T.V Raman [this message]
2022-10-03 19:56 ` Juri Linkov
2022-10-03 21:41 ` T.V Raman
2022-10-03 23:01 ` T.V Raman
2022-10-03 14:19 ` T.V Raman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=25402.61373.427526.906328@retriever.mtv.corp.google.com \
--to=raman@google.com \
--cc=emacs-devel@gnu.org \
--cc=juri@linkov.net \
--cc=monnier@iro.umontreal.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).