bug#3670: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd macros

[Carsten Dominik]

This looks to me as if it was a bug in keyboard macro execution,
which should call functions interactively.

- Carsten

On Jun 17, 2009, at 1:15 PM, Andreas Seltenreich wrote:

> > Your bug report will be posted to the emacs-pretest-bug@gnu.org  
> > mailing list.
>
> > Please describe exactly what actions triggered the bug
> > and the precise symptoms of the bug:
>
> org-store-link works fine when recording a keyboard macro but it is a
> NOP when executing it.  The following change appears to fix it.
>
> regards,
> andreas
>
> > In GNU Emacs 23.0.94.1 (x86_64-unknown-linux-gnu)
> > of 2009-05-28 on tengen
>
> --8<---------------cut here---------------start------------->8---
> *** org.el.~1.48.~      2009-02-20 10:02:49.000000000 +0100
> --- org.el      2009-06-17 12:55:15.000000000 +0200
> ***************
> *** 6594,6600 ****
>           desc (or desc cpltxt))
>      (if (equal desc "NONE") (setq desc nil))
>
> !     (if (and (interactive-p) link)
>         (progn
>           (setq org-stored-links
>                 (cons (list link desc) org-stored-links))
> --- 6594,6600 ----
>           desc (or desc cpltxt))
>      (if (equal desc "NONE") (setq desc nil))
>
> !     (if (and (or (interactive-p) executing-kbd-macro) link)
>         (progn
>           (setq org-stored-links
>                 (cons (list link desc) org-stored-links))
> --8<---------------cut here---------------end--------------->8---
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Remember: use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode