unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
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  🦮



  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).