all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Gustavo Barros <gusbrs.2016@gmail.com>
To: Juri Linkov <juri@linkov.net>
Cc: 48472@debbugs.gnu.org
Subject: bug#48472: [FR] A small suggestion for repeat-mode
Date: Tue, 18 May 2021 19:09:37 -0300	[thread overview]
Message-ID: <87h7izn17i.fsf@gmail.com> (raw)
In-Reply-To: <87a6orwy6x.fsf@mail.linkov.net>

Hi Juri,

On Tue, 18 May 2021 at 18:04, Juri Linkov <juri@linkov.net> wrote:

> It's amazing that the function returns exactly what we need
> even though its return value is not yet used anywhere.
> So I copied the same design to another similar function
> display-buffer-override-next-command to return exitfun as well.

Indeed it is pretty cool, nice that it has fulfilled its purpose and 
even "reproduced".

> You are welcome to sign papers, so the next time you could offer a 
> patch.

Well, I'd love to, and have tried to, but it turns out I can't sign 
them.

> Now I pushed the implementation based on your suggestion, thanks for 
> it!
>
> (The implementation also includes a lot of small details like 
> correctly
> handling the messages, etc.)

I thank you!

And I had assumed the timer would have to be stored and canceled at 
every repeat, and also that somehow the message would have to be cleared 
when the exit function timer ran.  I see you handled all that.  Looks 
good to me.

> If you are running on 27.2, you could still take the second part
> of repeat.el from Emacs 28.

I might!  :-)
But if you push me to you it, you run the risk of getting yet other 
"small suggestions".

Thanks again!

Gustavo.






      reply	other threads:[~2021-05-18 22:09 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-16 20:05 bug#48472: [FR] A small suggestion for repeat-mode Gustavo Barros
2021-05-17 21:24 ` Juri Linkov
2021-05-18 11:14   ` Gustavo Barros
2021-05-18 21:04     ` Juri Linkov
2021-05-18 22:09       ` Gustavo Barros [this message]

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h7izn17i.fsf@gmail.com \
    --to=gusbrs.2016@gmail.com \
    --cc=48472@debbugs.gnu.org \
    --cc=juri@linkov.net \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.