0.9.7.11:
[sbcl/tcr.git] / BUGS
blobc546bc47bf03496dcec297db6dcb499f0841d763
1 REPORTING BUGS
3 Bugs can be reported on the help mailing list
4   sbcl-help@lists.sourceforge.net
5 or on the development mailing list
6   sbcl-devel@lists.sourceforge.net
8 Please include enough information in a bug report that someone reading
9 it can reproduce the problem, i.e. don't write
10      Subject: apparent bug in PRINT-OBJECT (or *PRINT-LENGTH*?)
11      PRINT-OBJECT doesn't seem to work with *PRINT-LENGTH*. Is this a bug?
12 but instead
13      Subject: apparent bug in PRINT-OBJECT (or *PRINT-LENGTH*?)
14      In sbcl-1.2.3 running under OpenBSD 4.5 on my Alpha box, when
15      I compile and load the file
16        (DEFSTRUCT (FOO (:PRINT-OBJECT (LAMBDA (X Y)
17                                         (LET ((*PRINT-LENGTH* 4))
18                                           (PRINT X Y)))))
19          X Y)
20      then at the command line type
21        (MAKE-FOO)
22      the program loops endlessly instead of printing the object.
25 NOTES:
27 There is also some information on bugs in the manual page and
28 in the TODO file. Eventually more such information may move here.
30 The gaps in the number sequence belong to old bug descriptions which
31 have gone away (typically because they were fixed, but sometimes for
32 other reasons, e.g. because they were moved elsewhere).
36   DEFSTRUCT almost certainly should overwrite the old LAYOUT information
37   instead of just punting when a contradictory structure definition
38   is loaded. As it is, if you redefine DEFSTRUCTs in a way which 
39   changes their layout, you probably have to rebuild your entire
40   program, even if you know or guess enough about the internals of
41   SBCL to wager that this (undefined in ANSI) operation would be safe.
43 3: "type checking of structure slots"
44   a:
45   ANSI specifies that a type mismatch in a structure slot
46   initialization value should not cause a warning.
47 WORKAROUND:
48   This one might not be fixed for a while because while we're big
49   believers in ANSI compatibility and all, (1) there's no obvious
50   simple way to do it (short of disabling all warnings for type
51   mismatches everywhere), and (2) there's a good portable
52   workaround, and (3) by their own reasoning, it looks as though
53   ANSI may have gotten it wrong. ANSI justifies this specification
54   by saying 
55     The restriction against issuing a warning for type mismatches
56     between a slot-initform and the corresponding slot's :TYPE
57     option is necessary because a slot-initform must be specified
58     in order to specify slot options; in some cases, no suitable
59     default may exist.
60   However, in SBCL (as in CMU CL or, for that matter, any compiler
61   which really understands Common Lisp types) a suitable default
62   does exist, in all cases, because the compiler understands the
63   concept of functions which never return (i.e. has return type NIL).
64   Thus, as a portable workaround, you can use a call to some
65   known-never-to-return function as the default. E.g.
66     (DEFSTRUCT FOO
67       (BAR (ERROR "missing :BAR argument")
68            :TYPE SOME-TYPE-TOO-HAIRY-TO-CONSTRUCT-AN-INSTANCE-OF))
69   or 
70     (DECLAIM (FTYPE (FUNCTION () NIL) MISSING-ARG))
71     (DEFUN REQUIRED-ARG () ; workaround for SBCL non-ANSI slot init typing
72       (ERROR "missing required argument")) 
73     (DEFSTRUCT FOO
74       (BAR (REQUIRED-ARG) :TYPE TRICKY-TYPE-OF-SOME-SORT)
75       (BLETCH (REQUIRED-ARG) :TYPE TRICKY-TYPE-OF-SOME-SORT)
76       (N-REFS-SO-FAR 0 :TYPE (INTEGER 0)))
77   Such code should compile without complaint and work correctly either
78   on SBCL or on any other completely compliant Common Lisp system.
80   b: &AUX argument in a boa-constructor without a default value means
81      "do not initilize this slot" and does not cause type error. But
82      an error may be signalled at read time and it would be good if
83      SBCL did it.
85   d: (fixed in 0.8.1.5)
87 27:
88   Sometimes (SB-EXT:QUIT) fails with 
89         Argh! maximum interrupt nesting depth (4096) exceeded, exiting
90         Process inferior-lisp exited abnormally with code 1
91   I haven't noticed a repeatable case of this yet.
93 33:
94   And as long as we're wishing, it would be awfully nice if INSPECT could
95   also report on closures, telling about the values of the bound variables.
97   Currently INSPECT and DESCRIBE do show the values, but showing the
98   names of the bindings would be even nicer.
101   The compiler assumes that any time a function of declared FTYPE
102   doesn't signal an error, its arguments were of the declared type.
103   E.g. compiling and loading
104     (DECLAIM (OPTIMIZE (SAFETY 3)))
105     (DEFUN FACTORIAL (X) (GAMMA (1+ X)))
106     (DEFUN GAMMA (X) X)
107     (DECLAIM (FTYPE (FUNCTION (UNSIGNED-BYTE)) FACTORIAL))
108     (DEFUN FOO (X)
109       (COND ((> (FACTORIAL X) 1.0E6)
110              (FORMAT T "too big~%"))
111             ((INTEGERP X)
112              (FORMAT T "exactly ~S~%" (FACTORIAL X)))
113             (T
114              (FORMAT T "approximately ~S~%" (FACTORIAL X)))))
115   then executing
116     (FOO 1.5)
117   will cause the INTEGERP case to be selected, giving bogus output a la
118     exactly 2.5
119   This violates the "declarations are assertions" principle.
120   According to the ANSI spec, in the section "System Class FUNCTION",
121   this is a case of "lying to the compiler", but the lying is done
122   by the code which calls FACTORIAL with non-UNSIGNED-BYTE arguments,
123   not by the unexpectedly general definition of FACTORIAL. In any case,
124   "declarations are assertions" means that lying to the compiler should
125   cause an error to be signalled, and should not cause a bogus
126   result to be returned. Thus, the compiler should not assume
127   that arbitrary functions check their argument types. (It might
128   make sense to add another flag (CHECKED?) to DEFKNOWN to 
129   identify functions which *do* check their argument types.)
130   (Also, verify that the compiler handles declared function
131   return types as assertions.)
134   The definitions of SIGCONTEXT-FLOAT-REGISTER and
135   %SET-SIGCONTEXT-FLOAT-REGISTER in x86-vm.lisp say they're not
136   supported on FreeBSD because the floating point state is not saved,
137   but at least as of FreeBSD 4.0, the floating point state *is* saved,
138   so they could be supported after all. Very likely 
139   SIGCONTEXT-FLOATING-POINT-MODES could now be supported, too.
142   Compiling and loading
143     (DEFUN FAIL (X) (THROW 'FAIL-TAG X))
144     (FAIL 12)
145   then requesting a BACKTRACE at the debugger prompt gives no information
146   about where in the user program the problem occurred.
148   (this is apparently mostly fixed on the SPARC, PPC, and x86 architectures:
149   while giving the backtrace the non-x86 systems complains about "unknown
150   source location: using block start", but apart from that the
151   backtrace seems reasonable. On x86 this is masked by bug 353. See
152   tests/debug.impure.lisp for a test case)
155   Using the pretty-printer from the command prompt gives funny
156   results, apparently because the pretty-printer doesn't know
157   about user's command input, including the user's carriage return
158   that the user, and therefore the pretty-printer thinks that
159   the new output block should start indented 2 or more characters
160   rightward of the correct location.
163   As reported by Winton Davies on a CMU CL mailing list 2000-01-10,
164   and reported for SBCL by Martin Atzmueller 2000-10-20: (TRACE GETHASH)
165   crashes SBCL. In general tracing anything which is used in the 
166   implementation of TRACE is likely to have the same problem.
169   ANSI says in one place that type declarations can be abbreviated even
170   when the type name is not a symbol, e.g.
171     (DECLAIM ((VECTOR T) *FOOVECTOR*))
172   SBCL doesn't support this. But ANSI says in another place that this
173   isn't allowed. So it's not clear this is a bug after all. (See the
174   e-mail on cmucl-help@cons.org on 2001-01-16 and 2001-01-17 from WHN
175   and Pierre Mai.)
178   RANDOM-INTEGER-EXTRA-BITS=10 may not be large enough for the RANDOM
179   RNG to be high quality near RANDOM-FIXNUM-MAX; it looks as though
180   the mean of the distribution can be systematically O(0.1%) wrong.
181   Just increasing R-I-E-B is probably not a good solution, since
182   it would decrease efficiency more than is probably necessary. Perhaps
183   using some sort of accept/reject method would be better.
186   Internally the compiler sometimes evaluates
187     (sb-kernel:type/= (specifier-type '*) (specifier-type t))
188   (I stumbled across this when I added an
189     (assert (not (eq type1 *wild-type*)))
190   in the NAMED :SIMPLE-= type method.) '* isn't really a type, and
191   in a type context should probably be translated to T, and so it's
192   probably wrong to ask whether it's equal to the T type and then (using
193   the EQ type comparison in the NAMED :SIMPLE-= type method) return NIL.
194   (I haven't tried to investigate this bug enough to guess whether
195   there might be any user-level symptoms.)
197   In fact, the type system is likely to depend on this inequality not
198   holding... * is not equivalent to T in many cases, such as 
199     (VECTOR *) /= (VECTOR T).
202   In sbcl-0.6.11.41 (and in all earlier SBCL, and in CMU
203   CL), out-of-line structure slot setters are horribly inefficient
204   whenever the type of the slot is declared, because out-of-line
205   structure slot setters are implemented as closures to save space,
206   so the compiler doesn't compile the type test into code, but
207   instead just saves the type in a lexical closure and interprets it
208   at runtime.
209     To exercise the problem, compile and load
210       (cl:in-package :cl-user)
211       (defstruct foo
212         (bar (error "missing") :type bar))
213       (defvar *foo*)
214       (defun wastrel1 (x)
215         (loop (setf (foo-bar *foo*) x)))
216       (defstruct bar)
217       (defvar *bar* (make-bar))
218       (defvar *foo* (make-foo :bar *bar*))
219       (defvar *setf-foo-bar* #'(setf foo-bar))
220       (defun wastrel2 (x)
221         (loop (funcall *setf-foo-bar* x *foo*)))
222   then run (WASTREL1 *BAR*) or (WASTREL2 *BAR*), hit Ctrl-C, and
223   use BACKTRACE, to see it's spending all essentially all its time
224   in %TYPEP and VALUES-SPECIFIER-TYPE and so forth.
225     One possible solution would be simply to give up on 
226   representing structure slot accessors as functions, and represent
227   them as macroexpansions instead. This can be inconvenient for users,
228   but it's not clear that it's worse than trying to help by expanding
229   into a horribly inefficient implementation.
230     As a workaround for the problem, #'(SETF FOO) expressions
231   can be replaced with (EFFICIENT-SETF-FUNCTION FOO), where
232 (defmacro efficient-setf-function (place-function-name)
233   (or #+sbcl (and (sb-int:info :function :accessor-for place-function-name)
234                   ;; a workaround for the problem, encouraging the
235                   ;; inline expansion of the structure accessor, so
236                   ;; that the compiler can optimize its type test
237                   (let ((new-value (gensym "NEW-VALUE-"))
238                         (structure-value (gensym "STRUCTURE-VALUE-")))
239                     `(lambda (,new-value ,structure-value)
240                        (setf (,place-function-name ,structure-value)
241                              ,new-value))))
242       ;; no problem, can just use the ordinary expansion
243       `(function (setf ,place-function-name))))
245 100:
246   There's apparently a bug in CEILING optimization which caused 
247   Douglas Crosher to patch the CMU CL version. Martin Atzmueller
248   applied the patches to SBCL and they didn't seem to cause problems
249   (as reported sbcl-devel 2001-05-04). However, since the patches
250   modify nontrivial code which was apparently written incorrectly
251   the first time around, until regression tests are written I'm not 
252   comfortable merging the patches in the CVS version of SBCL.
254 108:
255   (TIME (ROOM T)) reports more than 200 Mbytes consed even for
256   a clean, just-started SBCL system. And it seems to be right:
257   (ROOM T) can bring a small computer to its knees for a *long*
258   time trying to GC afterwards. Surely there's some more economical
259   way to implement (ROOM T).
261   Daniel Barlow doesn't know what fixed this, but observes that it 
262   doesn't seem to be the case in 0.8.7.3 any more.  Instead, (ROOM T)
263   in a fresh SBCL causes
265     debugger invoked on a SB-INT:BUG in thread 5911:
266         failed AVER: "(SAP= CURRENT END)"
268   unless a GC has happened beforehand.
270 117:
271   When the compiler inline expands functions, it may be that different
272   kinds of return values are generated from different code branches.
273   E.g. an inline expansion of POSITION generates integer results 
274   from one branch, and NIL results from another. When that inline
275   expansion is used in a context where only one of those results
276   is acceptable, e.g.
277     (defun foo (x)
278       (aref *a1* (position x *a2*)))
279   and the compiler can't prove that the unacceptable branch is 
280   never taken, then bogus type mismatch warnings can be generated.
281   If you need to suppress the type mismatch warnings, you can
282   suppress the inline expansion,
283     (defun foo (x)
284       #+sbcl (declare (notinline position)) ; to suppress bug 117 bogowarnings
285       (aref *a1* (position x *a2*)))
286   or, sometimes, suppress them by declaring the result to be of an
287   appropriate type,
288     (defun foo (x)
289       (aref *a1* (the integer (position x *a2*))))
291   This is not a new compiler problem in 0.7.0, but the new compiler
292   transforms for FIND, POSITION, FIND-IF, and POSITION-IF make it 
293   more conspicuous. If you don't need performance from these functions,
294   and the bogus warnings are a nuisance for you, you can return to
295   your pre-0.7.0 state of grace with
296     #+sbcl (declaim (notinline find position find-if position-if)) ; bug 117..
298   (see also bug 279)
300 124:
301    As of version 0.pre7.14, SBCL's implementation of MACROLET makes
302    the entire lexical environment at the point of MACROLET available
303    in the bodies of the macroexpander functions. In particular, it
304    allows the function bodies (which run at compile time) to try to
305    access lexical variables (which are only defined at runtime).
306    It doesn't even issue a warning, which is bad.
308    The SBCL behavior arguably conforms to the ANSI spec (since the
309    spec says that the behavior is undefined, ergo anything conforms).
310    However, it would be better to issue a compile-time error.
311    Unfortunately I (WHN) don't see any simple way to detect this
312    condition in order to issue such an error, so for the meantime
313    SBCL just does this weird broken "conforming" thing.
315    The ANSI standard says, in the definition of the special operator
316    MACROLET,
317        The macro-expansion functions defined by MACROLET are defined
318        in the lexical environment in which the MACROLET form appears.
319        Declarations and MACROLET and SYMBOL-MACROLET definitions affect
320        the local macro definitions in a MACROLET, but the consequences
321        are undefined if the local macro definitions reference any
322        local variable or function bindings that are visible in that
323        lexical environment.
324    Then it seems to contradict itself by giving the example
325         (defun foo (x flag)
326            (macrolet ((fudge (z)
327                          ;The parameters x and flag are not accessible
328                          ; at this point; a reference to flag would be to
329                          ; the global variable of that name.
330                          ` (if flag (* ,z ,z) ,z)))
331             ;The parameters x and flag are accessible here.
332              (+ x
333                 (fudge x)
334                 (fudge (+ x 1)))))
335    The comment "a reference to flag would be to the global variable
336    of the same name" sounds like good behavior for the system to have.
337    but actual specification quoted above says that the actual behavior
338    is undefined.
340    (Since 0.7.8.23 macroexpanders are defined in a restricted version
341    of the lexical environment, containing no lexical variables and
342    functions, which seems to conform to ANSI and CLtL2, but signalling
343    a STYLE-WARNING for references to variables similar to locals might
344    be a good thing.)
346 135:
347   Ideally, uninterning a symbol would allow it, and its associated
348   FDEFINITION and PROCLAIM data, to be reclaimed by the GC. However,
349   at least as of sbcl-0.7.0, this isn't the case. Information about
350   FDEFINITIONs and PROCLAIMed properties is stored in globaldb.lisp
351   essentially in ordinary (non-weak) hash tables keyed by symbols.
352   Thus, once a system has an entry in this system, it tends to live
353   forever, even when it is uninterned and all other references to it
354   are lost.
356 143:
357   (reported by Jesse Bouwman 2001-10-24 through the unfortunately
358   prominent SourceForge web/db bug tracking system, which is 
359   unfortunately not a reliable way to get a timely response from
360   the SBCL maintainers)
361       In the course of trying to build a test case for an 
362     application error, I encountered this behavior: 
363       If you start up sbcl, and then lay on CTRL-C for a 
364     minute or two, the lisp process will eventually say: 
365          %PRIMITIVE HALT called; the party is over. 
366     and throw you into the monitor. If I start up lisp, 
367     attach to the process with strace, and then do the same 
368     (abusive) thing, I get instead: 
369          access failure in heap page not marked as write-protected 
370     and the monitor again. I don't know enough to have the 
371     faintest idea of what is going on here. 
372       This is with sbcl 6.12, uname -a reports: 
373          Linux prep 2.2.19 #4 SMP Tue Apr 24 13:59:52 CDT 2001 i686 unknown 
374   I (WHN) have verified that the same thing occurs on sbcl-0.pre7.141
375   under OpenBSD 2.9 on my X86 laptop. Do be patient when you try it:
376   it took more than two minutes (but less than five) for me.
378 145:
379   a.
380   ANSI allows types `(COMPLEX ,FOO) to use very hairy values for
381   FOO, e.g. (COMPLEX (AND REAL (SATISFIES ODDP))). The old CMU CL
382   COMPLEX implementation didn't deal with this, and hasn't been
383   upgraded to do so. (This doesn't seem to be a high priority
384   conformance problem, since seems hard to construct useful code
385   where it matters.)
387   [ partially fixed by CSR in 0.8.17.17 because of a PFD ansi-tests
388     report that (COMPLEX RATIO) was failing; still failing on types of
389     the form (AND NUMBER (SATISFIES REALP) (SATISFIES ZEROP)). ]
391   b. (fixed in 0.8.3.43)
393 146:
394   Floating point errors are reported poorly. E.g. on x86 OpenBSD
395   with sbcl-0.7.1, 
396         * (expt 2.0 12777)
397         debugger invoked on condition of type SB-KERNEL:FLOATING-POINT-EXCEPTION:
398           An arithmetic error SB-KERNEL:FLOATING-POINT-EXCEPTION was signalled.
399         No traps are enabled? How can this be?
400   It should be possible to be much more specific (overflow, division
401   by zero, etc.) and of course the "How can this be?" should be fixable.
403   See also bugs #45.c and #183
405 162:
406   (reported by Robert E. Brown 2002-04-16) 
407   When a function is called with too few arguments, causing the
408   debugger to be entered, the uninitialized slots in the bad call frame 
409   seem to cause GCish problems, being interpreted as tagged data even
410   though they're not. In particular, executing ROOM in the
411   debugger at that point causes AVER failures:
412     * (machine-type)
413     "X86"
414     * (lisp-implementation-version)
415     "0.7.2.12"
416     * (typep 10)
417     ...
418     0] (room)
419     ...
420     failed AVER: "(SAP= CURRENT END)"
421   (Christophe Rhodes reports that this doesn't occur on the SPARC, which
422   isn't too surprising since there are many differences in stack
423   implementation and GC conservatism between the X86 and other ports.)
425   This is probably the same bug as 216
427 173:
428   The compiler sometimes tries to constant-fold expressions before
429   it checks to see whether they can be reached. This can lead to 
430   bogus warnings about errors in the constant folding, e.g. in code
431   like 
432     (WHEN X
433       (WRITE-STRING (> X 0) "+" "0"))
434   compiled in a context where the compiler can prove that X is NIL,
435   and the compiler complains that (> X 0) causes a type error because
436   NIL isn't a valid argument to #'>. Until sbcl-0.7.4.10 or so this
437   caused a full WARNING, which made the bug really annoying because then 
438   COMPILE and COMPILE-FILE returned FAILURE-P=T for perfectly legal
439   code. Since then the warning has been downgraded to STYLE-WARNING, 
440   so it's still a bug but at least it's a little less annoying.
442 183: "IEEE floating point issues"
443   Even where floating point handling is being dealt with relatively
444   well (as of sbcl-0.7.5, on sparc/sunos and alpha; see bug #146), the
445   accrued-exceptions and current-exceptions part of the fp control
446   word don't seem to bear much relation to reality. E.g. on
447   SPARC/SunOS:
448   * (/ 1.0 0.0)
450   debugger invoked on condition of type DIVISION-BY-ZERO:
451     arithmetic error DIVISION-BY-ZERO signalled
452   0] (sb-vm::get-floating-point-modes)
454   (:TRAPS (:OVERFLOW :INVALID :DIVIDE-BY-ZERO)
455           :ROUNDING-MODE :NEAREST
456           :CURRENT-EXCEPTIONS NIL
457           :ACCRUED-EXCEPTIONS (:INEXACT)
458           :FAST-MODE NIL)
459   0] abort
460   * (sb-vm::get-floating-point-modes)
461   (:TRAPS (:OVERFLOW :INVALID :DIVIDE-BY-ZERO)
462           :ROUNDING-MODE :NEAREST
463           :CURRENT-EXCEPTIONS (:INEXACT)
464           :ACCRUED-EXCEPTIONS (:INEXACT)
465           :FAST-MODE NIL)
467 188: "compiler performance fiasco involving type inference and UNION-TYPE"
468     (time (compile
469            nil
470            '(lambda ()
471              (declare (optimize (safety 3)))
472              (declare (optimize (compilation-speed 2)))
473              (declare (optimize (speed 1) (debug 1) (space 1)))
474              (let ((start 4))
475                (declare (type (integer 0) start))
476                (print (incf start 22))
477                (print (incf start 26))
478                (print (incf start 28)))
479              (let ((start 6))
480                (declare (type (integer 0) start))
481                (print (incf start 22))
482                (print (incf start 26)))
483              (let ((start 10))
484                (declare (type (integer 0) start))
485                (print (incf start 22))
486                (print (incf start 26))))))
488   This example could be solved with clever enough constraint
489   propagation or with SSA, but consider
491     (let ((x 0))
492       (loop (incf x 2)))
494   The careful type of X is {2k} :-(. Is it really important to be
495   able to work with unions of many intervals?
497 191: "Miscellaneous PCL deficiencies"
498   (reported by Alexey Dejneka sbcl-devel 2002-08-04)
499   a. DEFCLASS does not inform the compiler about generated
500      functions. Compiling a file with
501        (DEFCLASS A-CLASS ()
502          ((A-CLASS-X)))
503        (DEFUN A-CLASS-X (A)
504          (WITH-SLOTS (A-CLASS-X) A
505            A-CLASS-X))
506      results in a STYLE-WARNING:
507        undefined-function 
508          SB-SLOT-ACCESSOR-NAME::|COMMON-LISP-USER A-CLASS-X slot READER|
510      APD's fix for this was checked in to sbcl-0.7.6.20, but Pierre
511      Mai points out that the declamation of functions is in fact
512      incorrect in some cases (most notably for structure
513      classes).  This means that at present erroneous attempts to use
514      WITH-SLOTS and the like on classes with metaclass STRUCTURE-CLASS
515      won't get the corresponding STYLE-WARNING.
516   c. (fixed in 0.8.4.23)
518 201: "Incautious type inference from compound types"
519   a. (reported by APD sbcl-devel 2002-09-17)
520     (DEFUN FOO (X)
521       (LET ((Y (CAR (THE (CONS INTEGER *) X))))
522         (SETF (CAR X) NIL)
523         (FORMAT NIL "~S IS ~S, Y = ~S"
524                 (CAR X)
525                 (TYPECASE (CAR X)
526                   (INTEGER 'INTEGER)
527                   (T '(NOT INTEGER)))
528                 Y)))
530     (FOO ' (1 . 2)) => "NIL IS INTEGER, Y = 1"
532   b.
533     * (defun foo (x)
534         (declare (type (array * (4 4)) x))
535         (let ((y x))
536           (setq x (make-array '(4 4)))
537           (adjust-array y '(3 5))
538           (= (array-dimension y 0) (eval `(array-dimension ,y 0)))))
539     FOO
540     * (foo (make-array '(4 4) :adjustable t))
541     NIL
543 205: "environment issues in cross compiler"
544   (These bugs have no impact on user code, but should be fixed or
545   documented.)
546   a. Macroexpanders introduced with MACROLET are defined in the null
547      lexical environment.
548   b. The body of (EVAL-WHEN (:COMPILE-TOPLEVEL) ...) is evaluated in
549      the null lexical environment.
550   c. The cross-compiler cannot inline functions defined in a non-null
551      lexical environment.
553 206: ":SB-FLUID feature broken"
554   (reported by Antonio Martinez-Shotton sbcl-devel 2002-10-07)
555   Enabling :SB-FLUID in the target-features list in sbcl-0.7.8 breaks
556   the build.
558 207: "poorly distributed SXHASH results for compound data"
559   SBCL's SXHASH could probably try a little harder. ANSI: "the
560   intent is that an implementation should make a good-faith
561   effort to produce hash-codes that are well distributed
562   within the range of non-negative fixnums". But
563         (let ((hits (make-hash-table)))
564           (dotimes (i 16)
565             (dotimes (j 16)
566               (let* ((ij (cons i j))
567                      (newlist (push ij (gethash (sxhash ij) hits))))
568                 (when (cdr newlist)
569                   (format t "~&collision: ~S~%" newlist))))))
570   reports lots of collisions in sbcl-0.7.8. A stronger MIX function
571   would be an obvious way of fix. Maybe it would be acceptably efficient
572   to redo MIX using a lookup into a 256-entry s-box containing
573   29-bit pseudorandom numbers?
575 211: "keywords processing"
576   a. :ALLOW-OTHER-KEYS T should allow a function to receive an odd
577      number of keyword arguments.
579 212: "Sequence functions and circular arguments"
580   COERCE, MERGE and CONCATENATE go into an infinite loop when given
581   circular arguments; it would be good for the user if they could be
582   given an error instead (ANSI 17.1.1 allows this behaviour on the part
583   of the implementation, as conforming code cannot give non-proper
584   sequences to these functions.  MAP also has this problem (and
585   solution), though arguably the convenience of being able to do
586     (MAP 'LIST '+ FOO '#1=(1 . #1#))
587   might be classed as more important (though signalling an error when
588   all of the arguments are circular is probably desireable).
590 213: "Sequence functions and type checking"
591   b. MAP, when given a type argument that is SUBTYPEP LIST, does not
592      check that it will return a sequence of the given type.  Fixing
593      it along the same lines as the others (cf. work done around
594      sbcl-0.7.8.45) is possible, but doing so efficiently didn't look
595      entirely straightforward.
596   c. All of these functions will silently accept a type of the form
597        (CONS INTEGER *)
598      whether or not the return value is of this type.  This is
599      probably permitted by ANSI (see "Exceptional Situations" under
600      ANSI MAKE-SEQUENCE), but the DERIVE-TYPE mechanism does not
601      know about this escape clause, so code of the form
602        (INTEGERP (CAR (MAKE-SEQUENCE '(CONS INTEGER *) 2)))
603      can erroneously return T.
605 215: ":TEST-NOT handling by functions"
606   a. FIND and POSITION currently signal errors when given non-NIL for
607      both their :TEST and (deprecated) :TEST-NOT arguments, but by
608      ANSI 17.2 "the consequences are unspecified", which by ANSI 1.4.2
609      means that the effect is "unpredictable but harmless".  It's not
610      clear what that actually means; it may preclude conforming
611      implementations from signalling errors.
612   b. COUNT, REMOVE and the like give priority to a :TEST-NOT argument
613      when conflict occurs.  As a quality of implementation issue, it
614      might be preferable to treat :TEST and :TEST-NOT as being in some
615      sense the same &KEY, and effectively take the first test function in
616      the argument list.
617   c. Again, a quality of implementation issue: it would be good to issue a
618      STYLE-WARNING at compile-time for calls with :TEST-NOT, and a
619      WARNING for calls with both :TEST and :TEST-NOT; possibly this
620      latter should be WARNed about at execute-time too.
622 216: "debugger confused by frames with invalid number of arguments"
623   In sbcl-0.7.8.51, executing e.g. (VECTOR-PUSH-EXTEND T), BACKTRACE, Q
624   leaves the system confused, enough so that (QUIT) no longer works.
625   It's as though the process of working with the uninitialized slot in
626   the bad VECTOR-PUSH-EXTEND frame causes GC problems, though that may
627   not be the actual problem. (CMU CL 18c doesn't have problems with this.)
629   This is probably the same bug as 162
631 217: "Bad type operations with FUNCTION types"
632   In sbcl.0.7.7:
634     * (values-type-union (specifier-type '(function (base-char)))
635                          (specifier-type '(function (integer))))
637     #<FUN-TYPE (FUNCTION (BASE-CHAR) *)>
639   It causes insertion of wrong type assertions into generated
640   code. E.g.
642     (defun foo (x s)
643       (let ((f (etypecase x
644                  (character #'write-char)
645                  (integer #'write-byte))))
646         (funcall f x s)
647         (etypecase x
648           (character (write-char x s))
649           (integer (write-byte x s)))))
651    Then (FOO #\1 *STANDARD-OUTPUT*) signals type error.
653   (In 0.7.9.1 the result type is (FUNCTION * *), so Python does not
654   produce invalid code, but type checking is not accurate.)
656 233: bugs in constraint propagation
657   b.
658   (declaim (optimize (speed 2) (safety 3)))
659   (defun foo (x y)
660     (if (typep (prog1 x (setq x y)) 'double-float)
661         (+ x 1d0)
662         (+ x 2)))
663   (foo 1d0 5) => segmentation violation
665 235: "type system and inline expansion"
666   a.
667   (declaim (ftype (function (cons) number) acc))
668   (declaim (inline acc))
669   (defun acc (c)
670     (the number (car c)))
672   (defun foo (x y)
673     (values (locally (declare (optimize (safety 0)))
674               (acc x))
675             (locally (declare (optimize (safety 3)))
676               (acc y))))
678   (foo '(nil) '(t)) => NIL, T.
680 237: "Environment arguments to type functions"
681   a. Functions SUBTYPEP, TYPEP, UPGRADED-ARRAY-ELEMENT-TYPE, and 
682      UPGRADED-COMPLEX-PART-TYPE now have an optional environment
683      argument, but they ignore it completely.  This is almost 
684      certainly not correct.
685   b. Also, the compiler's optimizers for TYPEP have not been informed
686      about the new argument; consequently, they will not transform
687      calls of the form (TYPEP 1 'INTEGER NIL), even though this is
688      just as optimizeable as (TYPEP 1 'INTEGER).
690 238: "REPL compiler overenthusiasm for CLOS code"
691   From the REPL,
692     * (defclass foo () ())
693     * (defmethod bar ((x foo) (foo foo)) (call-next-method))
694   causes approximately 100 lines of code deletion notes.  Some
695   discussion on this issue happened under the title 'Three "interesting"
696   bugs in PCL', resulting in a fix for this oververbosity from the
697   compiler proper; however, the problem persists in the interactor
698   because the notion of original source is not preserved: for the
699   compiler, the original source of the above expression is (DEFMETHOD
700   BAR ((X FOO) (FOO FOO)) (CALL-NEXT-METHOD)), while by the time the
701   compiler gets its hands on the code needing compilation from the REPL,
702   it has been macroexpanded several times.
704   A symptom of the same underlying problem, reported by Tony Martinez:
705     * (handler-case
706         (with-input-from-string (*query-io* "    no")
707           (yes-or-no-p))
708       (simple-type-error () 'error))
709     ; in: LAMBDA NIL
710     ;     (SB-KERNEL:FLOAT-WAIT)
711     ; 
712     ; note: deleting unreachable code
713     ; compilation unit finished
714     ;   printed 1 note
716 242: "WRITE-SEQUENCE suboptimality"
717   (observed from clx performance)
718   In sbcl-0.7.13, WRITE-SEQUENCE of a sequence of type 
719   (SIMPLE-ARRAY (UNSIGNED-BYTE 8) (*)) on a stream with element-type
720   (UNSIGNED-BYTE 8) will write to the stream one byte at a time,
721   rather than writing the sequence in one go, leading to severe
722   performance degradation.
723   As of sbcl-0.9.0.36, this is solved for fd-streams, so is less of a
724   problem in practice.  (Fully fixing this would require adding a
725   ansi-stream-n-bout slot and associated methods to write a byte
726   sequence to ansi-stream, similar to the existing ansi-stream-sout
727   slot/functions.)
729 243: "STYLE-WARNING overenthusiasm for unused variables"
730   (observed from clx compilation)
731   In sbcl-0.7.14, in the presence of the macros
732     (DEFMACRO FOO (X) `(BAR ,X))
733     (DEFMACRO BAR (X) (DECLARE (IGNORABLE X)) 'NIL)
734   somewhat surprising style warnings are emitted for
735     (COMPILE NIL '(LAMBDA (Y) (FOO Y))):
736   ; in: LAMBDA (Y)
737   ;     (LAMBDA (Y) (FOO Y))
738   ; 
739   ; caught STYLE-WARNING:
740   ;   The variable Y is defined but never used.
742 245: bugs in disassembler
743   b. On X86 operand size prefix is not recognized.
745 251:
746   (defun foo (&key (a :x))
747     (declare (fixnum a))
748     a)
750   does not cause a warning. (BTW: old SBCL issued a warning, but for a
751   function, which was never called!)
753 256:
754   Compiler does not emit warnings for
756   a. (lambda () (svref (make-array 8 :adjustable t) 1))
758   b. (lambda (x)
759        (list (let ((y (the real x)))
760                (unless (floatp y) (error ""))
761                y)
762              (integer-length x)))
764   c. (lambda (x)
765        (declare (optimize (debug 0)))
766        (declare (type vector x))
767        (list (fill-pointer x)
768              (svref x 1)))
770 257:
771   Complex array type does not have corresponding type specifier.
773   This is a problem because the compiler emits optimization notes when
774   you use a non-simple array, and without a type specifier for hairy
775   array types, there's no good way to tell it you're doing it
776   intentionally so that it should shut up and just compile the code.
778   Another problem is confusing error message "asserted type ARRAY
779   conflicts with derived type (VALUES SIMPLE-VECTOR &OPTIONAL)" during
780   compiling (LAMBDA (V) (VALUES (SVREF V 0) (VECTOR-POP V))).
782   The last problem is that when type assertions are converted to type
783   checks, types are represented with type specifiers, so we could lose
784   complex attribute. (Now this is probably not important, because
785   currently checks for complex arrays seem to be performed by
786   callees.)
788 259:
789   (compile nil '(lambda () (aref (make-array 0) 0))) compiles without
790   warning.  Analogous cases with the index and length being equal and
791   greater than 0 are warned for; the problem here seems to be that the
792   type required for an array reference of this type is (INTEGER 0 (0))
793   which is canonicalized to NIL.
795 260:
796   a.
797   (let* ((s (gensym))
798          (t1 (specifier-type s)))
799     (eval `(defstruct ,s))
800     (type= t1 (specifier-type s)))
801   => NIL, NIL
803   (fixed in 0.8.1.24)
805   b. The same for CSUBTYPEP.
807 262: "yet another bug in inline expansion of local functions"
808   During inline expansion of a local function Python can try to
809   reference optimized away objects (functions, variables, CTRANs from
810   tags and blocks), which later may lead to problems. Some of the
811   cases are worked around by forbidding expansion in such cases, but
812   the better way would be to reimplement inline expansion by copying
813   IR1 structures.
815 266:
816   David Lichteblau provided (sbcl-devel 2003-06-01) a patch to fix
817   behaviour of streams with element-type (SIGNED-BYTE 8).  The patch
818   looks reasonable, if not obviously correct; however, it caused the
819   PPC/Linux port to segfault during warm-init while loading
820   src/pcl/std-class.fasl.  A workaround patch was made, but it would
821   be nice to understand why the first patch caused problems, and to
822   fix the cause if possible.
824 268: "wrong free declaration scope"
825   The following code must signal type error:
827     (locally (declare (optimize (safety 3)))
828       (flet ((foo (x &optional (y (car x)))
829                (declare (optimize (safety 0)))
830                (list x y)))
831         (funcall (eval #'foo) 1)))
833 270:
834   In the following function constraint propagator optimizes nothing:
836     (defun foo (x)
837       (declare (integer x))
838       (declare (optimize speed))
839       (typecase x
840         (fixnum "hala")
841         (fixnum "buba")
842         (bignum "hip")
843         (t "zuz")))
845 273:
846   Compilation of the following two forms causes "X is unbound" error:
848     (symbol-macrolet ((x pi))
849       (macrolet ((foo (y) (+ x y)))
850         (declaim (inline bar))
851         (defun bar (z)
852           (* z (foo 4)))))
853     (defun quux (z)
854       (bar z))
856   (See (COERCE (CDR X) 'FUNCTION) in IR1-CONVERT-INLINE-LAMBDA.)
858 274:
859   CLHS says that type declaration of a symbol macro should not affect
860   its expansion, but in SBCL it does. (If you like magic and want to
861   fix it, don't forget to change all uses of MACROEXPAND to
862   MACROEXPAND*.)
864 275:
865   The following code (taken from CLOCC) takes a lot of time to compile:
867     (defun foo (n)
868       (declare (type (integer 0 #.large-constant) n))
869       (expt 1/10 n))
871   (fixed in 0.8.2.51, but a test case would be good)
873 276:
874   b. The same as in a., but using MULTIPLE-VALUE-SETQ instead of SETQ.
875   c. (defvar *faa*)
876      (defmethod faa ((*faa* double-float))
877            (set '*faa* (when (< *faa* 0) (- *faa*)))
878            (1+ *faa*))
879      (faa 1d0) => type error
881 279: type propagation error -- correctly inferred type goes astray?
882   In sbcl-0.8.3 and sbcl-0.8.1.47, the warning
883        The binding of ABS-FOO is a (VALUES (INTEGER 0 0)
884        &OPTIONAL), not a (INTEGER 1 536870911)
885   is emitted when compiling this file:
886     (declaim (ftype (function ((integer 0 #.most-positive-fixnum))
887                               (integer #.most-negative-fixnum 0))
888                     foo))
889     (defun foo (x)
890       (- x))
891     (defun bar (x)
892       (let* (;; Uncomment this for a type mismatch warning indicating 
893              ;; that the type of (FOO X) is correctly understood.
894              #+nil (fs-foo (float-sign (foo x)))
895                    ;; Uncomment this for a type mismatch warning 
896                    ;; indicating that the type of (ABS (FOO X)) is
897                    ;; correctly understood.
898              #+nil (fs-abs-foo (float-sign (abs (foo x))))
899              ;; something wrong with this one though
900              (abs-foo (abs (foo x))))
901         (declare (type (integer 1 100) abs-foo))
902         (print abs-foo)))
904  (see also bug 117)
906 283: Thread safety: libc functions
907   There are places that we call unsafe-for-threading libc functions
908   that we should find alternatives for, or put locks around.  Known or
909   strongly suspected problems, as of 0.8.3.10: please update this
910   bug instead of creating new ones
912     localtime() - called for timezone calculations in code/time.lisp
914 284: Thread safety: special variables
915   There are lots of special variables in SBCL, and I feel sure that at
916   least some of them are indicative of potentially thread-unsafe 
917   parts of the system.  See doc/internals/notes/threading-specials
919 286: "recursive known functions"
920   Self-call recognition conflicts with known function
921   recognition. Currently cross compiler and target COMPILE do not
922   recognize recursion, and in target compiler it can be disabled. We
923   can always disable it for known functions with RECURSIVE attribute,
924   but there remains a possibility of a function with a
925   (tail)-recursive simplification pass and transforms/VOPs for base
926   cases.
928 287: PPC/Linux miscompilation or corruption in first GC
929   When the runtime is compiled with -O3 on certain PPC/Linux machines, a
930   segmentation fault is reported at the point of first triggered GC,
931   during the compilation of DEFSTRUCT WRAPPER.  As a temporary workaround,
932   the runtime is no longer compiled with -O3 on PPC/Linux, but it is likely
933   that this merely obscures, not solves, the underlying problem; as and when
934   underlying problems are fixed, it would be worth trying again to provoke
935   this problem.
937 288: fundamental cross-compilation issues (from old UGLINESS file)
938   Using host floating point numbers to represent target floating point
939   numbers, or host characters to represent target characters, is
940   theoretically shaky. (The characters are OK as long as the characters
941   are in the ANSI-guaranteed character set, though, so they aren't a
942   real problem as long as the sources don't need anything but that;
943   the floats are a real problem.)
945 289: "type checking and source-transforms"
946   a.
947     (block nil (let () (funcall #'+ (eval 'nil) (eval '1) (return :good))))
948   signals type error.
950   Our policy is to check argument types at the moment of a call. It
951   disagrees with ANSI, which says that type assertions are put
952   immediately onto argument expressions, but is easier to implement in
953   IR1 and is more compatible to type inference, inline expansion,
954   etc. IR1-transforms automatically keep this policy, but source
955   transforms for associative functions (such as +), being applied
956   during IR1-convertion, do not. It may be tolerable for direct calls
957   (+ x y z), but for (FUNCALL #'+ x y z) it is non-conformant.
959   b. Another aspect of this problem is efficiency. [x y + z +]
960   requires less registers than [x y z + +]. This transformation is
961   currently performed with source transforms, but it would be good to
962   also perform it in IR1 optimization phase.
964 290: Alpha floating point and denormalized traps
965   In SBCL 0.8.3.6x on the alpha, we work around what appears to be a
966   hardware or kernel deficiency: the status of the enable/disable
967   denormalized-float traps bit seems to be ambiguous; by the time we
968   get to os_restore_fp_control after a trap, denormalized traps seem
969   to be enabled.  Since we don't want a trap every time someone uses a
970   denormalized float, in general, we mask out that bit when we restore
971   the control word; however, this clobbers any change the user might
972   have made.
974 296:
975   (reported by Adam Warner, sbcl-devel 2003-09-23)
977   The --load toplevel argument does not perform any sanitization of its
978   argument.  As a result, files with Lisp pathname pattern characters
979   (#\* or #\?, for instance) or quotation marks can cause the system
980   to perform arbitrary behaviour.
982 297:
983   LOOP with non-constant arithmetic step clauses suffers from overzealous
984   type constraint: code of the form 
985     (loop for d of-type double-float from 0d0 to 10d0 by x collect d)
986   compiles to a type restriction on X of (AND DOUBLE-FLOAT (REAL
987   (0))).  However, an integral value of X should be legal, because
988   successive adds of integers to double-floats produces double-floats,
989   so none of the type restrictions in the code is violated.
991 300: (reported by Peter Graves) Function PEEK-CHAR checks PEEK-TYPE
992   argument type only after having read a character. This is caused
993   with EXPLICIT-CHECK attribute in DEFKNOWN. The similar problem
994   exists with =, /=, <, >, <=, >=. They were fixed, but it is probably
995   less error prone to have EXPLICIT-CHECK be a local declaration,
996   being put into the definition, instead of an attribute being kept in
997   a separate file; maybe also put it into SB-EXT?
999 301: ARRAY-SIMPLE-=-TYPE-METHOD breaks on corner cases which can arise
1000      in NOTE-ASSUMED-TYPES
1001   In sbcl-0.8.7.32, compiling the file
1002         (defun foo (x y)
1003           (declare (type integer x))
1004           (declare (type (vector (or hash-table bit)) y))
1005           (bletch 2 y))
1006         (defun bar (x y)
1007           (declare (type integer x))
1008           (declare (type (simple-array base (2)) y))
1009           (bletch 1 y))
1010   gives the error
1011     failed AVER: "(NOT (AND (NOT EQUALP) CERTAINP))"
1013 303: "nonlinear LVARs" (aka MISC.293)
1014     (defun buu (x)
1015       (multiple-value-call #'list
1016         (block foo
1017           (multiple-value-prog1
1018               (eval '(values :a :b :c))
1019             (catch 'bar
1020               (if (> x 0)
1021                   (return-from foo
1022                     (eval `(if (> ,x 1)
1023                                1
1024                                (throw 'bar (values 3 4)))))))))))
1026   (BUU 1) returns garbage.
1028   The problem is that both EVALs sequentially write to the same LVAR.
1030 306: "Imprecise unions of array types"
1031   a.(defun foo (x)
1032       (declare (optimize speed)
1033                (type (or (array cons) (array vector)) x))
1034       (elt (aref x 0) 0))
1035     (foo #((0))) => TYPE-ERROR
1037   relatedly,
1039   b.(subtypep 
1040      'array
1041      `(or
1042        ,@(loop for x across sb-vm:*specialized-array-element-type-properties*
1043                collect `(array ,(sb-vm:saetp-specifier x)))))
1044     => NIL, T (when it should be T, T)
1046 309: "Dubious values for implementation limits"
1047     (reported by Bruno Haible sbcl-devel "Incorrect value of
1048     multiple-values-limit" 2004-04-19)
1049   (values-list (make-list 1000000)), on x86/linux, signals a stack
1050   exhaustion condition, despite MULTIPLE-VALUES-LIMIT being
1051   significantly larger than 1000000.  There are probably similar
1052   dubious values for CALL-ARGUMENTS-LIMIT (see cmucl-help/cmucl-imp
1053   around the same time regarding a call to LIST on sparc with 1000
1054   arguments) and other implementation limit constants.
1056 314: "LOOP :INITIALLY clauses and scope of initializers"
1057   reported by Bruno Haible sbcl-devel "various SBCL bugs" from CLISP
1058   test suite, originally by Thomas F. Burdick.
1059     ;; <http://www.lisp.org/HyperSpec/Body/sec_6-1-7-2.html>
1060     ;; According to the HyperSpec 6.1.2.1.4, in for-as-equals-then, var is
1061     ;; initialized to the result of evaluating form1.  6.1.7.2 says that
1062     ;; initially clauses are evaluated in the loop prologue, which precedes all
1063     ;; loop code except for the initial settings provided by with, for, or as.
1064     (loop :for x = 0 :then (1+ x) 
1065           :for y = (1+ x) :then (ash y 1)
1066           :for z :across #(1 3 9 27 81 243) 
1067           :for w = (+ x y z)
1068           :initially (assert (zerop x)) :initially (assert (= 2 w))
1069           :until (>= w 100) :collect w)
1070     Expected: (2 6 15 38)
1071     Got:      ERROR
1073 318: "stack overflow in compiler warning with redefined class"
1074   reported by Bruno Haible sbcl-devel "various SBCL bugs" from CLISP
1075   test suite.
1076     (defstruct foo a)
1077     (setf (find-class 'foo) nil)
1078     (defstruct foo slot-1)
1079   This used to give a stack overflow from within the printer, which has
1080   been fixed as of 0.8.16.11. Current result:
1081     ; caught ERROR:
1082     ;   can't compile TYPEP of anonymous or undefined class:
1083     ;     #<SB-KERNEL:STRUCTURE-CLASSOID FOO>
1084     ...
1085     debugger invoked on a TYPE-ERROR in thread 19973:
1086       The value NIL is not of type FUNCTION.
1088   CSR notes: it's not really clear what it should give: is (SETF FIND-CLASS)
1089     meant to be enough to delete structure classes from the system?
1091 319: "backquote with comma inside array"
1092   reported by Bruno Haible sbcl-devel "various SBCL bugs" from CLISP
1093   test suite.
1094     (read-from-string "`#1A(1 2 ,(+ 2 2) 4)") 
1095   gives
1096     #(1 2 ((SB-IMPL::|,|) + 2 2) 4)
1097   which probably isn't intentional.
1099 324: "STREAMs and :ELEMENT-TYPE with large bytesize"
1100   In theory, (open foo :element-type '(unsigned-byte <x>)) should work
1101   for all positive integral <x>.  At present, it only works for <x> up
1102   to about 1024 (and similarly for signed-byte), so
1103     (open "/dev/zero" :element-type '(unsigned-byte 1025))
1104   gives an error in sbcl-0.8.10.
1106 325: "CLOSE :ABORT T on supeseding streams"
1107   Closing a stream opened with :IF-EXISTS :SUPERSEDE with :ABORT T leaves no
1108   file on disk, even if one existed before opening.
1110   The illegality of this is not crystal clear, as the ANSI dictionary
1111   entry for CLOSE says that when :ABORT is T superseded files are not
1112   superseded (ie. the original should be restored), whereas the OPEN
1113   entry says about :IF-EXISTS :SUPERSEDE "If possible, the
1114   implementation should not destroy the old file until the new stream
1115   is closed." -- implying that even though undesirable, early deletion
1116   is legal. Restoring the original would none the less be the polite
1117   thing to do.
1119 326: "*PRINT-CIRCLE* crosstalk between streams"
1120   In sbcl-0.8.10.48 it's possible for *PRINT-CIRCLE* references to be
1121   mixed between streams when output operations are intermingled closely
1122   enough (as by doing output on S2 from within (PRINT-OBJECT X S1) in the
1123   test case below), so that e.g. the references #2# appears on a stream
1124   with no preceding #2= on that stream to define it (because the #2= was
1125   sent to another stream).
1126     (cl:in-package :cl-user)
1127     (defstruct foo index)
1128     (defparameter *foo* (make-foo :index 4))
1129     (defstruct bar)
1130     (defparameter *bar* (make-bar))
1131     (defparameter *tangle* (list *foo* *bar* *foo*))
1132     (defmethod print-object ((foo foo) stream)
1133       (let ((index (foo-index foo)))
1134         (format *trace-output*
1135             "~&-$- emitting FOO ~D, ambient *BAR*=~S~%"
1136             index *bar*)
1137         (format stream "[FOO ~D]" index))
1138       foo)
1139     (let ((tsos (make-string-output-stream))
1140           (ssos (make-string-output-stream)))
1141       (let ((*print-circle* t)
1142             (*trace-output* tsos)
1143             (*standard-output* ssos))
1144         (prin1 *tangle* *standard-output*))
1145       (let ((string (get-output-stream-string ssos)))
1146         (unless (string= string "(#1=[FOO 4] #S(BAR) #1#)")
1147           ;; In sbcl-0.8.10.48 STRING was "(#1=[FOO 4] #2# #1#)".:-(
1148           (error "oops: ~S" string)))))
1149   It might be straightforward to fix this by turning the
1150   *CIRCULARITY-HASH-TABLE* and *CIRCULARITY-COUNTER* variables into
1151   per-stream slots, but (1) it would probably be sort of messy faking
1152   up the special variable binding semantics using UNWIND-PROTECT and
1153   (2) it might be sort of a pain to test that no other bugs had been
1154   introduced. 
1156 328: "Profiling generic functions", transplanted from #241
1157   (from tonyms on #lisp IRC 2003-02-25)
1158   In sbcl-0.7.12.55, typing
1159     (defclass foo () ((bar :accessor foo-bar)))
1160     (profile foo-bar)
1161     (unintern 'foo-bar)
1162     (defclass foo () ((bar :accessor foo-bar)))
1163   gives the error message
1164     "#:FOO-BAR already names an ordinary function or a macro."
1166   Problem: when a generic function is profiled, it appears as an ordinary
1167   function to PCL. (Remembering the uninterned accessor is OK, as the
1168   redefinition must be able to remove old accessors from their generic
1169   functions.)
1171 329: "Sequential class redefinition"
1172   reported by Bruno Haible:
1173    (defclass reactor () ((max-temp :initform 10000000)))
1174    (defvar *r1* (make-instance 'reactor))
1175    (defvar *r2* (make-instance 'reactor))
1176    (slot-value *r1* 'max-temp)
1177    (slot-value *r2* 'max-temp)
1178    (defclass reactor () ((uptime :initform 0)))
1179    (slot-value *r1* 'uptime)
1180    (defclass reactor () ((uptime :initform 0) (max-temp :initform 10000)))
1181    (slot-value *r1* 'max-temp) ; => 10000
1182    (slot-value *r2* 'max-temp) ; => 10000000 oops...
1184   Possible solution: 
1185    The method effective when the wrapper is obsoleted can be saved
1186      in the wrapper, and then to update the instance just run through
1187      all the old wrappers in order from oldest to newest.
1189 332: "fasl stack inconsistency in structure redefinition"
1190   (reported by Tim Daly Jr sbcl-devel 2004-05-06)
1191   Even though structure redefinition is undefined by the standard, the
1192   following behaviour is suboptimal: running
1193     (defun stimulate-sbcl ()
1194       (let ((filename (format nil "/tmp/~A.lisp" (gensym))))
1195         ;;create a file which redefines a structure incompatibly
1196         (with-open-file (f filename :direction :output :if-exists :supersede)
1197           (print '(defstruct astruct foo) f)
1198           (print '(defstruct astruct foo bar) f))
1199         ;;compile and load the file, then invoke the continue restart on
1200         ;;the structure redefinition error
1201         (handler-bind ((error (lambda (c) (continue c))))
1202           (load (compile-file filename)))))
1203     (stimulate-sbcl)
1204   and choosing the CONTINUE restart yields the message
1205     debugger invoked on a SB-INT:BUG in thread 27726:
1206       fasl stack not empty when it should be
1208 336: "slot-definitions must retain the generic functions of accessors"
1209   reported by Tony Martinez:
1210     (defclass foo () ((bar :reader foo-bar)))
1211     (defun foo-bar (x) x)
1212     (defclass foo () ((bar :reader get-bar))) ; => error, should work
1214   Note: just punting the accessor removal if the fdefinition
1215   is not a generic function is not enough:
1217    (defclass foo () ((bar :reader foo-bar)))
1218    (defvar *reader* #'foo-bar)
1219    (defun foo-bar (x) x)
1220    (defclass foo () ((bar :initform 'ok :reader get-bar)))
1221    (funcall *reader* (make-instance 'foo)) ; should be an error, since
1222                                            ; the method must be removed
1223                                            ; by the class redefinition
1225   Fixing this should also fix a subset of #328 -- update the
1226   description with a new test-case then.
1228 337: MAKE-METHOD and user-defined method classes
1229   (reported by Bruno Haible sbcl-devel 2004-06-11)
1231   In the presence of  
1233 (defclass user-method (standard-method) (myslot))
1234 (defmacro def-user-method (name &rest rest)
1235   (let* ((lambdalist-position (position-if #'listp rest))
1236          (qualifiers (subseq rest 0 lambdalist-position))
1237          (lambdalist (elt rest lambdalist-position))
1238          (body (subseq rest (+ lambdalist-position 1)))
1239          (required-part 
1240           (subseq lambdalist 0 (or 
1241                                 (position-if 
1242                                  (lambda (x) (member x lambda-list-keywords))
1243                                  lambdalist)
1244                                 (length lambdalist))))
1245          (specializers (mapcar #'find-class 
1246                                (mapcar (lambda (x) (if (consp x) (second x) t))
1247                                        required-part)))
1248          (unspecialized-required-part 
1249           (mapcar (lambda (x) (if (consp x) (first x) x)) required-part))
1250          (unspecialized-lambdalist 
1251           (append unspecialized-required-part 
1252            (subseq lambdalist (length required-part)))))
1253     `(PROGN
1254        (ADD-METHOD #',name
1255          (MAKE-INSTANCE 'USER-METHOD
1256           :QUALIFIERS ',qualifiers
1257           :LAMBDA-LIST ',unspecialized-lambdalist
1258           :SPECIALIZERS ',specializers
1259           :FUNCTION
1260           (LAMBDA (ARGUMENTS NEXT-METHODS-LIST)
1261             (FLET ((NEXT-METHOD-P () NEXT-METHODS-LIST)
1262                    (CALL-NEXT-METHOD (&REST NEW-ARGUMENTS)
1263                      (UNLESS NEW-ARGUMENTS (SETQ NEW-ARGUMENTS ARGUMENTS))
1264                      (IF (NULL NEXT-METHODS-LIST)
1265                          (ERROR "no next method for arguments ~:S" ARGUMENTS)
1266                          (FUNCALL (SB-PCL:METHOD-FUNCTION 
1267                                    (FIRST NEXT-METHODS-LIST))
1268                                   NEW-ARGUMENTS (REST NEXT-METHODS-LIST)))))
1269               (APPLY #'(LAMBDA ,unspecialized-lambdalist ,@body) ARGUMENTS)))))
1270        ',name)))
1272   (progn
1273     (defgeneric test-um03 (x))
1274     (defmethod test-um03 ((x integer))
1275       (list* 'integer x (not (null (next-method-p))) (call-next-method)))
1276     (def-user-method test-um03 ((x rational))
1277       (list* 'rational x (not (null (next-method-p))) (call-next-method)))
1278     (defmethod test-um03 ((x real))
1279       (list 'real x (not (null (next-method-p)))))
1280     (test-um03 17))
1281   works, but
1283   a.(progn
1284       (defgeneric test-um10 (x))
1285       (defmethod test-um10 ((x integer))
1286         (list* 'integer x (not (null (next-method-p))) (call-next-method)))
1287       (defmethod test-um10 ((x rational))
1288         (list* 'rational x (not (null (next-method-p))) (call-next-method)))
1289       (defmethod test-um10 ((x real))
1290         (list 'real x (not (null (next-method-p)))))
1291       (defmethod test-um10 :after ((x real)))
1292       (def-user-method test-um10 :around ((x integer))
1293         (list* 'around-integer x 
1294          (not (null (next-method-p))) (call-next-method)))
1295       (defmethod test-um10 :around ((x rational))
1296         (list* 'around-rational x 
1297          (not (null (next-method-p))) (call-next-method)))
1298       (defmethod test-um10 :around ((x real))
1299         (list* 'around-real x (not (null (next-method-p))) (call-next-method)))
1300       (test-um10 17))
1301     fails with a type error, and
1303   b.(progn
1304       (defgeneric test-um12 (x))
1305       (defmethod test-um12 ((x integer))
1306         (list* 'integer x (not (null (next-method-p))) (call-next-method)))
1307       (defmethod test-um12 ((x rational))
1308         (list* 'rational x (not (null (next-method-p))) (call-next-method)))
1309       (defmethod test-um12 ((x real))
1310         (list 'real x (not (null (next-method-p)))))
1311       (defmethod test-um12 :after ((x real)))
1312       (defmethod test-um12 :around ((x integer))
1313         (list* 'around-integer x 
1314          (not (null (next-method-p))) (call-next-method)))
1315       (defmethod test-um12 :around ((x rational))
1316         (list* 'around-rational x 
1317          (not (null (next-method-p))) (call-next-method)))
1318       (def-user-method test-um12 :around ((x real))
1319         (list* 'around-real x (not (null (next-method-p))) (call-next-method)))
1320       (test-um12 17))
1321     fails with NO-APPLICABLE-METHOD.
1323 339: "DEFINE-METHOD-COMBINATION bugs"
1324   (reported by Bruno Haible via the clisp test suite)
1326   a. Syntax checking laxity (should produce errors):
1327      i. (define-method-combination foo :documentation :operator)
1328     ii. (define-method-combination foo :documentation nil)
1329    iii. (define-method-combination foo nil)
1330     iv. (define-method-combination foo nil nil
1331          (:arguments order &aux &key))
1332      v. (define-method-combination foo nil nil (:arguments &whole))
1333     vi. (define-method-combination foo nil nil (:generic-function))
1334    vii. (define-method-combination foo nil nil (:generic-function bar baz))
1335   viii. (define-method-combination foo nil nil (:generic-function (bar)))
1336     ix. (define-method-combination foo nil ((3)))
1337      x. (define-method-combination foo nil ((a)))
1339   b. define-method-combination arguments lambda list badness
1340      i. &aux args are currently unsupported;
1341     ii. default values of &optional and &key arguments are ignored;
1342    iii. supplied-p variables for &optional and &key arguments are not
1343         bound.
1345   c. qualifier matching incorrect
1346   (progn
1347     (define-method-combination mc27 () 
1348         ((normal ()) 
1349          (ignored (:ignore :unused)))
1350       `(list 'result 
1351              ,@(mapcar #'(lambda (method) `(call-method ,method)) normal)))
1352     (defgeneric test-mc27 (x)
1353       (:method-combination mc27)
1354       (:method :ignore ((x number)) (/ 0)))
1355     (test-mc27 7))
1357   should signal an invalid-method-error, as the :IGNORE (NUMBER)
1358   method is applicable, and yet matches neither of the method group
1359   qualifier patterns.
1361 343: MOP:COMPUTE-DISCRIMINATING-FUNCTION overriding causes error
1362   Even the simplest possible overriding of
1363   COMPUTE-DISCRIMINATING-FUNCTION, suggested in the PCL implementation
1364   as "canonical", does not work:
1365     (defclass my-generic-function (standard-generic-function) ()
1366       (:metaclass funcallable-standard-class))
1367     (defmethod compute-discriminating-function ((gf my-generic-function))
1368       (let ((dfun (call-next-method)))
1369         (lambda (&rest args)
1370           (apply dfun args))))
1371     (defgeneric foo (x)
1372       (:generic-function-class my-generic-function))
1373     (defmethod foo (x) (+ x x))
1374     (foo 5)
1375   signals an error.  This error is the same even if the LAMBDA is
1376   replaced by (FUNCTION (SB-KERNEL:INSTANCE-LAMBDA ...)).  Maybe the
1377   SET-FUNCALLABLE-INSTANCE-FUN scary stuff in
1378   src/code/target-defstruct.lisp is broken?  This seems to be broken
1379   in CMUCL 18e, so it's not caused by a recent change.
1381 344: more (?) ROOM T problems (possibly part of bug 108)
1382   In sbcl-0.8.12.51, and off and on leading up to it, the
1383   SB!VM:MEMORY-USAGE operations in ROOM T caused 
1384         unhandled condition (of type SB-INT:BUG):
1385             failed AVER: "(SAP= CURRENT END)"
1386   Several clever people have taken a shot at this without fixing
1387   it; this time around (before sbcl-0.8.13 release) I (WHN) just
1388   commented out the SB!VM:MEMORY-USAGE calls until someone figures
1389   out how to make them work reliably with the rest of the GC.
1391   (Note: there's at least one dubious thing in room.lisp: see the
1392   comment in VALID-OBJ)
1394 346: alpha backtrace
1395   In sbcl-0.8.13, all backtraces from errors caused by internal errors
1396   on the alpha seem to have a "bogus stack frame".
1398 349: PPRINT-INDENT rounding implementation decisions
1399   At present, pprint-indent (and indeed the whole pretty printer)
1400   more-or-less assumes that it's using a monospace font.  That's
1401   probably not too silly an assumption, but one piece of information
1402   the current implementation loses is from requests to indent by a
1403   non-integral amount.  As of sbcl-0.8.15.9, the system silently
1404   truncates the indentation to an integer at the point of request, but
1405   maybe the non-integral value should be propagated through the
1406   pprinter and only truncated at output?  (So that indenting by 1/2
1407   then 3/2 would indent by two spaces, not one?)
1409 352: forward-referenced-class trouble
1410  reported by Bruno Haible on sbcl-devel
1411    (defclass c (a) ())
1412    (setf (class-name (find-class 'a)) 'b)
1413    (defclass a () (x))
1414    (defclass b () (y))
1415    (make-instance 'c)
1416  Expected: an instance of c, with a slot named x
1417  Got: debugger invoked on a SIMPLE-ERROR in thread 78906:
1418         While computing the class precedence list of the class named C.
1419         The class named B is a forward referenced class.
1420         The class named B is a direct superclass of the class named C.
1422 353: debugger suboptimalities on x86
1423  On x86 backtraces for undefined functions start with a bogus stack
1424  frame, and  backtraces for throws to unknown catch tags with a "no 
1425  debug information" frame. These are both due to CODE-COMPONENT-FROM-BITS
1426  (used on non-x86 platforms) being a more complete solution then what
1427  is done on x86.
1429  On x86/linux large portions of tests/debug.impure.lisp have been commented
1430  out as failures. The probable culprit for these problems is in x86-call-context
1431  (things work fine on x86/freebsd).
1433  More generally, the debugger internals suffer from excessive x86/non-x86
1434  conditionalization and OAOOMization: refactoring the common parts would
1435  be good.
1437 354: XEPs in backtraces
1438  Under default compilation policy
1439    (defun test ()
1440      (throw :unknown t))
1441    (test)
1442  Has the XEP for TEST in the backtrace, not the TEST frame itself.
1443  (sparc and x86 at least)
1445  Since SBCL 0.8.20.1 this is hidden unless *SHOW-ENTRY-POINT-DETAILS*
1446  is true (instead there appear two TEST frames at least on ppc). The
1447  underlying cause seems to be that SB-C::TAIL-ANNOTATE will not merge
1448  the tail-call for the XEP, since Python has by that time proved that
1449  the function can never return; same happens if the function holds an
1450  unconditional call to ERROR.
1452 355: change-class of generic-function
1453     (reported by Bruno Haible)
1454   The MOP doesn't support change-class on a generic-function. However, SBCL
1455   apparently supports it, since it doesn't give an error or warning when doing
1456   so so. Then, however, it produces wrong results for calls to this generic 
1457   function.
1458   ;;; The effective-methods cache:
1459   (progn
1460     (defgeneric testgf35 (x))
1461     (defmethod testgf35 ((x integer))
1462       (cons 'integer (if (next-method-p) (call-next-method))))
1463     (defmethod testgf35 ((x real))
1464       (cons 'real (if (next-method-p) (call-next-method))))
1465     (defclass customized5-generic-function (standard-generic-function)
1466       ()
1467       (:metaclass sb-pcl:funcallable-standard-class))
1468     (defmethod sb-pcl:compute-effective-method ((gf customized5-generic-function) method-combination methods)
1469       `(REVERSE ,(call-next-method)))
1470     (list
1471       (testgf35 3)
1472       (progn
1473         (change-class #'testgf35 'customized5-generic-function)
1474         (testgf35 3))))
1475   Expected: ((INTEGER REAL) (REAL INTEGER))
1476   Got:      ((INTEGER REAL) (INTEGER REAL))
1477   ;;; The discriminating-function cache:
1478   (progn
1479     (defgeneric testgf36 (x))
1480     (defmethod testgf36 ((x integer))
1481       (cons 'integer (if (next-method-p) (call-next-method))))
1482     (defmethod testgf36 ((x real))
1483       (cons 'real (if (next-method-p) (call-next-method))))
1484     (defclass customized6-generic-function (standard-generic-function)
1485       ()
1486       (:metaclass sb-pcl:funcallable-standard-class))
1487     (defmethod sb-pcl:compute-discriminating-function ((gf customized6-generic-function))
1488       (let ((orig-df (call-next-method)))
1489         #'(lambda (&rest arguments)
1490             (reverse (apply orig-df arguments)))))
1491     (list
1492       (testgf36 3)
1493       (progn
1494         (change-class #'testgf36 'customized6-generic-function)
1495         (testgf36 3))))
1496   Expected: ((INTEGER REAL) (REAL INTEGER))
1497   Got:      ((INTEGER REAL) (INTEGER REAL))
1499 356: PCL corruption
1500     (reported by Bruno Haible)
1501   After the "layout depth conflict" error, the CLOS is left in a state where
1502   it's not possible to define new standard-class subclasses any more.
1503   Test case:
1504   (defclass prioritized-dispatcher ()
1505     ((dependents :type list :initform nil)))
1506   (defmethod sb-pcl:validate-superclass ((c1 sb-pcl:funcallable-standard-class) 
1507                                          (c2 (eql (find-class 'prioritized-dispatcher))))
1508     t)
1509   (defclass prioritized-generic-function (prioritized-dispatcher standard-generic-function)
1510     ()
1511     (:metaclass sb-pcl:funcallable-standard-class))
1512   ;; ERROR, Quit the debugger with ABORT
1513   (defclass typechecking-reader-class (standard-class)
1514     ())
1515   Expected: #<STANDARD-CLASS TYPECHECKING-READER-CLASS>
1516   Got:      ERROR "The assertion SB-PCL::WRAPPERS failed."
1518 357: defstruct inheritance of initforms
1519     (reported by Bruno Haible)
1520   When defstruct and defclass (with :metaclass structure-class) are mixed,
1521   1. some slot initforms are ignored by the DEFSTRUCT generated constructor
1522      function, and 
1523   2. all slot initforms are ignored by MAKE-INSTANCE. (This can be arguably
1524      OK for initforms that were given in a DEFSTRUCT form, but for those
1525      given in a DEFCLASS form, I think it qualifies as a bug.)
1526   Test case:
1527   (defstruct structure02a
1528     slot1
1529     (slot2 t)
1530     (slot3 (floor pi)))
1531   (defclass structure02b (structure02a)
1532     ((slot4 :initform -44)
1533      (slot5)
1534      (slot6 :initform t)
1535      (slot7 :initform (floor (* pi pi)))
1536      (slot8 :initform 88))
1537     (:metaclass structure-class))
1538   (defstruct (structure02c (:include structure02b (slot8 -88)))
1539     slot9 
1540     (slot10 t)
1541     (slot11 (floor (exp 3))))
1542   ;; 1. Form:
1543   (let ((a (make-structure02c)))
1544     (list (structure02c-slot4 a)
1545           (structure02c-slot5 a)
1546           (structure02c-slot6 a)
1547           (structure02c-slot7 a)))
1548   Expected: (-44 nil t 9)
1549   Got: (SB-PCL::..SLOT-UNBOUND.. SB-PCL::..SLOT-UNBOUND..
1550         SB-PCL::..SLOT-UNBOUND.. SB-PCL::..SLOT-UNBOUND..)
1551   ;; 2. Form:
1552   (let ((b (make-instance 'structure02c)))
1553     (list (structure02c-slot2 b)
1554           (structure02c-slot3 b)
1555           (structure02c-slot4 b)
1556           (structure02c-slot6 b)
1557           (structure02c-slot7 b)
1558           (structure02c-slot8 b)
1559           (structure02c-slot10 b)
1560           (structure02c-slot11 b)))
1561   Expected: (t 3 -44 t 9 -88 t 20)
1562   Got: (0 0 0 0 0 0 0 0)
1564 358: :DECLARE argument to ENSURE-GENERIC-FUNCTION 
1565     (reported by Bruno Haible)
1566   According to ANSI CL, ensure-generic-function must accept a :DECLARE
1567   keyword argument. In SBCL 0.8.16 it does not.
1568   Test case:
1569   (progn
1570     (ensure-generic-function 'foo113 :declare '((optimize (speed 3))))
1571     (sb-pcl:generic-function-declarations #'foo113))
1572   Expected: ((OPTIMIZE (SPEED 3)))
1573   Got: ERROR
1574     Invalid initialization argument:
1575       :DECLARE
1576     in call for class #<SB-MOP:FUNCALLABLE-STANDARD-CLASS STANDARD-GENERIC-FUNCTION>.
1577   See also:
1578     The ANSI Standard, Section 7.1.2
1580   Bruno notes: The MOP specifies that ensure-generic-function accepts :DECLARATIONS.
1581   The easiest way to be compliant to both specs is to accept both (exclusively
1582   or cumulatively).
1584 359: wrong default value for ensure-generic-function's :generic-function-class argument
1585     (reported by Bruno Haible)
1586   ANSI CL is silent on this, but the MOP's specification of ENSURE-GENERIC-FUNCTION says:
1587    "The remaining arguments are the complete set of keyword arguments
1588     received by ENSURE-GENERIC-FUNCTION."
1589   and the spec of ENSURE-GENERIC-FUNCTION-USING-CLASS:
1590    ":GENERIC-FUNCTION-CLASS - a class metaobject or a class name. If it is not
1591     supplied, it defaults to the class named STANDARD-GENERIC-FUNCTION."
1592   This is not the case in SBCL. Test case:
1593    (defclass my-generic-function (standard-generic-function)
1594      ()
1595      (:metaclass sb-pcl:funcallable-standard-class))
1596    (setf (fdefinition 'foo1)
1597          (make-instance 'my-generic-function :name 'foo1))
1598    (ensure-generic-function 'foo1
1599      :generic-function-class (find-class 'standard-generic-function))
1600    (class-of #'foo1)
1601    ; => #<SB-MOP:FUNCALLABLE-STANDARD-CLASS STANDARD-GENERIC-FUNCTION>
1602    (setf (fdefinition 'foo2)
1603          (make-instance 'my-generic-function :name 'foo2))
1604    (ensure-generic-function 'foo2)
1605    (class-of #'foo2)
1606   Expected: #<SB-MOP:FUNCALLABLE-STANDARD-CLASS STANDARD-GENERIC-FUNCTION>
1607   Got:      #<SB-MOP:FUNCALLABLE-STANDARD-CLASS MY-GENERIC-FUNCTION>
1609 360: CALL-METHOD not recognized in method-combination body
1610   (reported by Bruno Haible)
1611   This method combination, which adds 'redo' and 'return' restarts for each
1612   method invocation to standard method combination, gives an error in SBCL.
1613   (defun prompt-for-new-values ()
1614     (format *debug-io* "~&New values: ")
1615     (list (read *debug-io*)))
1616   (defun add-method-restarts (form method)
1617     (let ((block (gensym))
1618           (tag (gensym)))
1619       `(BLOCK ,block
1620          (TAGBODY
1621            ,tag
1622            (RETURN-FROM ,block
1623              (RESTART-CASE ,form
1624                (METHOD-REDO ()
1625                  :REPORT (LAMBDA (STREAM) (FORMAT STREAM "Try calling ~S again." ,method))
1626                  (GO ,tag))
1627                (METHOD-RETURN (L)
1628                  :REPORT (LAMBDA (STREAM) (FORMAT STREAM "Specify return values for ~S call." ,method))
1629                  :INTERACTIVE (LAMBDA () (PROMPT-FOR-NEW-VALUES))
1630                  (RETURN-FROM ,block (VALUES-LIST L)))))))))
1631   (defun convert-effective-method (efm)
1632     (if (consp efm)
1633       (if (eq (car efm) 'CALL-METHOD)
1634         (let ((method-list (third efm)))
1635           (if (or (typep (first method-list) 'method) (rest method-list))
1636             ; Reduce the case of multiple methods to a single one.
1637             ; Make the call to the next-method explicit.
1638             (convert-effective-method
1639               `(CALL-METHOD ,(second efm)
1640                  ((MAKE-METHOD
1641                     (CALL-METHOD ,(first method-list) ,(rest method-list))))))
1642             ; Now the case of at most one method.
1643             (if (typep (second efm) 'method)
1644               ; Wrap the method call in a RESTART-CASE.
1645               (add-method-restarts
1646                 (cons (convert-effective-method (car efm))
1647                       (convert-effective-method (cdr efm)))
1648                 (second efm))
1649               ; Normal recursive processing.
1650               (cons (convert-effective-method (car efm))
1651                     (convert-effective-method (cdr efm))))))
1652         (cons (convert-effective-method (car efm))
1653               (convert-effective-method (cdr efm))))
1654       efm))
1655   (define-method-combination standard-with-restarts ()
1656          ((around (:around))
1657           (before (:before))
1658           (primary () :required t)
1659           (after (:after)))
1660     (flet ((call-methods-sequentially (methods)
1661              (mapcar #'(lambda (method)
1662                          `(CALL-METHOD ,method))
1663                      methods)))
1664       (let ((form (if (or before after (rest primary))
1665                     `(MULTIPLE-VALUE-PROG1
1666                        (PROGN
1667                          ,@(call-methods-sequentially before)
1668                          (CALL-METHOD ,(first primary) ,(rest primary)))
1669                        ,@(call-methods-sequentially (reverse after)))
1670                     `(CALL-METHOD ,(first primary)))))
1671         (when around
1672           (setq form
1673                 `(CALL-METHOD ,(first around)
1674                               (,@(rest around) (MAKE-METHOD ,form)))))
1675         (convert-effective-method form))))
1676   (defgeneric testgf16 (x) (:method-combination standard-with-restarts))
1677   (defclass testclass16a () ())
1678   (defclass testclass16b (testclass16a) ())
1679   (defclass testclass16c (testclass16a) ())
1680   (defclass testclass16d (testclass16b testclass16c) ())
1681   (defmethod testgf16 ((x testclass16a))
1682     (list 'a
1683           (not (null (find-restart 'method-redo)))
1684           (not (null (find-restart 'method-return)))))
1685   (defmethod testgf16 ((x testclass16b))
1686     (cons 'b (call-next-method)))
1687   (defmethod testgf16 ((x testclass16c))
1688     (cons 'c (call-next-method)))
1689   (defmethod testgf16 ((x testclass16d))
1690     (cons 'd (call-next-method)))
1691   (testgf16 (make-instance 'testclass16d))
1693   Expected: (D B C A T T)
1694   Got: ERROR CALL-METHOD outside of a effective method form
1696   This is a bug because ANSI CL HyperSpec/Body/locmac_call-m__make-method
1697   says
1698    "The macro call-method invokes the specified method, supplying it with
1699     arguments and with definitions for call-next-method and for next-method-p.
1700     If the invocation of call-method is lexically inside of a make-method,
1701     the arguments are those that were supplied to that method. Otherwise
1702     the arguments are those that were supplied to the generic function."
1703   and the example uses nothing more than these two cases (as you can see by
1704   doing (trace convert-effective-method)).
1706 361: initialize-instance of standard-reader-method ignores :function argument
1707     (reported by Bruno Haible)
1708   Pass a custom :function argument to initialize-instance of a
1709   standard-reader-method instance, but it has no effect.
1710   ;; Check that it's possible to define reader methods that do typechecking.
1711   (progn
1712     (defclass typechecking-reader-method (sb-pcl:standard-reader-method)
1713       ())
1714     (defmethod initialize-instance ((method typechecking-reader-method) &rest initargs
1715                                     &key slot-definition)
1716       (let ((name (sb-pcl:slot-definition-name slot-definition))
1717             (type (sb-pcl:slot-definition-type slot-definition)))
1718         (apply #'call-next-method method
1719                :function #'(lambda (args next-methods)
1720                              (declare (ignore next-methods))
1721                              (apply #'(lambda (instance)
1722                                         (let ((value (slot-value instance name)))
1723                                           (unless (typep value type)
1724                                             (error "Slot ~S of ~S is not of type ~S: ~S"
1725                                                    name instance type value))
1726                                           value))
1727                                     args))
1728                initargs)))
1729     (defclass typechecking-reader-class (standard-class)
1730       ())
1731     (defmethod sb-pcl:validate-superclass ((c1 typechecking-reader-class) (c2 standard-class))
1732       t)
1733     (defmethod reader-method-class ((class typechecking-reader-class) direct-slot &rest args)
1734       (find-class 'typechecking-reader-method))
1735     (defclass testclass25 ()
1736       ((pair :type (cons symbol (cons symbol null)) :initarg :pair :accessor testclass25-pair))
1737       (:metaclass typechecking-reader-class))
1738    (macrolet ((succeeds (form)
1739                  `(not (nth-value 1 (ignore-errors ,form)))))
1740       (let ((p (list 'abc 'def))
1741             (x (make-instance 'testclass25)))
1742         (list (succeeds (make-instance 'testclass25 :pair '(seventeen 17)))
1743               (succeeds (setf (testclass25-pair x) p))
1744               (succeeds (setf (second p) 456))
1745               (succeeds (testclass25-pair x))
1746               (succeeds (slot-value x 'pair))))))
1747   Expected: (t t t nil t)
1748   Got:      (t t t t t)
1750   (inspect (first (sb-pcl:generic-function-methods #'testclass25-pair)))
1751   shows that the method was created with a FAST-FUNCTION slot but with a
1752   FUNCTION slot of NIL.
1754 362: missing error when a slot-definition is created without a name
1755     (reported by Bruno Haible)
1756   The MOP says about slot-definition initialization:
1757   "The :NAME argument is a slot name. An ERROR is SIGNALled if this argument
1758    is not a symbol which can be used as a variable name. An ERROR is SIGNALled
1759    if this argument is not supplied."
1760   Test case:
1761    (make-instance (find-class 'sb-pcl:standard-direct-slot-definition))
1762   Expected: ERROR
1763   Got: #<SB-MOP:STANDARD-DIRECT-SLOT-DEFINITION NIL>
1765 363: missing error when a slot-definition is created with a wrong documentation object
1766     (reported by Bruno Haible)
1767   The MOP says about slot-definition initialization:
1768   "The :DOCUMENTATION argument is a STRING or NIL. An ERROR is SIGNALled
1769    if it is not. This argument default to NIL during initialization."
1770   Test case:
1771    (make-instance (find-class 'sb-pcl:standard-direct-slot-definition)
1772                   :name 'foo
1773                   :documentation 'not-a-string)
1774   Expected: ERROR
1775   Got: #<SB-MOP:STANDARD-DIRECT-SLOT-DEFINITION FOO>
1777 364: does not support class objects as specializer names
1778    (reported by Bruno Haible)
1779   According to ANSI CL 7.6.2, class objects are valid specializer names,
1780   and "Parameter specializer names are used in macros intended as the
1781   user-level interface (defmethod)". DEFMETHOD's syntax section doesn't
1782   mention this possibility in the BNF for parameter-specializer-name;
1783   however, this appears to be an editorial omission, since the CLHS
1784   mentions issue CLASS-OBJECT-SPECIALIZER:AFFIRM as being approved
1785   by X3J13. SBCL doesn't support it:
1786    (defclass foo () ())
1787    (defmethod goo ((x #.(find-class 'foo))) x)
1788   Expected: #<STANDARD-METHOD GOO (#<STANDARD-CLASS FOO>)>
1789   Got:      ERROR "#<STANDARD-CLASS FOO> is not a legal class name."
1791 365: mixin on generic-function subclass
1792     (reported by Bruno Haible)
1793   a mixin class
1794     (defclass prioritized-dispatcher ()
1795       ((dependents :type list :initform nil)))
1796   on a generic-function subclass:
1797     (defclass prioritized-generic-function (prioritized-dispatcher standard-generic-function)
1798       ()
1799       (:metaclass sb-pcl:funcallable-standard-class))
1800   SBCL gives an error on this, telling to define a method on SB-MOP:VALIDATE-SUPERCLASS. If done,
1801     (defmethod sb-pcl:validate-superclass ((c1 sb-pcl:funcallable-standard-class) 
1802                                            (c2 (eql (find-class 'prioritized-dispatcher))))
1803       t)
1804   then, however,
1805     (defclass prioritized-generic-function (prioritized-dispatcher standard-generic-function)
1806       ()
1807       (:metaclass sb-pcl:funcallable-standard-class))
1808   => debugger invoked on a SIMPLE-ERROR in thread 6687:
1809     layout depth conflict: #(#<SB-KERNEL:LAYOUT for T {500E1E9}> ...)
1811   Further discussion on this: http://thread.gmane.org/gmane.lisp.steel-bank.general/491
1813 366: cannot define two generic functions with user-defined class
1814    (reported by Bruno Haible)
1815   it is possible to define one generic function class and an instance
1816   of it. But attempting to do the same thing again, in the same session,
1817   leads to a "Control stack exhausted" error. Test case:
1818     (defclass my-generic-function-1 (standard-generic-function)
1819       ()
1820       (:metaclass sb-pcl:funcallable-standard-class))
1821     (defgeneric testgf-1 (x) (:generic-function-class my-generic-function-1)
1822       (:method ((x integer)) (cons 'integer nil)))
1823     (defclass my-generic-function-2 (standard-generic-function)
1824       ()
1825       (:metaclass sb-pcl:funcallable-standard-class))
1826     (defgeneric testgf-2 (x) (:generic-function-class my-generic-function-2)
1827       (:method ((x integer)) (cons 'integer nil)))
1828   => SB-KERNEL::CONTROL-STACK-EXHAUSTED
1830 367: TYPE-ERROR at compile time, undetected TYPE-ERROR at runtime
1831   This test program
1832     (declaim (optimize (safety 3) (debug 2) (speed 2) (space 1)))
1833     (defstruct e367)
1834     (defstruct i367)
1835     (defstruct g367
1836       (i367s (make-array 0 :fill-pointer t) :type (or (vector i367) null)))
1837     (defstruct s367
1838       (g367 (error "missing :G367") :type g367 :read-only t))
1839     ;;; In sbcl-0.8.18, commenting out this (DECLAIM (FTYPE ... R367))
1840     ;;; gives an internal error at compile time:
1841     ;;;    The value #<SB-KERNEL:NAMED-TYPE NIL> is not of
1842     ;;;    type SB-KERNEL:VALUES-TYPE.
1843     (declaim (ftype (function ((vector i367) e367) (or s367 null)) r367))
1844     (declaim (ftype (function ((vector e367)) (values)) h367))
1845     (defun frob (v w)
1846       (let ((x (g367-i367s (make-g367))))
1847         (let* ((y (or (r367 x w)
1848                       (h367 x)))
1849                (z (s367-g367 y)))
1850           (format t "~&Y=~S Z=~S~%" y z)
1851           (g367-i367s z))))
1852     (defun r367 (x y) (declare (ignore x y)) nil)
1853     (defun h367 (x) (declare (ignore x)) (values))
1854     ;;; In sbcl-0.8.18, executing this form causes an low-level error
1855     ;;;   segmentation violation at #X9B0E1F4
1856     ;;; (instead of the TYPE-ERROR that one might like).
1857     (frob 0 (make-e367))
1858   can be made to cause two different problems, as noted in the comments:
1859     bug 367a: Compile and load the file. No TYPE-ERROR is signalled at 
1860       run time (in the (S367-G367 Y) form of FROB, when Y is NIL 
1861       instead of an instance of S367). Instead (on x86/Linux at least)
1862       we end up with a segfault.
1863     bug 367b: Comment out the (DECLAIM (FTYPE ... R367)), and compile 
1864       the file. The compiler fails with TYPE-ERROR at compile time.
1866 368: miscompiled OR (perhaps related to bug 367)
1867   Trying to relax type declarations to find a workaround for bug 367,
1868   it turns out that even when the return type isn't declared (or 
1869   declared to be T, anyway) the system remains confused about type 
1870   inference in code similar to that for bug 367:
1871     (in-package :cl-user)
1872     (declaim (optimize (safety 3) (debug 2) (speed 2) (space 1)))
1873     (defstruct e368)
1874     (defstruct i368)
1875     (defstruct g368
1876       (i368s (make-array 0 :fill-pointer t) :type (or (vector i368) null)))
1877     (defstruct s368
1878       (g368 (error "missing :G368") :type g368 :read-only t))
1879     (declaim (ftype (function (fixnum (vector i368) e368) t) r368))
1880     (declaim (ftype (function (fixnum (vector e368)) t) h368))
1881     (defparameter *h368-was-called-p* nil)
1882     (defun nsu (vertices e368)
1883       (let ((i368s (g368-i368s (make-g368))))
1884         (let ((fuis (r368 0 i368s e368)))
1885           (format t "~&FUIS=~S~%" fuis)
1886           (or fuis (h368 0 i368s)))))
1887     (defun r368 (w x y)
1888       (declare (ignore w x y))
1889       nil)
1890     (defun h368 (w x)
1891       (declare (ignore w x))
1892       (setf *h368-was-called-p* t)
1893       (make-s368 :g368 (make-g368)))
1894     (trace r368 h368)
1895     (format t "~&calling NSU~%")
1896     (let ((nsu (nsu #() (make-e368))))
1897       (format t "~&NSU returned ~S~%" nsu)
1898       (format t "~&*H368-WAS-CALLED-P*=~S~%" *h368-was-called-p*)
1899       (assert (s368-p nsu))
1900       (assert *h368-was-called-p*))
1901   In sbcl-0.8.18, both ASSERTs fail, and (DISASSEMBLE 'NSU) shows
1902   that no call to H368 is compiled.
1904 369: unlike-an-intersection behavior of VALUES-TYPE-INTERSECTION
1905   In sbcl-0.8.18.2, the identity $(x \cap y \cap y)=(x \cap y)$ 
1906   does not hold for VALUES-TYPE-INTERSECTION, even for types which
1907   can be intersected exactly, so that ASSERTs fail in this test case:
1908     (in-package :cl-user)
1909     (let ((types (mapcar #'sb-c::values-specifier-type 
1910                          '((values (vector package) &optional)
1911                            (values (vector package) &rest t)
1912                            (values (vector hash-table) &rest t)
1913                            (values (vector hash-table) &optional)
1914                            (values t &optional)
1915                            (values t &rest t)
1916                            (values nil &optional)
1917                            (values nil &rest t)
1918                            (values sequence &optional)
1919                            (values sequence &rest t)
1920                            (values list &optional)
1921                            (values list &rest t)))))
1922        (dolist (x types)
1923          (dolist (y types)
1924            (let ((i (sb-c::values-type-intersection x y)))
1925              (assert (sb-c::type= i (sb-c::values-type-intersection i x)))
1926              (assert (sb-c::type= i (sb-c::values-type-intersection i y)))))))
1928 370: reader misbehaviour on large-exponent floats
1929     (read-from-string "1.0s1000000000000000000000000000000000000000")
1930   causes the reader to attempt to create a very large bignum (which it
1931   will then attempt to coerce to a rational).  While this isn't
1932   completely wrong, it is probably not ideal -- checking the floating
1933   point control word state and then returning the relevant float
1934   (most-positive-short-float or short-float-infinity) or signalling an
1935   error immediately would seem to make more sense.
1937 372: floating-point overflow not signalled on ppc/darwin
1938  The following assertions in float.pure.lisp fail on ppc/darwin 
1939  (Mac OS X version 10.3.7):
1940    (assert (raises-error? (scale-float 1.0 most-positive-fixnum)
1941                           floating-point-overflow))
1942    (assert (raises-error? (scale-float 1.0d0 (1+ most-positive-fixnum))
1943                           floating-point-overflow)))
1944  as the SCALE-FLOAT just returns 
1945  #.SB-EXT:SINGLE/DOUBLE-FLOAT-POSITIVE-INFINITY. These tests have been
1946  disabled on Darwin for now.
1948 375: MISC.555
1949     (compile nil '(lambda (p1)
1950                     (declare (optimize (speed 1) (safety 2) (debug 2) (space 0))
1951                              (type keyword p1))
1952                     (keywordp p1)))
1954   fails on hairy type check in IR2.
1956   1. KEYWORDP is MAYBE-INLINE expanded (before TYPEP-like
1957      transformation could eliminate it).
1959   2. From the only call of KEYWORDP the type of its argument is
1960      derived to be KEYWORD.
1962   2. Type check for P1 is generated; it uses KEYWORDP to perform the
1963      check, and so references the local function; from the KEYWORDP
1964      argument type new CAST to KEYWORD is generated. The compiler
1965      loops forever.
1967 377: Memory fault error reporting
1968   On those architectures where :C-STACK-IS-CONTROL-STACK is in
1969   *FEATURES*, we handle SIG_MEMORY_FAULT (SEGV or BUS) on an altstack,
1970   so we cannot handle the signal directly (as in interrupt_handle_now())
1971   in the case when the signal comes from some external agent (the user
1972   using kill(1), or a fault in some foreign code, for instance).  As
1973   of sbcl-0.8.20.20, this is fixed by calling
1974   arrange_return_to_lisp_function() to a new error-signalling
1975   function, but as a result the error reporting is poor: we cannot
1976   even tell the user at which address the fault occurred.  We should
1977   arrange such that arguments can be passed to the function called from
1978   arrange_return_to_lisp_function(), but this looked hard to do in
1979   general without suffering from memory leaks.
1981 379: TRACE :ENCAPSULATE NIL broken on ppc/darwin
1982   See commented-out test-case in debug.impure.lisp.
1984 380: Accessor redefinition fails because of old accessor name
1985   When redefining an accessor, SB-PCL::FIX-SLOT-ACCESSORS may try to
1986   find the generic function named by the old accessor name using
1987   ENSURE-GENERIC-FUNCTION and then remove the old accessor's method in
1988   the GF. If the old name does not name a function, or if the old name
1989   does not name a generic function, no attempt to find the GF or remove
1990   any methods is made.
1992   However, if an unrelated GF with an incompatible lambda list exists,
1993   the class redefinition will fail when SB-PCL::REMOVE-READER-METHOD
1994   tries to find and remove a method with an incompatible lambda list
1995   from the unrelated generic function.
1997 381: incautious calls to EQUAL in fasl dumping
1998   Compiling 
1999     (frob #(#1=(a #1#)))
2000     (frob #(#1=(b #1#)))
2001     (frob #(#1=(a #1#)))
2002   in sbcl-0.9.0 causes CONTROL-STACK-EXHAUSTED. My (WHN) impression 
2003   is that this follows from the use of (MAKE-HASH-TABLE :TEST 'EQUAL)
2004   to detect sharing, in which case fixing it might require either 
2005   getting less ambitious about detecting shared list structure, or 
2006   implementing the moral equivalent of EQUAL hash tables in a 
2007   cycle-tolerant way.
2009 382: externalization unexpectedly changes array simplicity
2010   COMPILE-FILE and LOAD
2011     (defun foo ()
2012       (let ((x #.(make-array 4 :fill-pointer 0)))
2013         (values (eval `(typep ',x 'simple-array))
2014                 (typep x 'simple-array))))
2015   then (FOO) => T, NIL.
2017   Similar problems exist with SIMPLE-ARRAY-P, ARRAY-HEADER accessors
2018   and all array dimension functions.
2020 383: ASH'ing non-constant zeros
2021   Compiling
2022     (lambda (b)
2023       (declare (type (integer -2 14) b))
2024       (declare (ignorable b))
2025       (ash (imagpart b) 57))
2026   on PPC (and other platforms, presumably) gives an error during the
2027   emission of FASH-ASH-LEFT/FIXNUM=>FIXNUM as the assembler attempts to
2028   stuff a too-large constant into the immediate field of a PPC
2029   instruction.  Either the VOP should be fixed or the compiler should be
2030   taught how to transform this case away, paying particular attention
2031   to side-effects that might occur in the arguments to ASH.
2033 384: Compiler runaway on very large character types
2035   (compile nil '(lambda (x)
2036                     (declare (type (member #\a 1) x))
2037                     (the (member 1 nil) x)))
2039   The types apparently normalize into a very large type, and the compiler
2040   gets lost in REMOVE-DUPLICATES.  Perhaps the latter should use
2041   a better algorithm (one based on hash tables, say) on very long lists
2042   when :TEST has its default value?
2044   A simpler example:
2046   (compile nil '(lambda (x) (the (not (eql #\a)) x)))
2048   (partially fixed in 0.9.3.1, but a better representation for these
2049    types is needed.)
2051 385:
2052   (format nil "~4,1F" 0.001) => "0.00" (should be " 0.0");
2053   (format nil "~4,1@F" 0.001) => "+.00" (should be "+0.0").
2055 386: SunOS/x86 stack exhaustion handling broken
2056   According to <http://alfa.s145.xrea.com/sbcl/solaris-x86.html>, the
2057   stack exhaustion checking (implemented with a write-protected guard
2058   page) does not work on SunOS/x86.
2060 387:
2061   12:10 < jsnell> the package-lock test is basically due to a change in the test 
2062                   behaviour when you install a handler for error around it. I 
2063                   thought I'd disabled the test for now, but apparently that was 
2064                   my imagination
2065   12:19 < Xophe> jsnell: ah, I see the problem in the package-locks stuff
2066   12:19 < Xophe> it's the same problem as we had with compiler-error conditions
2067   12:19 < Xophe> the thing that's signalled up and down the stack is a subtype of
2068                   ERROR, where it probably shouldn't be
2070 388:
2071   (found by Dmitry Bogomolov)
2073     (defclass foo () ((x :type (unsigned-byte 8))))
2074     (defclass bar () ((x :type symbol)))
2075     (defclass baz (foo bar) ())
2077   causes error
2079     SB-PCL::SPECIALIZER-APPLICABLE-USING-TYPE-P cannot handle the second argument
2080     (UNSIGNED-BYTE 8).
2082 389:
2083   (reported several times on sbcl-devel, by Rick Taube, Brian Rowe and
2084   others)
2086   ROUND-NUMERIC-BOUND assumes that float types always have a FORMAT
2087   specifying whether they're SINGLE or DOUBLE.  This is true for types
2088   computed by the type system itself, but the compiler type derivation
2089   short-circuits this and constructs non-canonical types.  A temporary
2090   fix was made to ROUND-NUMERIC-BOUND for the sbcl-0.9.6 release, but
2091   the right fix is to remove the abstraction violation in the
2092   compiler's type deriver.
2094 392: slot-accessor for subclass misses obsoleted superclass
2095   (fixed in sbcl-0.9.7.9)