From: Christophe R. <cs...@ca...> - 2002-03-08 15:47:22
|
Um. Am I going mad? In src/compiler/main.lisp, the use and definition of COMPILE-MAKE-LOAD-FORM-INIT-FORMS don't match. I don't know what the correct patch is (I presume removing the second argument in the call, but that's just a guess), so I'll just point this out and let someone more qualified fix it... Cheers, Christophe -- Jesus College, Cambridge, CB5 8BL +44 1223 510 299 http://www-jcsu.jesus.cam.ac.uk/~csr21/ (defun pling-dollar (str schar arg) (first (last +))) (make-dispatch-macro-character #\! t) (set-dispatch-macro-character #\! #\$ #'pling-dollar) |
From: William H. N. <wil...@ai...> - 2002-03-09 04:50:47
|
On Fri, Mar 08, 2002 at 03:46:58PM +0000, Christophe Rhodes wrote: > Um. Am I going mad? No, probably not, just becoming bewildered by my behavior. Sorry. > In src/compiler/main.lisp, the use and definition of > COMPILE-MAKE-LOAD-FORM-INIT-FORMS don't match. I don't know what the > correct patch is (I presume removing the second argument in the call, > but that's just a guess), so I'll just point this out and let someone > more qualified fix it... The changes in 0.7.1.32 seem to've messed it up, and there was nothing in the regression tests to exercise that path through the code, so I checked it into CVS without realizing the problem. I dunno whether rebuilding SBCL with itself exercises that control path, because I neglected to install either .32 or .33 in /usr/local/* after checking them into CVS. I've fixed the arg list mismatch in 0.7.1.35, and added a new MAKE-LOAD-FORM test in tests/dump.impure-cload.lisp to try to make it less likely that future CVS checkins will have MAKE-LOAD-FORM as badly broken as this. Thanks for pointing out the problem. -- William Harold Newman <wil...@ai...> "Of course, if I dig my house foundations by biting the earth while banging my own head with a spade, then upgrading to mechanical digger maybe won't help..." -- Graham Perkins <gpe...@dm...> in comp.lang.eiffel PGP key fingerprint 85 CE 1C BA 79 8D 51 8C B9 25 FB EE E0 C3 E5 7C |