bug#13811: [Leo Liu] Byte-compile lexical-scoped elisp

bug#13811: [Leo Liu] Byte-compile lexical-scoped elisp

[Stefan Monnier]

[-- Attachment #1: Type: text/plain, Size: 33 bytes --]

Forwarding to get a bug-number.


[-- Attachment #2: Type: message/rfc822, Size: 2731 bytes --]

From: Leo Liu <sdl.web@gmail.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Subject: Byte-compile lexical-scoped elisp
Date: Mon, 25 Feb 2013 20:43:53 +0800
Message-ID: <m2hal0h8c6.fsf@gmail.com>

Hello Stefan,

I got a bug report on ack.el (https://github.com/leoliu/ack-el/issues/1)
when running ack.el uncompiled.

The steps to reproduce:

1. wget https://github.com/leoliu/ack-el/raw/master/ack.el
2. Emacs -q -l ack.el
3. M-x ack

file-name-nondirectory: Symbol's value as variable is void: project-root

Could you help me understand what is wrong here? It seems when
uncompiled project-root is not treated as a special variable.

Thank you,
Leo

bug#13811: Byte-compile lexical-scoped elisp

[Stefan Monnier]

> The steps to reproduce:
> 1. wget https://github.com/leoliu/ack-el/raw/master/ack.el
> 2. Emacs -q -l ack.el
> 3. M-x ack
> file-name-nondirectory: Symbol's value as variable is void: project-root

Indeed, the problem is that the dynamic binding of project-root is
created inside the `interactive' spec and that spec is not run in the
proper lexical environment (it is run in lexical mode, but in an empty
lexical environment).

> Could you help me understand what is wrong here?

A bug in Emacs.  Should be fixed on the trunk now.

BTW, please add an "ack--" prefix to your `project-root' var: all
`defvar'd variables should have such a prefix.


        Stefan