Will Noble | 1 Dec 2011 03:30

Macro expander bug

Hi,

I tried to load the package defs from SSAX and it killed the macro expander.
It appears that when BIND-ALIASES was switched to the new cenv
representation it stopped including macro eval in the cenvs it generated,
and SSAX touches it somehow, causing a (force #f).

I put up a one-line patch to BIND-ALIASES so that it includes macro eval, as
it did before, at https://bitbucket.org/wnoble/s48. However, I don't know
exactly how SSAX triggers the access and didn't try too hard to come up with
a simple macro to reproduce the faulty expansion. So, if someone would
explain whether this is the correct fix, and why or why not, I would be
grateful!

And Mike: thanks for integrating my POSIX patch!

Best,
Will

Michael Sperber | 13 Dec 2011 20:29
Picon

Re: Macro expander bug


Will Noble <will <at> cow9.org> writes:

> Hi,
>
> I tried to load the package defs from SSAX and it killed the macro expander.
> It appears that when BIND-ALIASES was switched to the new cenv
> representation it stopped including macro eval in the cenvs it generated,
> and SSAX touches it somehow, causing a (force #f).
>
> I put up a one-line patch to BIND-ALIASES so that it includes macro eval, as
> it did before, at https://bitbucket.org/wnoble/s48. However, I don't know
> exactly how SSAX triggers the access and didn't try too hard to come up with
> a simple macro to reproduce the faulty expansion. So, if someone would
> explain whether this is the correct fix, and why or why not, I would be
> grateful!

I've pulled & pushed the change - thanks!  I'm reluctant to get into the
old macro expander, which has many bugs. One of these days, I'll finish
up the new one started at:

http://www.s48.org/cgi-bin/hgwebdir.cgi/s48-syntax-case/

--

-- 
Cheers =8-} Mike
Friede, Völkerverständigung und überhaupt blabla


Gmane