GNU bug report logs -
#31243
26.1; kmacro-insert-counter increments incorrectly
Previous Next
Reported by: Allen Li <darkfeline <at> felesatra.moe>
Date: Mon, 23 Apr 2018 02:34:01 UTC
Severity: minor
Found in version 26.1
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
Message #19 received at 31243-done <at> debbugs.gnu.org (full text, mbox):
> From: Allen Li <darkfeline <at> felesatra.moe>
> Date: Mon, 23 Apr 2018 00:58:00 -0700
> Cc: 31243 <at> debbugs.gnu.org,
> bug-gnu-emacs <bug-gnu-emacs-bounces+psainty=orcon.net.nz <at> gnu.org>
>
> On Sun, Apr 22, 2018 at 9:56 PM, Phil Sainty <psainty <at> orcon.net.nz> wrote:
> > Not a bug? I believe your expectations are incorrect.
> >
> >> C-u <f3> ;; kmacro-start-macro-or-insert-counter
> >
> >
> > This insert the *previous* macro counter. Which is sort of "undefined" for
> > the first iteration, but zero in practice.
>
> Ah, I see. I guess what I'm looking for is C-u 0 <f3> (or
> alternatively C-u <f3> the second time rather than the first time).
>
> I don't think the documentaton is very clear, because the meaning of
> "previous" is ambiguous:
>
> With C-u, inserts previous ‘kmacro-counter’ (but do not modify counter).
Indeed, the doc string could use an improvement, and in particular,
using "previous" was misleading. I've now clarified the doc strings,
and made a point of using "current value" in all places, instead ofg
sometimes using "previous".
One comment about what's been said by Phil:
> >> C-u <f3> ;; kmacro-start-macro-or-insert-counter
> >
> >
> > This insert the *previous* macro counter. Which is sort of "undefined" for
> > the first iteration, but zero in practice.
Actually, the value of the counter is documented to be initialized to
zero before defining the macro, so it's non "undefined" at this stage.
Thanks, I'm closing the bug report.
This bug report was last modified 7 years and 16 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.