From: Peter G. <pe...@ar...> - 2006-08-18 10:40:48
|
On Thu, 17 Aug 2006 at 11:42:41 +0200, Eric Marsden wrote: > Hi, > > 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! -Peter |