From: Peter Graves <peter@ar...> - 2006-08-18 10:40:48
On Thu, 17 Aug 2006 at 11:42:41 +0200, Eric Marsden wrote:
> I have run into two bugs in ABCL built from current CVS:
> * there seems to be a missing tag for the toplevel ; ":reset" causes
> a org.armedbear.lisp.Go exception and exits the JVM
> * compiling the attached file and executing the form below generates
> a backtrace. The form executes ok when run interpreted.
These bugs are now fixed in CVS.
For the record, the problem was the recent change to fix the special
binding leak in READ that Alan reported earlier this week.
It turns out that the change to readPreservingWhitespace() that fixes
this leak breaks things rather badly when applied to the nearly
identical code in faslReadPreservingWhitespace(), which appears to be
susceptible to the same leak but in fact is not, since it is only ever
called from an outer context which saves and restores the special
binding stack. (There's some magic here that I don't fully understand
right now, even though I wrote that code myself; ABCL's fasl-reading
mechanism should probably go on the list of things that should one day
be reviewed and possibly redone.)
In any case, ABCL now runs ansi-tests with only the expected failures
and has built SBCL 0.9.15.37 successfully (on Linux-x86, at least).
Thanks for reporting these bugs!