GNU bug report logs - #79288
[PATCH] Add edebug-bounce-to-previous-value

Previous Next

Full log

Message #23 received at 79288 <at> debbugs.gnu.org (full text, mbox):

From: Jens Schmidt <jschmidt4gnu <at> vodafonemail.de>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 79288 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca
Subject: Re: bug#79288: [PATCH] Add edebug-bounce-to-previous-value
Date: Sat, 23 Aug 2025 22:41:09 +0200

[Message part 1 (text/plain, inline)]

On 2025-08-23  08:56, Eli Zaretskii wrote:

> We could rebind "P", since that command has another binding.  Or we
> could use "C-x p" ("M-p" has a disadvantage of meaning "previous" in
> many other modes, so it is less optimal).  But I can also live with
> "." or "j".

If you're OK with "P", I am too.  Changing that if somebody else
objects wouldn't be difficult.

Other than that here is what I would consider a feature-complete
patch:

- It is mostly based on what I have previously sent as v2 in message
  #17.
- Plus it adds documentation in edebug.texi (merged already with
  your recent index changes), ERTs (also for `edebug-bounce-point'),
  and a news entry.
- In the Edebug ERTs I noticed and cleaned up a side-effect in the
  otherwise unrelated test
  `edebug-tests-evaluation-of-current-buffer-bug-19611'.

I haven't yet described the actual use case of that new command, as
I did in the initial message of this bug report.  Do you think this
should be mentioned somewhere?  Without that, the function might
sound a bit arcane ...

Thanks!

[v3-0001-Add-edebug-bounce-to-previous-value.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.