Re: "void-variable: tree" in agenda in emacs 23.4.1

[Nicolas Goaziou <n.goaziou@gmail.com>]

Hello,

Daniel Clemente <n142857@gmail.com> writes:

>   since some days ago I get an export error after C-a a, batch export, C-e on .org files, … Backtrace at the end.
>
>   Even (avl-tree--root org-element--cache) or (avl-tree--root nil) produce the same error.
>
>   This is not a bug in org, but in Emacs 23.4.1. avl-tree.el says:
>
> (defmacro avl-tree--root (tree)
>   ;; Return the root node for an avl-tree.  INTERNAL USE ONLY.
>   `(avl-tree--node-left (avl-tree--dummyroot tree)))
>
>   And it should have a comma before the word „tree“ in the body. Like this (org-mode works with this):
>
> (defmacro avl-tree--root (tree)
>   ;; Return the root node for an avl-tree.  INTERNAL USE ONLY.
>   `(avl-tree--node-left (avl-tree--dummyroot ,tree)))
>
>   This was fixed in emacs from Bazaar (in rev. 104392, Fri 2011-05-27
>   20:03:26 -0300).

I wasn't aware of this bug. Thank you.

>   But at the moment org-mode does not work in Emacs 23.4.1, so I think it should have a work-around.
>   The fix can be: „if Emacs version is lower than […] then define the macro again with the correct code“. Like:
>
> (if (version< emacs-version "24")
>   (defmacro avl-tree--root (tree)
>   ;; Backport fix for older versions due to missing comma in ,tree
>   `(avl-tree--node-left (avl-tree--dummyroot ,tree)))
> )

I just avoided to use `avl-tree--root' altogether. Hopefully, this
should fix the problem.


Regards,

-- 
Nicolas Goaziou