X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=tests%2Fclos.impure.lisp;h=2113f59fc33eeaa18066a1587402f03b78319cc9;hb=062283b901155792f65775491aea51481c56faaa;hp=05b8f8713e86f49b1c340a08eb0d4ca6254f6bd5;hpb=1da7e361946dc94df2fae31cd8f2bad2497fa1d0;p=sbcl.git diff --git a/tests/clos.impure.lisp b/tests/clos.impure.lisp index 05b8f87..2113f59 100644 --- a/tests/clos.impure.lisp +++ b/tests/clos.impure.lisp @@ -6,15 +6,14 @@ ;;;; While most of SBCL is derived from the CMU CL system, the test ;;;; files (like this one) were written from scratch after the fork ;;;; from CMU CL. -;;;; +;;;; ;;;; This software is in the public domain and is provided with ;;;; absolutely no warranty. See the COPYING and CREDITS files for ;;;; more information. -(load "assertoid.lisp") - +(load "compiler-test-util.lisp") (defpackage "CLOS-IMPURE" - (:use "CL" "ASSERTOID")) + (:use "CL" "ASSERTOID" "TEST-UTIL" "COMPILER-TEST-UTIL")) (in-package "CLOS-IMPURE") ;;; It should be possible to do DEFGENERIC and DEFMETHOD referring to @@ -63,9 +62,7 @@ (ignore-errors (progn ,@body)) (declare (ignore res)) (typep condition 'error)))) -(assert (expect-error - (macroexpand-1 - '(defmethod foo0 ((x t) &rest) nil)))) +(assert (expect-error (defmethod foo0 ((x t) &rest) nil))) (assert (expect-error (defgeneric foo1 (x &rest)))) (assert (expect-error (defgeneric foo2 (x a &rest)))) (defgeneric foo3 (x &rest y)) @@ -73,7 +70,16 @@ (defmethod foo4 ((x t) &rest z &key y) nil) (defgeneric foo4 (x &rest z &key y)) (assert (expect-error (defgeneric foo5 (x &rest)))) -(assert (expect-error (macroexpand-1 '(defmethod foo6 (x &rest))))) +(assert (expect-error (defmethod foo6 (x &rest)))) + +;;; legal method specializers +(defclass bug-525916-1 () ()) +(defclass bug-525916-2 () ()) +(with-test (:name :bug-525916) +(assert (expect-error (defmethod invalid ((arg)) arg))) +(assert (expect-error (defmethod invalid (nil) 1))) +(assert (expect-error (defmethod invalid ((arg . bug-525916-1)) arg))) +(assert (expect-error (defmethod invalid ((arg bug-525916-1 bug-525916-2)) arg)))) ;;; more lambda-list checking ;;; @@ -81,26 +87,26 @@ ;;; section 3.4.2 of the ANSI spec. Since Alexey Dejneka's patch for ;;; bug 191-b ca. sbcl-0.7.22, these limitations should be enforced. (labels ((coerce-to-boolean (x) - (if x t nil)) - (%like-or-dislike (expr expected-failure-p) + (if x t nil)) + (%like-or-dislike (expr expected-failure-p) (declare (type boolean expected-failure-p)) (format t "~&trying ~S~%" expr) (multiple-value-bind (fun warnings-p failure-p) - (compile nil - `(lambda () + (compile nil + `(lambda () ,expr)) - (declare (ignore fun)) - ;; In principle the constraint on WARNINGS-P below seems - ;; reasonable, but in practice we get warnings about - ;; undefined functions from the DEFGENERICs, apparently - ;; because the DECLAIMs which ordinarily prevent such - ;; warnings don't take effect because EVAL-WHEN - ;; (:COMPILE-TOPLEVEL) loses its magic when compiled - ;; within a LAMBDA. So maybe we can't test WARNINGS-P - ;; after all? + (declare (ignore fun)) + ;; In principle the constraint on WARNINGS-P below seems + ;; reasonable, but in practice we get warnings about + ;; undefined functions from the DEFGENERICs, apparently + ;; because the DECLAIMs which ordinarily prevent such + ;; warnings don't take effect because EVAL-WHEN + ;; (:COMPILE-TOPLEVEL) loses its magic when compiled + ;; within a LAMBDA. So maybe we can't test WARNINGS-P + ;; after all? ;;(unless expected-failure-p - ;; (assert (not warnings-p))) - (assert (eq (coerce-to-boolean failure-p) expected-failure-p)))) + ;; (assert (not warnings-p))) + (assert (eq (coerce-to-boolean failure-p) expected-failure-p)))) (like (expr) (%like-or-dislike expr nil)) (dislike (expr) @@ -110,14 +116,14 @@ (like '(defgeneric gf-for-ll-test-1 ())) (like '(defgeneric gf-for-ll-test-2 (x))) ;; forbidden default or supplied-p for &OPTIONAL or &KEY arguments - (dislike '(defgeneric gf-for-ll-test-3 (x &optional (y 0)))) - (like '(defgeneric gf-for-ll-test-4 (x &optional y))) - (dislike '(defgeneric gf-for-ll-test-5 (x y &key (z :z z-p)))) + (dislike '(defgeneric gf-for-ll-test-3 (x &optional (y 0)))) + (like '(defgeneric gf-for-ll-test-4 (x &optional y))) + (dislike '(defgeneric gf-for-ll-test-5 (x y &key (z :z z-p)))) (like '(defgeneric gf-for-ll-test-6 (x y &key z))) - (dislike '(defgeneric gf-for-ll-test-7 (x &optional (y 0) &key z))) - (like '(defgeneric gf-for-ll-test-8 (x &optional y &key z))) - (dislike '(defgeneric gf-for-ll-test-9 (x &optional y &key (z :z)))) - (like '(defgeneric gf-for-ll-test-10 (x &optional y &key z))) + (dislike '(defgeneric gf-for-ll-test-7 (x &optional (y 0) &key z))) + (like '(defgeneric gf-for-ll-test-8 (x &optional y &key z))) + (dislike '(defgeneric gf-for-ll-test-9 (x &optional y &key (z :z)))) + (like '(defgeneric gf-for-ll-test-10 (x &optional y &key z))) (dislike '(defgeneric gf-for-ll-test-11 (&optional &key (k :k k-p)))) (like '(defgeneric gf-for-ll-test-12 (&optional &key k))) ;; forbidden &AUX @@ -148,7 +154,7 @@ ;;; DEFGENERIC's blow-away-old-methods behavior is specified to have ;;; special hacks to distinguish between defined-with-DEFGENERIC-:METHOD ;;; methods and defined-with-DEFMETHOD methods, so that reLOADing -;;; DEFGENERIC-containing files does the right thing instead of +;;; DEFGENERIC-containing files does the right thing instead of ;;; randomly slicing your generic functions. (APD made this work ;;; in sbcl-0.7.0.2.) (defgeneric born-to-be-redefined (x) @@ -193,9 +199,9 @@ (c-slot :initarg :c-slot :accessor c-slot))) (let ((foo (make-instance 'class-with-slots - :a-slot 1 - :b-slot 2 - :c-slot 3))) + :a-slot 1 + :b-slot 2 + :c-slot 3))) (let ((bar (change-class foo 'class-with-slots))) (assert (= (a-slot bar) 1)) (assert (= (b-slot bar) 2)) @@ -291,49 +297,49 @@ ;;; Until sbcl-0.7.7.20, some conditions weren't being signalled, and ;;; some others were of the wrong type: (macrolet ((assert-program-error (form) - `(multiple-value-bind (value error) - (ignore-errors ,form) - (unless (and (null value) (typep error 'program-error)) + `(multiple-value-bind (value error) + (ignore-errors ,form) + (unless (and (null value) (typep error 'program-error)) (error "~S failed: ~S, ~S" ',form value error))))) (assert-program-error (defclass foo001 () (a b a))) - (assert-program-error (defclass foo002 () - (a b) - (:default-initargs x 'a x 'b))) + (assert-program-error (defclass foo002 () + (a b) + (:default-initargs x 'a x 'b))) (assert-program-error (defclass foo003 () - ((a :allocation :class :allocation :class)))) + ((a :allocation :class :allocation :class)))) (assert-program-error (defclass foo004 () - ((a :silly t)))) + ((a :silly t)))) ;; and some more, found by Wolfhard Buss and fixed for cmucl by Gerd ;; Moellmann in sbcl-0.7.8.x: (assert-program-error (progn - (defmethod odd-key-args-checking (&key (key 42)) key) - (odd-key-args-checking 3))) + (defmethod odd-key-args-checking (&key (key 42)) key) + (odd-key-args-checking 3))) (assert (= (odd-key-args-checking) 42)) (assert (eq (odd-key-args-checking :key t) t)) ;; yet some more, fixed in sbcl-0.7.9.xx (assert-program-error (defclass foo005 () - (:metaclass sb-pcl::funcallable-standard-class) - (:metaclass 1))) + (:metaclass sb-pcl::funcallable-standard-class) + (:metaclass 1))) (assert-program-error (defclass foo006 () - ((a :reader (setf a))))) + ((a :reader (setf a))))) (assert-program-error (defclass foo007 () - ((a :initarg 1)))) + ((a :initarg 1)))) (assert-program-error (defclass foo008 () - (a :initarg :a) - (:default-initargs :a 1) - (:default-initargs :a 2))) + (a :initarg :a) + (:default-initargs :a 1) + (:default-initargs :a 2))) ;; and also BUG 47d, fixed in sbcl-0.8alpha.0.26 (assert-program-error (defgeneric if (x))) ;; DEFCLASS should detect an error if slot names aren't suitable as ;; variable names: (assert-program-error (defclass foo009 () - ((:a :initarg :a)))) + ((:a :initarg :a)))) (assert-program-error (defclass foo010 () - (("a" :initarg :a)))) + (("a" :initarg :a)))) (assert-program-error (defclass foo011 () - ((#1a() :initarg :a)))) + ((#1a() :initarg :a)))) (assert-program-error (defclass foo012 () - ((t :initarg :t)))) + ((t :initarg :t)))) (assert-program-error (defclass foo013 () ("a"))) ;; specialized lambda lists have certain restrictions on ordering, ;; repeating keywords, and the like: @@ -341,7 +347,7 @@ (assert-program-error (defmethod foo015 ((foo t) &rest x y) nil)) (assert-program-error (defmethod foo016 ((foo t) &allow-other-keys) nil)) (assert-program-error (defmethod foo017 ((foo t) - &optional x &optional y) nil)) + &optional x &optional y) nil)) (assert-program-error (defmethod foo018 ((foo t) &rest x &rest y) nil)) (assert-program-error (defmethod foo019 ((foo t) &rest x &optional y) nil)) (assert-program-error (defmethod foo020 ((foo t) &key x &optional y) nil)) @@ -350,21 +356,25 @@ ;;; DOCUMENTATION's argument-precedence-order wasn't being faithfully ;;; preserved through the bootstrap process until sbcl-0.7.8.39. ;;; (thanks to Gerd Moellmann) -(let ((answer (documentation '+ 'function))) - (assert (stringp answer)) - (defmethod documentation ((x (eql '+)) y) "WRONG") - (assert (string= (documentation '+ 'function) answer))) +(with-test (:name :documentation-argument-precedence-order) + (defun foo022 () + "Documentation" + t) + (let ((answer (documentation 'foo022 'function))) + (assert (stringp answer)) + (defmethod documentation ((x (eql 'foo022)) y) "WRONG") + (assert (string= (documentation 'foo022 'function) answer)))) ;;; only certain declarations are permitted in DEFGENERIC (macrolet ((assert-program-error (form) - `(multiple-value-bind (value error) - (ignore-errors ,form) - (assert (null value)) - (assert (typep error 'program-error))))) + `(multiple-value-bind (value error) + (ignore-errors ,form) + (assert (null value)) + (assert (typep error 'program-error))))) (assert-program-error (defgeneric bogus-declaration (x) - (declare (special y)))) + (declare (special y)))) (assert-program-error (defgeneric bogus-declaration2 (x) - (declare (notinline concatenate))))) + (declare (notinline concatenate))))) ;;; CALL-NEXT-METHOD should call NO-NEXT-METHOD if there is no next ;;; method. (defmethod no-next-method-test ((x integer)) (call-next-method)) @@ -409,46 +419,46 @@ (:method-combination dmc-test-mc)) (defmethod dmc-test-mc dmc-test-mc (&key k) - k) + k) (dmc-test-mc :k 1) ;;; While I'm at it, DEFINE-METHOD-COMBINATION is defined to return ;;; the NAME argument, not some random method object. So: (assert (eq (define-method-combination dmc-test-return-foo) - 'dmc-test-return-foo)) + 'dmc-test-return-foo)) (assert (eq (define-method-combination dmc-test-return-bar :operator and) - 'dmc-test-return-bar)) + 'dmc-test-return-bar)) (assert (eq (define-method-combination dmc-test-return - (&optional (order :most-specific-first)) - ((around (:around)) - (primary (dmc-test-return) :order order :required t)) - (let ((form (if (rest primary) - `(and ,@(mapcar #'(lambda (method) - `(call-method ,method)) - primary)) - `(call-method ,(first primary))))) - (if around - `(call-method ,(first around) - (,@(rest around) - (make-method ,form))) - form))) - 'dmc-test-return)) + (&optional (order :most-specific-first)) + ((around (:around)) + (primary (dmc-test-return) :order order :required t)) + (let ((form (if (rest primary) + `(and ,@(mapcar #'(lambda (method) + `(call-method ,method)) + primary)) + `(call-method ,(first primary))))) + (if around + `(call-method ,(first around) + (,@(rest around) + (make-method ,form))) + form))) + 'dmc-test-return)) ;;; DEFINE-METHOD-COMBINATION should, according to the description in 7.7, -;;; allow you to do everything in the body forms yourself if you specify +;;; allow you to do everything in the body forms yourself if you specify ;;; exactly one method group whose qualifier-pattern is * ;;; ;;; The specific language is: -;;; "The use of method group specifiers provides a convenient syntax to select -;;; methods, to divide them among the possible roles, and to perform the -;;; necessary error checking. It is possible to perform further filtering of -;;; methods in the body forms by using normal list-processing operations and +;;; "The use of method group specifiers provides a convenient syntax to select +;;; methods, to divide them among the possible roles, and to perform the +;;; necessary error checking. It is possible to perform further filtering of +;;; methods in the body forms by using normal list-processing operations and ;;; the functions method-qualifiers and invalid-method-error. It is permissible -;;; to use setq on the variables named in the method group specifiers and to +;;; to use setq on the variables named in the method group specifiers and to ;;; bind additional variables. It is also possible to bypass the method group -;;; specifier mechanism and do everything in the body forms. This is -;;; accomplished by writing a single method group with * as its only -;;; qualifier-pattern; the variable is then bound to a list of all of the +;;; specifier mechanism and do everything in the body forms. This is +;;; accomplished by writing a single method group with * as its only +;;; qualifier-pattern; the variable is then bound to a list of all of the ;;; applicable methods, in most-specific-first order." (define-method-combination wam-test-method-combination-a () ((all-methods *)) @@ -457,23 +467,23 @@ (around nil)) ((null methods) (let ((primary (nreverse primary)) - (around (nreverse around))) - (if primary - (let ((form (if (rest primary) - `(call-method ,(first primary) ,(rest primary)) - `(call-method ,(first primary))))) - (if around - `(call-method ,(first around) (,@(rest around) - (make-method ,form))) - form)) - `(make-method (error "No primary methods"))))) + (around (nreverse around))) + (if primary + (let ((form (if (rest primary) + `(call-method ,(first primary) ,(rest primary)) + `(call-method ,(first primary))))) + (if around + `(call-method ,(first around) (,@(rest around) + (make-method ,form))) + form)) + `(make-method (error "No primary methods"))))) (let* ((method (first methods)) - (qualifier (first (method-qualifiers method)))) + (qualifier (first (method-qualifiers method)))) (cond - ((equal :around qualifier) - (push method around)) - ((null qualifier) - (push method primary)))))) + ((equal :around qualifier) + (push method around)) + ((null qualifier) + (push method primary)))))) (defgeneric wam-test-mc-a (val) (:method-combination wam-test-method-combination-a)) @@ -485,8 +495,8 @@ (+ val (if (next-method-p) (call-next-method) 0))) (assert (= (wam-test-mc-a 13) 26)) -;;; DEFINE-METHOD-COMBINATION -;;; When two methods are in the same method group and have the same +;;; DEFINE-METHOD-COMBINATION +;;; When two methods are in the same method group and have the same ;;; specializers, their sort order within the group may be ambiguous. Therefore, ;;; we should throw an error when we have two methods in the same group with ;;; the same specializers /as long as/ we have more than one method group @@ -494,14 +504,14 @@ ;;; apparent conflict with the above 'It is also possible to bypass' language. ;;; ;;; The language specifying this behavior is: -;;; "Note that two methods with identical specializers, but with different -;;; qualifiers, are not ordered by the algorithm described in Step 2 of the -;;; method selection and combination process described in Section 7.6.6 +;;; "Note that two methods with identical specializers, but with different +;;; qualifiers, are not ordered by the algorithm described in Step 2 of the +;;; method selection and combination process described in Section 7.6.6 ;;; (Method Selection and Combination). Normally the two methods play different -;;; roles in the effective method because they have different qualifiers, and -;;; no matter how they are ordered in the result of Step 2, the effective -;;; method is the same. If the two methods play the same role and their order -;;; matters, an error is signaled. This happens as part of the qualifier +;;; roles in the effective method because they have different qualifiers, and +;;; no matter how they are ordered in the result of Step 2, the effective +;;; method is the same. If the two methods play the same role and their order +;;; matters, an error is signaled. This happens as part of the qualifier ;;; pattern matching in define-method-combination." ;;; ;;; Note that the spec pretty much equates 'method group' and 'role'. @@ -511,12 +521,12 @@ ((around (:around)) (primary * :required t)) (let ((form (if (rest primary) - `(call-method ,(first primary) ,(rest primary)) - `(call-method ,(first primary))))) + `(call-method ,(first primary) ,(rest primary)) + `(call-method ,(first primary))))) (if around - `(call-method ,(first around) (,@(rest around) - (make-method ,form))) - form))) + `(call-method ,(first around) (,@(rest around) + (make-method ,form))) + form))) (defgeneric wam-test-mc-b (val) (:method-combination wam-test-method-combination-b)) @@ -526,7 +536,7 @@ (defmethod wam-test-mc-b :around ((val number)) (+ val (if (next-method-p) (call-next-method) 0))) (assert (= (wam-test-mc-b 13) 26)) -(defmethod wam-test-mc-b :somethingelse ((val number)) +(defmethod wam-test-mc-b :somethingelse ((val number)) (+ val (if (next-method-p) (call-next-method) 0))) (assert (raises-error? (wam-test-mc-b 13))) @@ -555,16 +565,16 @@ (assert (raises-error? (defmethod incompatible-ll-test-1 (x &rest y) y))) ;;; Sneakily using a bit of MOPness to check some consistency (assert (= (length - (sb-pcl:generic-function-methods #'incompatible-ll-test-1)) 1)) + (sb-pcl:generic-function-methods #'incompatible-ll-test-1)) 1)) (defmethod incompatible-ll-test-2 (x &key bar) bar) (assert (raises-error? (defmethod incompatible-ll-test-2 (x) x))) (defmethod incompatible-ll-test-2 (x &rest y) y) (assert (= (length - (sb-pcl:generic-function-methods #'incompatible-ll-test-2)) 1)) + (sb-pcl:generic-function-methods #'incompatible-ll-test-2)) 1)) (defmethod incompatible-ll-test-2 ((x integer) &key bar) bar) (assert (= (length - (sb-pcl:generic-function-methods #'incompatible-ll-test-2)) 2)) + (sb-pcl:generic-function-methods #'incompatible-ll-test-2)) 2)) ;;; Per Christophe, this is an illegal method call because of 7.6.5 (assert (raises-error? (incompatible-ll-test-2 t 1 2))) @@ -615,9 +625,9 @@ (:arguments object) `(unwind-protect (progn (lock (object-lock ,object)) - ,@(mapcar #'(lambda (method) - `(call-method ,method)) - methods)) + ,@(mapcar #'(lambda (method) + `(call-method ,method)) + methods)) (unlock (object-lock ,object)))) (defun object-lock (obj) (push "object-lock" *d-m-c-args-test*) @@ -636,11 +646,11 @@ (error "foo")) (assert (equal (d-m-c-args-test t) '("primary" "lock" "object-lock"))) (assert (equal *d-m-c-args-test* - '("unlock" "object-lock" "primary" "lock" "object-lock"))) + '("unlock" "object-lock" "primary" "lock" "object-lock"))) (setf *d-m-c-args-test* nil) (ignore-errors (d-m-c-args-test 1)) (assert (equal *d-m-c-args-test* - '("unlock" "object-lock" "lock" "object-lock"))) + '("unlock" "object-lock" "lock" "object-lock"))) ;;; The walker (on which DEFMETHOD depended) didn't know how to handle ;;; SYMBOL-MACROLET properly. In fact, as of sbcl-0.7.10.20 it still @@ -654,6 +664,7 @@ (assert (= (bug222 t) 1)) ;;; also, a test case to guard against bogus environment hacking: + (eval-when (:compile-toplevel :load-toplevel :execute) (setq bug222-b 3)) ;;; this should at the least compile: @@ -664,8 +675,10 @@ ;;; and it would be nice (though not specified by ANSI) if the answer ;;; were as follows: (let ((x (make-string-output-stream))) - ;; not specified by ANSI - (assert (= (bug222-b t x) 3)) + (let ((value (bug222-b t x))) + ;; not specified by ANSI + #+#.(cl:if (cl:eq sb-ext:*evaluator-mode* :compile) '(and) '(or)) + (assert (= value 3))) ;; specified. (assert (char= (char (get-output-stream-string x) 0) #\1))) @@ -683,8 +696,8 @@ (incf *bug234*)) (assert (typep (subbug-234) 'subclass234)) (assert (= *bug234* - ;; once for MAKE-INSTANCE, once for REINITIALIZE-INSTANCE - 2)) + ;; once for MAKE-INSTANCE, once for REINITIALIZE-INSTANCE + 2)) ;;; also, some combinations of MAKE-INSTANCE and subclassing missed ;;; new methods (Gerd Moellmann sbcl-devel 2002-12-29): @@ -708,19 +721,19 @@ ;;; SLOT-MISSING should be called when there are missing slots. (defclass class-with-all-slots-missing () ()) (defmethod slot-missing (class (o class-with-all-slots-missing) - slot-name op - &optional new-value) + slot-name op + &optional new-value) op) (assert (eq (slot-value (make-instance 'class-with-all-slots-missing) 'foo) - 'slot-value)) + 'slot-value)) (assert (eq (funcall (lambda (x) (slot-value x 'bar)) - (make-instance 'class-with-all-slots-missing)) - 'slot-value)) + (make-instance 'class-with-all-slots-missing)) + 'slot-value)) (assert (eq (funcall (lambda (x) (setf (slot-value x 'baz) 'baz)) - (make-instance 'class-with-all-slots-missing)) - ;; SLOT-MISSING's value is specified to be ignored; we - ;; return NEW-VALUE. - 'baz)) + (make-instance 'class-with-all-slots-missing)) + ;; SLOT-MISSING's value is specified to be ignored; we + ;; return NEW-VALUE. + 'baz)) ;;; we should be able to specialize on anything that names a class. (defclass name-for-class () ()) @@ -729,7 +742,7 @@ (defmethod something-that-specializes ((x other-name-for-class)) 2) (assert (= (something-that-specializes (make-instance 'name-for-class)) 2)) (assert (= (something-that-specializes (make-instance 'other-name-for-class)) - 2)) + 2)) ;;; more forward referenced classes stuff (defclass frc-1 (frc-2) ()) @@ -755,7 +768,7 @@ ;;; DEFSTRUCT (and not DEFCLASS :METACLASS STRUCTURE-CLASS). (defstruct allocatable-structure a) (assert (typep (allocate-instance (find-class 'allocatable-structure)) - 'allocatable-structure)) + 'allocatable-structure)) ;;; Bug found by Paul Dietz when devising CPL tests: somewhat ;;; amazingly, calls to CPL would work a couple of times, and then @@ -774,11 +787,11 @@ (assert (equal (cpl 0) '(integer number))) (assert (equal (cpl 0) '(integer number))) (assert (equal (cpl (make-broadcast-stream)) - '(broadcast-stream stream structure-object))) + '(broadcast-stream stream structure-object))) (assert (equal (cpl (make-broadcast-stream)) - '(broadcast-stream stream structure-object))) + '(broadcast-stream stream structure-object))) (assert (equal (cpl (make-broadcast-stream)) - '(broadcast-stream stream structure-object))) + '(broadcast-stream stream structure-object))) ;;; Bug in CALL-NEXT-METHOD: assignment to the method's formal ;;; parameters shouldn't affect the arguments to the next method for a @@ -786,7 +799,7 @@ (defgeneric cnm-assignment (x) (:method (x) x) (:method ((x integer)) (setq x 3) - (list x (call-next-method) (call-next-method x)))) + (list x (call-next-method) (call-next-method x)))) (assert (equal (cnm-assignment 1) '(3 1 3))) ;;; Bug reported by Istvan Marko 2003-07-09 @@ -834,17 +847,17 @@ (defmethod width ((c character-class) &key font) font) (defmethod width ((p picture-class) &key pixel-size) pixel-size) -(assert (raises-error? - (width (make-instance 'character-class :char #\Q) - :font 'baskerville :pixel-size 10) - program-error)) (assert (raises-error? - (width (make-instance 'picture-class :glyph #\Q) - :font 'baskerville :pixel-size 10) - program-error)) + (width (make-instance 'character-class :char #\Q) + :font 'baskerville :pixel-size 10) + program-error)) +(assert (raises-error? + (width (make-instance 'picture-class :glyph #\Q) + :font 'baskerville :pixel-size 10) + program-error)) (assert (eq (width (make-instance 'character-picture-class :char #\Q) - :font 'baskerville :pixel-size 10) - 'baskerville)) + :font 'baskerville :pixel-size 10) + 'baskerville)) ;;; class redefinition shouldn't give any warnings, in the usual case (defclass about-to-be-redefined () ((some-slot :accessor some-slot))) @@ -855,8 +868,8 @@ ;;; complex lambda lists should fail (defgeneric accessoroid (object &key &allow-other-keys)) (assert (raises-error? - (defclass accessoroid-class () ((slot :accessor accessoroid))) - program-error)) + (defclass accessoroid-class () ((slot :accessor accessoroid))) + program-error)) ;;; reported by Bruno Haible sbcl-devel 2004-04-15 (defclass shared-slot-and-redefinition () @@ -892,29 +905,76 @@ (slot-boundp *obsoleted* 'a) (assert (= *obsoleted-counter* 1)) +;;; yet another MAKE-INSTANCES-OBSOLETE test, this time from Nikodemus +;;; Siivola. Not all methods for accessing slots are created equal... +(defclass yet-another-obsoletion-super () ((obs :accessor obs-of :initform 0))) +(defclass yet-another-obsoletion-sub (yet-another-obsoletion-super) ()) +(defmethod shared-initialize :after ((i yet-another-obsoletion-super) + slots &rest init) + (incf (obs-of i))) + +(defvar *yao-super* (make-instance 'yet-another-obsoletion-super)) +(defvar *yao-sub* (make-instance 'yet-another-obsoletion-sub)) + +(assert (= (obs-of *yao-super*) 1)) +(assert (= (obs-of *yao-sub*) 1)) +(make-instances-obsolete 'yet-another-obsoletion-super) +(assert (= (obs-of *yao-sub*) 2)) +(assert (= (obs-of *yao-super*) 2)) +(make-instances-obsolete 'yet-another-obsoletion-super) +(assert (= (obs-of *yao-super*) 3)) +(assert (= (obs-of *yao-sub*) 3)) +(assert (= (slot-value *yao-super* 'obs) 3)) +(assert (= (slot-value *yao-sub* 'obs) 3)) + +;;; one more MIO test: variable slot names +(defclass mio () ((x :initform 42))) +(defvar *mio-slot* 'x) +(defparameter *mio-counter* 0) +(defmethod update-instance-for-redefined-class ((instance mio) new old plist &key) + (incf *mio-counter*)) + +(let ((x (make-instance 'mio))) + (make-instances-obsolete 'mio) + (slot-value x *mio-slot*)) + +(let ((x (make-instance 'mio))) + (make-instances-obsolete 'mio) + (setf (slot-value x *mio-slot*) 13)) + +(let ((x (make-instance 'mio))) + (make-instances-obsolete 'mio) + (slot-boundp x *mio-slot*)) + +(let ((x (make-instance 'mio))) + (make-instances-obsolete 'mio) + (slot-makunbound x *mio-slot*)) + +(assert (= 4 *mio-counter*)) + ;;; shared -> local slot transfers of inherited slots, reported by ;;; Bruno Haible (let (i) - (defclass super-with-magic-slot () + (defclass super-with-magic-slot () ((magic :initarg :size :initform 1 :allocation :class))) (defclass sub-of-super-with-magic-slot (super-with-magic-slot) ()) (setq i (make-instance 'sub-of-super-with-magic-slot)) - (defclass super-with-magic-slot () + (defclass super-with-magic-slot () ((magic :initarg :size :initform 2))) (assert (= 1 (slot-value i 'magic)))) ;;; MAKE-INSTANCES-OBSOLETE return values (defclass one-more-to-obsolete () ()) -(assert (eq 'one-more-to-obsolete - (make-instances-obsolete 'one-more-to-obsolete))) -(assert (eq (find-class 'one-more-to-obsolete) - (make-instances-obsolete (find-class 'one-more-to-obsolete)))) +(assert (eq 'one-more-to-obsolete + (make-instances-obsolete 'one-more-to-obsolete))) +(assert (eq (find-class 'one-more-to-obsolete) + (make-instances-obsolete (find-class 'one-more-to-obsolete)))) ;;; Sensible error instead of a BUG. Reported by Thomas Burdick. (multiple-value-bind (value err) (ignore-errors (defclass slot-def-with-duplicate-accessors () - ((slot :writer get-slot :reader get-slot)))) + ((slot :writer get-slot :reader get-slot)))) (assert (typep err 'error)) (assert (not (typep err 'sb-int:bug)))) @@ -988,36 +1048,36 @@ ;;; methods on all of these. (progn (defgeneric method-for-defined-classes (x)) - (dolist (c '(arithmetic-error - generic-function simple-error array hash-table - simple-type-error - bit-vector integer simple-warning - broadcast-stream list standard-class - built-in-class logical-pathname standard-generic-function - cell-error method standard-method - character method-combination standard-object - class null storage-condition - complex number stream - concatenated-stream package stream-error - condition package-error string - cons parse-error string-stream - control-error pathname structure-class - division-by-zero print-not-readable structure-object - echo-stream program-error style-warning - end-of-file random-state symbol - error ratio synonym-stream - file-error rational t - file-stream reader-error two-way-stream - float readtable type-error - floating-point-inexact real unbound-slot - floating-point-invalid-operation restart unbound-variable - floating-point-overflow sequence undefined-function - floating-point-underflow serious-condition vector - function simple-condition warning)) + (dolist (c '(arithmetic-error + generic-function simple-error array hash-table + simple-type-error + bit-vector integer simple-warning + broadcast-stream list standard-class + built-in-class logical-pathname standard-generic-function + cell-error method standard-method + character method-combination standard-object + class null storage-condition + complex number stream + concatenated-stream package stream-error + condition package-error string + cons parse-error string-stream + control-error pathname structure-class + division-by-zero print-not-readable structure-object + echo-stream program-error style-warning + end-of-file random-state symbol + error ratio synonym-stream + file-error rational t + file-stream reader-error two-way-stream + float readtable type-error + floating-point-inexact real unbound-slot + floating-point-invalid-operation restart unbound-variable + floating-point-overflow sequence undefined-function + floating-point-underflow serious-condition vector + function simple-condition warning)) (eval `(defmethod method-for-defined-classes ((x ,c)) (princ x)))) (assert (string= (with-output-to-string (*standard-output*) - (method-for-defined-classes #\3)) - "3"))) + (method-for-defined-classes #\3)) + "3"))) @@ -1042,26 +1102,977 @@ (load "package-ctor-bug.lisp") (assert (= (package-ctor-bug:test) 3)) -(deftype defined-type () 'integer) -(assert (raises-error? - (defmethod method-on-defined-type ((x defined-type)) x))) -(deftype defined-type-and-class () 'integer) -(setf (find-class 'defined-type-and-class) (find-class 'integer)) -(defmethod method-on-defined-type-and-class ((x defined-type-and-class)) - (1+ x)) -(assert (= (method-on-defined-type-and-class 3) 4)) +(with-test (:name (:defmethod (setf find-class) integer)) + (mapcar #'eval + '( + (deftype defined-type () 'integer) + (assert (raises-error? + (defmethod method-on-defined-type ((x defined-type)) x))) + (deftype defined-type-and-class () 'integer) + (setf (find-class 'defined-type-and-class) (find-class 'integer)) + (defmethod method-on-defined-type-and-class + ((x defined-type-and-class)) + (1+ x)) + (assert (= (method-on-defined-type-and-class 3) 4))))) ;; bug 281 -(let ((sb-pcl::*max-emf-precomputation-methods* 0)) +(let (#+nil ; no more sb-pcl::*max-emf-precomputation-methods* as of + ; sbcl-1.0.41.x + (sb-pcl::*max-emf-precomputation-methods* 0)) (eval '(defgeneric bug-281 (x) - (:method-combination +) - (:method ((x symbol)) 1) - (:method + ((x number)) x))) + (:method-combination +) + (:method ((x symbol)) 1) + (:method + ((x number)) x))) (assert (= 1 (bug-281 1))) (assert (= 4.2 (bug-281 4.2))) (multiple-value-bind (val err) (ignore-errors (bug-281 'symbol)) (assert (not val)) (assert (typep err 'error)))) + +;;; RESTART-CASE and CALL-METHOD + +;;; from Bruno Haible + +(defun rc-cm/prompt-for-new-values () + (format *debug-io* "~&New values: ") + (finish-output *debug-io*) + (list (read *debug-io*))) + +(defun rc-cm/add-method-restarts (form method) + (let ((block (gensym)) + (tag (gensym))) + `(block ,block + (tagbody + ,tag + (return-from ,block + (restart-case ,form + (method-redo () + :report (lambda (stream) + (format stream "Try calling ~S again." ,method)) + (go ,tag)) + (method-return (l) + :report (lambda (stream) + (format stream "Specify return values for ~S call." + ,method)) + :interactive (lambda () (rc-cm/prompt-for-new-values)) + (return-from ,block (values-list l))))))))) + +(defun rc-cm/convert-effective-method (efm) + (if (consp efm) + (if (eq (car efm) 'call-method) + (let ((method-list (third efm))) + (if (or (typep (first method-list) 'method) (rest method-list)) + ;; Reduce the case of multiple methods to a single one. + ;; Make the call to the next-method explicit. + (rc-cm/convert-effective-method + `(call-method ,(second efm) + ((make-method + (call-method ,(first method-list) ,(rest method-list)))))) + ;; Now the case of at most one method. + (if (typep (second efm) 'method) + ;; Wrap the method call in a RESTART-CASE. + (rc-cm/add-method-restarts + (cons (rc-cm/convert-effective-method (car efm)) + (rc-cm/convert-effective-method (cdr efm))) + (second efm)) + ;; Normal recursive processing. + (cons (rc-cm/convert-effective-method (car efm)) + (rc-cm/convert-effective-method (cdr efm)))))) + (cons (rc-cm/convert-effective-method (car efm)) + (rc-cm/convert-effective-method (cdr efm)))) + efm)) + +(define-method-combination standard-with-restarts () + ((around (:around)) + (before (:before)) + (primary () :required t) + (after (:after))) + (flet ((call-methods-sequentially (methods) + (mapcar #'(lambda (method) + `(call-method ,method)) + methods))) + (let ((form (if (or before after (rest primary)) + `(multiple-value-prog1 + (progn + ,@(call-methods-sequentially before) + (call-method ,(first primary) ,(rest primary))) + ,@(call-methods-sequentially (reverse after))) + `(call-method ,(first primary))))) + (when around + (setq form + `(call-method ,(first around) + (,@(rest around) (make-method ,form))))) + (rc-cm/convert-effective-method form)))) + +(defgeneric rc-cm/testgf16 (x) + (:method-combination standard-with-restarts)) +(defclass rc-cm/testclass16a () ()) +(defclass rc-cm/testclass16b (rc-cm/testclass16a) ()) +(defclass rc-cm/testclass16c (rc-cm/testclass16a) ()) +(defclass rc-cm/testclass16d (rc-cm/testclass16b rc-cm/testclass16c) ()) +(defmethod rc-cm/testgf16 ((x rc-cm/testclass16a)) + (list 'a + (not (null (find-restart 'method-redo))) + (not (null (find-restart 'method-return))))) +(defmethod rc-cm/testgf16 ((x rc-cm/testclass16b)) + (cons 'b (call-next-method))) +(defmethod rc-cm/testgf16 ((x rc-cm/testclass16c)) + (cons 'c (call-next-method))) +(defmethod rc-cm/testgf16 ((x rc-cm/testclass16d)) + (cons 'd (call-next-method))) +(assert (equal (rc-cm/testgf16 (make-instance 'rc-cm/testclass16d)) + '(d b c a t t))) + +;;; test case from Gerd Moellmann +(define-method-combination r-c/c-m-1 () + ((primary () :required t)) + `(restart-case (call-method ,(first primary)))) + +(defgeneric r-c/c-m-1-gf () + (:method-combination r-c/c-m-1) + (:method () nil)) + +(assert (null (r-c/c-m-1-gf))) + +(handler-bind ((warning #'error)) + (eval '(defclass class-for-ctor/class-slot () + ((class-slot :initarg :class-slot :allocation :class)))) + (eval '(let ((c1 (make-instance 'class-for-ctor/class-slot)) + (c2 (make-instance 'class-for-ctor/class-slot :class-slot 1))) + (assert (equal (list (slot-value c1 'class-slot) + (slot-value c2 'class-slot)) + (list 1 1)))))) + +;;; tests of ctors on anonymous classes +(defparameter *unnamed* (defclass ctor-unnamed-literal-class () ())) +(setf (class-name *unnamed*) nil) +(setf (find-class 'ctor-unnamed-literal-class) nil) +(defparameter *unnamed2* (defclass ctor-unnamed-literal-class2 () ())) +(defun ctor-unnamed-literal-class () + (make-instance '#.*unnamed*)) +(compile 'ctor-unnamed-literal-class) +(defun ctor-unnamed-literal-class2 () + (make-instance '#.(find-class 'ctor-unnamed-literal-class2))) +(compile 'ctor-unnamed-literal-class2) +(defun ctor-unnamed-literal-class2/symbol () + (make-instance 'ctor-unnamed-literal-class2)) +(compile 'ctor-unnamed-literal-class2/symbol) +(setf (class-name *unnamed2*) nil) +(setf (find-class 'ctor-unnamed-literal-class2) nil) +(with-test (:name (:ctor :unnamed-before)) + (assert (typep (ctor-unnamed-literal-class) *unnamed*))) +(with-test (:name (:ctor :unnamed-after)) + (assert (typep (ctor-unnamed-literal-class2) *unnamed2*))) +(with-test (:name (:ctor :unnamed-after/symbol)) + (assert (raises-error? (ctor-unnamed-literal-class2/symbol)))) + +;;; classes with slot types shouldn't break if the types don't name +;;; classes (bug #391) +(defclass slot-type-superclass () ((slot :type fixnum))) +(defclass slot-type-subclass (slot-type-superclass) + ((slot :type (integer 1 5)))) +(let ((instance (make-instance 'slot-type-subclass))) + (setf (slot-value instance 'slot) 3)) + +;;; ctors where there's a non-standard SHARED-INITIALIZE method and an +;;; initarg which isn't self-evaluating (kpreid on #lisp 2006-01-29) +(defclass kpreid-enode () + ((slot :initarg not-a-keyword))) +(defmethod shared-initialize ((o kpreid-enode) slots &key &allow-other-keys) + (call-next-method)) +(defun make-kpreid-enode () + (make-instance 'kpreid-enode 'not-a-keyword 3)) +(with-test (:name (:ctor :non-keyword-initarg)) + (let ((x (make-kpreid-enode)) + (y (make-kpreid-enode))) + (= (slot-value x 'slot) (slot-value y 'slot)))) + +;;; defining a class hierarchy shouldn't lead to spurious classoid +;;; errors on TYPEP questions (reported by Tim Moore on #lisp +;;; 2006-03-10) +(defclass backwards-2 (backwards-1) (a b)) +(defclass backwards-3 (backwards-2) ()) +(defun typep-backwards-3 (x) + (typep x 'backwards-3)) +(defclass backwards-1 () (a b)) +(assert (not (typep-backwards-3 1))) +(assert (not (typep-backwards-3 (make-instance 'backwards-2)))) +(assert (typep-backwards-3 (make-instance 'backwards-3))) + +(defgeneric remove-method-1 (x) + (:method ((x integer)) (1+ x))) +(defgeneric remove-method-2 (x) + (:method ((x integer)) (1- x))) +(assert (eq #'remove-method-1 + (remove-method #'remove-method-1 + (find-method #'remove-method-2 + nil + (list (find-class 'integer)))))) +(assert (= (remove-method-1 3) 4)) +(assert (= (remove-method-2 3) 2)) + +;;; ANSI doesn't require these restarts, but now that we have them we +;;; better test them too. +(defclass slot-unbound-restart-test () ((x))) +(let ((test (make-instance 'slot-unbound-restart-test))) + (assert (not (slot-boundp test 'x))) + (assert (= 42 (handler-bind ((unbound-slot (lambda (c) (use-value 42 c)))) + (slot-value test 'x)))) + (assert (not (slot-boundp test 'x))) + (assert (= 13 (handler-bind ((unbound-slot (lambda (c) (store-value 13 c)))) + (slot-value test 'x)))) + (assert (= 13 (slot-value test 'x)))) + +;;; Using class instances as specializers, reported by Pascal Costanza, ref CLHS 7.6.2 +(defclass class-as-specializer-test () + ()) +(eval `(defmethod class-as-specializer-test1 ((x ,(find-class 'class-as-specializer-test))) + 'foo)) +(assert (eq 'foo (class-as-specializer-test1 (make-instance 'class-as-specializer-test)))) +(funcall (compile nil `(lambda () + (defmethod class-as-specializer-test2 ((x ,(find-class 'class-as-specializer-test))) + 'bar)))) +(assert (eq 'bar (class-as-specializer-test2 (make-instance 'class-as-specializer-test)))) + +;;; CHANGE-CLASS and tricky allocation. +(defclass foo-to-be-changed () + ((a :allocation :class :initform 1))) +(defclass bar-to-be-changed (foo-to-be-changed) ()) +(defvar *bar-to-be-changed* (make-instance 'bar-to-be-changed)) +(defclass baz-to-be-changed () + ((a :allocation :instance :initform 2))) +(change-class *bar-to-be-changed* 'baz-to-be-changed) +(assert (= (slot-value *bar-to-be-changed* 'a) 1)) + +;;; proper name and class redefinition +(defvar *to-be-renamed1* (defclass to-be-renamed1 () ())) +(defvar *to-be-renamed2* (defclass to-be-renamed2 () ())) +(setf (find-class 'to-be-renamed1) (find-class 'to-be-renamed2)) +(defvar *renamed1* (defclass to-be-renamed1 () ())) +(assert (not (eq *to-be-renamed1* *to-be-renamed2*))) +(assert (not (eq *to-be-renamed1* *renamed1*))) +(assert (not (eq *to-be-renamed2* *renamed1*))) + +;;; CLASS-NAME (and various other standardized generic functions) have +;;; their effective methods precomputed; in the process of rearranging +;;; (SETF FIND-CLASS) and FINALIZE-INHERITANCE, this broke. +(defclass class-with-odd-class-name-method () + ((a :accessor class-name))) + +;;; another case where precomputing (this time on PRINT-OBJECT) and +;;; lazily-finalized classes caused problems. (report from James Y +;;; Knight sbcl-devel 20-07-2006) + +(defclass base-print-object () ()) +;;; this has the side-effect of finalizing BASE-PRINT-OBJECT, and +;;; additionally the second specializer (STREAM) changes the cache +;;; structure to require two keys, not just one. +(defmethod print-object ((o base-print-object) (s stream)) + nil) + +;;; unfinalized as yet +(defclass sub-print-object (base-print-object) ()) +;;; the accessor causes an eager finalization +(defclass subsub-print-object (sub-print-object) + ((a :accessor a))) + +;;; triggers a discriminating function (and so cache) recomputation. +;;; The method on BASE-PRINT-OBJECT will cause the system to attempt +;;; to fill the cache for all subclasses of BASE-PRINT-OBJECT which +;;; have valid wrappers; however, in the course of doing so, the +;;; SUB-PRINT-OBJECT class gets finalized, which invalidates the +;;; SUBSUB-PRINT-OBJECT wrapper; if an invalid wrapper gets into a +;;; cache with more than one key, then failure ensues. +(reinitialize-instance #'print-object) + +;;; bug in long-form method combination: if there's an applicable +;;; method not part of any method group, we need to call +;;; INVALID-METHOD-ERROR. (MC27 test case from Bruno Haible) +(define-method-combination mc27 () + ((normal ()) + (ignored (:ignore :unused))) + `(list 'result + ,@(mapcar #'(lambda (method) `(call-method ,method)) normal))) +(defgeneric test-mc27 (x) + (:method-combination mc27) + (:method :ignore ((x number)) (/ 0))) +(assert (raises-error? (test-mc27 7))) + +(define-method-combination mc27prime () + ((normal ()) + (ignored (:ignore))) + `(list 'result ,@(mapcar (lambda (m) `(call-method ,m)) normal))) +(defgeneric test-mc27prime (x) + (:method-combination mc27prime) + (:method :ignore ((x number)) (/ 0))) +(assert (equal '(result) (test-mc27prime 3))) +(assert (raises-error? (test-mc27 t))) ; still no-applicable-method + +;;; more invalid wrappers. This time for a long-standing bug in the +;;; compiler's expansion for TYPEP on various class-like things, with +;;; user-visible consequences. +(defclass obsolete-again () ()) +(defvar *obsolete-again* (make-instance 'obsolete-again)) +(defvar *obsolete-again-hash* (sxhash *obsolete-again*)) +(make-instances-obsolete (find-class 'obsolete-again)) +(assert (not (streamp *obsolete-again*))) +(make-instances-obsolete (find-class 'obsolete-again)) +(assert (= (sxhash *obsolete-again*) *obsolete-again-hash*)) +(compile (defun is-a-structure-object-p (x) (typep x 'structure-object))) +(make-instances-obsolete (find-class 'obsolete-again)) +(assert (not (is-a-structure-object-p *obsolete-again*))) + +;;; overeager optimization of slot-valuish things +(defclass listoid () + ((caroid :initarg :caroid) + (cdroid :initarg :cdroid :initform nil))) +(defmethod lengthoid ((x listoid)) + (let ((result 0)) + (loop until (null x) + do (incf result) (setq x (slot-value x 'cdroid))) + result)) +(with-test (:name ((:setq :method-parameter) slot-value)) + (assert (= (lengthoid (make-instance 'listoid)) 1)) + (assert (= (lengthoid + (make-instance 'listoid :cdroid + (make-instance 'listoid :cdroid + (make-instance 'listoid)))) + 3))) + + + +;;;; Tests for argument parsing in fast-method-functions. + +(defvar *foo* 0) + +(eval-when (:compile-toplevel :load-toplevel :execute) + (setf (symbol-value 'a) 'invalid)) + +(defmacro test1 (lambda-list values args &key declarations cnm) + `(progn + (fmakunbound 'll-method) + (fmakunbound 'll-function) + (defmethod ll-method ,lambda-list + ,@declarations + ,@(when cnm + `((when nil (call-next-method)))) + (list ,@values)) + (defun ll-function ,lambda-list + ,@declarations + (list ,@values)) + (dotimes (i 2) + (assert (equal (ll-method ,@args) + (ll-function ,@args)))))) + +(defmacro test (&rest args) + `(progn + (test1 ,@args :cnm nil) + (test1 ,@args :cnm t))) + +;; Just plain arguments + +(test (a) (a) (1)) +(test (a b c d e f g h i) (a b c d e f g h i) (1 2 3 4 5 6 7 8 9)) + +(test (*foo*) (*foo* (symbol-value '*foo*)) (1)) + +(test (a) (a (symbol-value 'a)) (1) + :declarations ((declare (special a)))) + +;; Optionals + +(test (a &optional b c) (a b c) (1)) +(test (a &optional b c) (a b c) (1 2)) +(test (a &optional b c) (a b c) (1 2 3)) + +(test (a &optional (b 'b b-p) (c 'c c-p)) (a b c b-p c-p) (1)) +(test (a &optional (b 'b b-p) (c 'c c-p)) (a b c b-p c-p) (1 2)) +(test (a &optional (b 'b b-p) (c 'c c-p)) (a b c b-p c-p) (1 2 3)) + +(test (&optional *foo*) (*foo* (symbol-value '*foo*)) ()) +(test (&optional *foo*) (*foo* (symbol-value '*foo*)) (1)) + +(test (&optional (*foo* 'z foo-p)) (*foo* (symbol-value '*foo*) foo-p) ()) +(test (&optional (*foo* 'z foo-p)) (*foo* (symbol-value '*foo*) foo-p) (1)) + +(test (&optional a) (a (symbol-value 'a)) () + :declarations ((declare (special a)))) +(test (&optional a) (a (symbol-value 'a)) (1) + :declarations ((declare (special a)))) + +(test (&optional (a 'z a-p)) (a (symbol-value 'a) a-p) () + :declarations ((declare (special a)))) +(test (&optional (a 'z a-p)) (a (symbol-value 'a) a-p) (1) + :declarations ((declare (special a)))) + +(defparameter *count* 0) + +(test (&optional (a (incf *count*)) (b (incf *count*))) + (a b *count* (setf *count* 0)) + ()) + +;; Keywords with some &RESTs thrown in + +(dolist (args '((1) + (1 :b 2) + (1 :c 3) + (1 :b 2 :c 3) + (1 :c 3 :b 2) + (1 :c 3 :c 1 :b 2 :b 4))) + (eval `(test (a &key b c) (a b c) ,args)) + (eval `(test (a &key (b 'b b-p) (c 'c c-p)) + (a b c b-p c-p) + ,args)) + (eval `(test (a &rest rest &key (b 'b b-p) (c 'c c-p)) + (a b c b-p c-p rest) + ,args)) + (eval `(test (a &rest *foo* &key (b 'b b-p) (c 'c c-p)) + (a b c b-p c-p *foo* (symbol-value '*foo*)) + ,args)) + (eval `(test (a &rest *foo* &key (b 'b b-p) (c 'c c-p)) + (a b c b-p c-p *foo* (symbol-value '*foo*)) + ,args + :declarations ((declare (special b-p)))))) + +(dolist (args '(() + (:*foo* 1) + (:*foo* 1 :*foo* 2))) + (eval `(test (&key *foo*) (*foo* (symbol-value '*foo*)) ,args)) + (eval `(test (&key (*foo* 'z foo-p)) (*foo* (symbol-value '*foo*) foo-p) + ,args)) + (eval `(test (&key ((:*foo* a) 'z foo-p)) (a (symbol-value 'a) foo-p) + ,args)) + (eval `(test (&key ((:*foo* a) 'z foo-p)) (a (symbol-value 'a) foo-p) + ,args + :declarations ((declare (special a)))))) + +(defparameter *count* 0) + +(test (&key (a (incf *count*)) (b (incf *count*))) + (a b *count* (setf *count* 0)) + ()) + +(test (&key a b &allow-other-keys) (a b) (:a 1 :b 2 :c 3)) + +(defmethod clim-style-lambda-list-test (a b &optional c d &key x y) + (list a b c d x y)) + +(clim-style-lambda-list-test 1 2) + +(setf *count* 0) + +(test (&aux (a (incf *count*)) (b (incf *count*))) + (a b *count* (setf *count* 0)) + ()) + +;;;; long-form method combination with &rest in :arguments +;;;; (this had a bug what with fixed in 1.0.4.something) +(define-method-combination long-form-with-&rest () + ((methods *)) + (:arguments x &rest others) + `(progn + ,@(mapcar (lambda (method) + `(call-method ,method)) + methods) + (list ,x (length ,others)))) + +(defgeneric test-long-form-with-&rest (x &rest others) + (:method-combination long-form-with-&rest)) + +(defmethod test-long-form-with-&rest (x &rest others) + nil) + +(assert (equal '(:foo 13) + (apply #'test-long-form-with-&rest :foo (make-list 13)))) + +;;;; slot-missing for non-standard classes on SLOT-VALUE +;;;; +;;;; FIXME: This is arguably not right, actually: CLHS seems to say +;;;; we should just signal an error at least for built-in classes, but +;;;; for a while we were hitting NO-APPLICABLE-METHOD, which is definitely +;;;; wrong -- so test this for now at least. + +(defvar *magic-symbol* (gensym "MAGIC")) + +(set *magic-symbol* 42) + +(defmethod slot-missing (class instance (slot-name (eql *magic-symbol*)) op + &optional new) + (if (eq 'setf op) + (setf (symbol-value *magic-symbol*) new) + (symbol-value *magic-symbol*))) + +(assert (eql 42 (slot-value (cons t t) *magic-symbol*))) +(assert (eql 13 (setf (slot-value 123 *magic-symbol*) 13))) +(assert (eql 13 (slot-value 'foobar *magic-symbol*))) + +;;;; Built-in structure and condition layouts should have NIL in +;;;; LAYOUT-FOR-STD-CLASS-P, and classes should have T. + +(assert (not (sb-pcl::layout-for-std-class-p (sb-pcl::find-layout 'warning)))) +(assert (not (sb-pcl::layout-for-std-class-p (sb-pcl::find-layout 'hash-table)))) +(assert (eq t (sb-pcl::layout-for-std-class-p (sb-pcl::find-layout 'standard-object)))) + +;;;; bug 402: PCL used to warn about non-standard declarations +(declaim (declaration bug-402-d)) +(defgeneric bug-402-gf (x)) +(with-test (:name :bug-402) + (handler-bind ((warning #'error)) + (eval '(defmethod bug-402-gf (x) + (declare (bug-402-d x)) + x)))) + +;;;; non-keyword :default-initargs + :before method on shared initialize +;;;; interacted badly with CTOR optimizations +(defclass ctor-default-initarg-problem () + ((slot :initarg slotto)) + (:default-initargs slotto 123)) +(defmethod shared-initialize :before ((instance ctor-default-initarg-problem) slot-names &rest initargs) + (format t "~&Rock on: ~A~%" initargs)) +(defun provoke-ctor-default-initarg-problem () + (make-instance 'ctor-default-initarg-problem)) +(handler-bind ((warning #'error)) + (assert (= 123 (slot-value (provoke-ctor-default-initarg-problem) 'slot)))) + +;;;; discriminating net on streams used to generate code deletion notes on +;;;; first call +(defgeneric stream-fd (stream direction)) +(defmethod stream-fd ((stream sb-sys:fd-stream) direction) + (declare (ignore direction)) + (sb-sys:fd-stream-fd stream)) +(defmethod stream-fd ((stream synonym-stream) direction) + (stream-fd (symbol-value (synonym-stream-symbol stream)) direction)) +(defmethod stream-fd ((stream two-way-stream) direction) + (ecase direction + (:input + (stream-fd + (two-way-stream-input-stream stream) direction)) + (:output + (stream-fd + (two-way-stream-output-stream stream) direction)))) +(with-test (:name (:discriminating-name :code-deletion-note)) + (handler-bind ((compiler-note #'error)) + (stream-fd sb-sys:*stdin* :output) + (stream-fd sb-sys:*stdin* :output))) + +(with-test (:name :bug-380) + (defclass bug-380 () + ((slot :accessor bug380-slot))) + (fmakunbound 'foo-slot) + (defgeneric foo-slot (x y z)) + (defclass foo () + ((slot :accessor foo-slot-value)))) + +;;; SET and (SETF SYMBOL-VALUE) used to confuse permuation vector +;;; optimizations +(defclass fih () + ((x :initform :fih))) +(defclass fah () + ((x :initform :fah))) +(declaim (special *fih*)) +(defmethod fihfah ((*fih* fih)) + (set '*fih* (make-instance 'fah)) + (list (slot-value *fih* 'x) + (eval '(slot-value *fih* 'x)))) +(defmethod fihfah ((fah fah)) + (declare (special fah)) + (set 'fah (make-instance 'fih)) + (list (slot-value fah 'x) + (eval '(slot-value fah 'x)))) +(with-test (:name :set-of-a-method-specializer) + (assert (equal '(:fah :fah) (fihfah (make-instance 'fih)))) + (assert (equal '(:fih :fih) (fihfah (make-instance 'fah))))) + +(defmethod no-implicit-declarations-for-local-specials ((faax double-float)) + (declare (special faax)) + (set 'faax (when (< faax 0) (- faax))) + faax) +(with-test (:name :no-implicit-declarations-for-local-specials) + (assert (not (no-implicit-declarations-for-local-specials 1.0d0)))) + +(defstruct bug-357-a + slot1 + (slot2 t) + (slot3 (coerce pi 'single-float) :type single-float)) +(defclass bug-357-b (bug-357-a) + ((slot2 :initform 't2) + (slot4 :initform -44) + (slot5) + (slot6 :initform t) + (slot7 :initform (floor (* pi pi))) + (slot8 :initform 88)) + (:metaclass structure-class)) +(defstruct (bug-357-c (:include bug-357-b (slot8 -88) (slot5 :ok))) + slot9 + (slot10 t) + (slot11 (floor (exp 3)))) +(with-test (:name :bug-357) + (flet ((slots (x) + (list (bug-357-c-slot1 x) + (bug-357-c-slot2 x) + (bug-357-c-slot3 x) + (bug-357-c-slot4 x) + (bug-357-c-slot5 x) + (bug-357-c-slot6 x) + (bug-357-c-slot7 x) + (bug-357-c-slot8 x) + (bug-357-c-slot9 x) + (bug-357-c-slot10 x) + (bug-357-c-slot11 x)))) + (let ((base (slots (make-bug-357-c)))) + (assert (equal base (slots (make-instance 'bug-357-c)))) + (assert (equal base '(nil t2 3.1415927 -44 :ok t 9 -88 nil t 20)))))) + +(defclass class-slot-shared-initialize () + ((a :allocation :class :initform :ok))) +(with-test (:name :class-slot-shared-initialize) + (let ((x (make-instance 'class-slot-shared-initialize))) + (assert (eq :ok (slot-value x 'a))) + (slot-makunbound x 'a) + (assert (not (slot-boundp x 'a))) + (shared-initialize x '(a)) + (assert (slot-boundp x 'a)) + (assert (eq :ok (slot-value x 'a))))) + +(declaim (ftype (function (t t t) (values single-float &optional)) + i-dont-want-to-be-clobbered-1 + i-dont-want-to-be-clobbered-2)) +(defgeneric i-dont-want-to-be-clobbered-1 (t t t)) +(defmethod i-dont-want-to-be-clobbered-2 ((x cons) y z) + y) +(defun i-cause-an-gf-info-update () + (i-dont-want-to-be-clobbered-2 t t t)) +(with-test (:name :defgeneric-should-clobber-ftype) + ;; (because it doesn't check the argument or result types) + (assert (equal '(function (t t t) *) + (sb-kernel:type-specifier + (sb-int:info :function + :type 'i-dont-want-to-be-clobbered-1)))) + (assert (equal '(function (t t t) *) + (sb-kernel:type-specifier + (sb-int:info :function + :type 'i-dont-want-to-be-clobbered-2)))) + (assert (eq :defined-method + (sb-int:info :function + :where-from 'i-dont-want-to-be-clobbered-1))) + (assert (eq :defined-method + (sb-int:info :function + :where-from 'i-dont-want-to-be-clobbered-2)))) + +(with-test (:name :bogus-parameter-specializer-name-error) + (assert (eq :ok + (handler-case + (eval `(defmethod #:fii ((x "a string")) 'string)) + (sb-int:reference-condition (c) + (when (member '(:ansi-cl :macro defmethod) + (sb-int:reference-condition-references c) + :test #'equal) + :ok)))))) + +(defclass remove-default-initargs-test () + ((x :initarg :x :initform 42))) +(defclass remove-default-initatgs-test () + ((x :initarg :x :initform 42)) + (:default-initargs :x 0)) +(defclass remove-default-initargs-test () + ((x :initarg :x :initform 42))) +(with-test (:name :remove-default-initargs) + (assert (= 42 (slot-value (make-instance 'remove-default-initargs-test) + 'x)))) + +(with-test (:name :bug-485019) + ;; there was a bug in WALK-SETQ, used in method body walking, in the + ;; presence of declarations on symbol macros. + (defclass bug-485019 () + ((array :initarg :array))) + (defmethod bug-485019 ((bug-485019 bug-485019)) + (with-slots (array) bug-485019 + (declare (type (or null simple-array) array)) + (setf array (make-array 4))) + bug-485019) + (bug-485019 (make-instance 'bug-485019))) + +;;; The compiler didn't propagate the declarared type before applying +;;; the transform for (SETF SLOT-VALUE), so the generic accessor was used. +(defstruct foo-520366 + slot) +(defun quux-520366 (cont) + (funcall cont)) +(defun bar-520366 (foo-struct) + (declare (type foo-520366 foo-struct)) + (with-slots (slot) foo-struct + (tagbody + (quux-520366 #'(lambda () + (setf slot :value) + (go TAG))) + TAG))) +(with-test (:name :bug-520366) + (let ((callees (find-named-callees #'bar-520366))) + (assert (equal (list #'quux-520366) callees)))) + +(defgeneric no-applicable-method/retry (x)) +(defmethod no-applicable-method/retry ((x string)) + "string") +(with-test (:name :no-applicable-method/retry) + (assert (equal "cons" + (handler-bind ((error + (lambda (c) + (declare (ignore c)) + (let ((r (find-restart 'sb-pcl::retry))) + (when r + (eval `(defmethod no-applicable-method/retry ((x cons)) + "cons")) + (invoke-restart r)))))) + (no-applicable-method/retry (cons t t)))))) + +(defgeneric no-primary-method/retry (x)) +(defmethod no-primary-method/retry :before (x) (assert x)) +(with-test (:name :no-primary-method/retry) + (assert (equal "ok!" + (handler-bind ((error + (lambda (c) + (declare (ignore c)) + (let ((r (find-restart 'sb-pcl::retry))) + (when r + (eval `(defmethod no-primary-method/retry (x) + "ok!")) + (invoke-restart r)))))) + (no-primary-method/retry (cons t t)))))) + +;;; test that a cacheing strategy for make-instance initargs checking +;;; can handle class redefinitions +(defclass cacheing-initargs-redefinitions-check () + ((slot :initarg :slot))) +(defun cacheing-initargs-redefinitions-check-fun (&optional (initarg :slot)) + (declare (notinline make-instance)) + (make-instance 'cacheing-initargs-redefinitions-check) + (make-instance 'cacheing-initargs-redefinitions-check initarg 3)) +(with-test (:name :make-instance-initargs) + (make-instance 'cacheing-initargs-redefinitions-check) + (make-instance 'cacheing-initargs-redefinitions-check :slot 3) + (cacheing-initargs-redefinitions-check-fun :slot) + (assert (raises-error? (cacheing-initargs-redefinitions-check-fun :slot2)))) +(defclass cacheing-initargs-redefinitions-check () + ((slot :initarg :slot2))) +(with-test (:name :make-instance-redefined-initargs) + (make-instance 'cacheing-initargs-redefinitions-check) + (make-instance 'cacheing-initargs-redefinitions-check :slot2 3) + (cacheing-initargs-redefinitions-check-fun :slot2) + (assert (raises-error? (cacheing-initargs-redefinitions-check-fun :slot)))) +(defmethod initialize-instance :after ((class cacheing-initargs-redefinitions-check) &key slot) + nil) +(with-test (:name :make-instance-new-method-initargs) + (make-instance 'cacheing-initargs-redefinitions-check) + (make-instance 'cacheing-initargs-redefinitions-check :slot2 3) + (cacheing-initargs-redefinitions-check-fun :slot2) + (let ((thing (cacheing-initargs-redefinitions-check-fun :slot))) + (assert (not (slot-boundp thing 'slot))))) + +(with-test (:name :defmethod-specializer-builtin-class-alias) + (let ((alias (gensym))) + (setf (find-class alias) (find-class 'symbol)) + (eval `(defmethod lp-618387 ((s ,alias)) + (symbol-name s))) + (assert (equal "FOO" (funcall 'lp-618387 :foo))))) + +(with-test (:name :pcl-spurious-ignore-warnings) + (defgeneric no-spurious-ignore-warnings (req &key key)) + (handler-bind ((warning (lambda (x) (error "~A" x)))) + (eval + '(defmethod no-spurious-ignore-warnings ((req number) &key key) + (declare (ignore key)) + (check-type req integer)))) + (defgeneric should-get-an-ignore-warning (req &key key)) + (let ((warnings 0)) + (handler-bind ((warning (lambda (c) (setq warnings 1) (muffle-warning c)))) + (eval '(defmethod should-get-an-ignore-warning ((req integer) &key key) + (check-type req integer)))) + (assert (= warnings 1)))) + +(defgeneric generic-function-pretty-arglist-optional-and-key (req &optional opt &key key) + (:method (req &optional opt &key key) + (list req opt key))) + +(with-test (:name :generic-function-pretty-arglist-optional-and-key) + (handler-bind ((warning #'error)) + ;; Used to signal a style-warning + (assert (equal '(req &optional opt &key key) + (sb-pcl::generic-function-pretty-arglist + #'generic-function-pretty-arglist-optional-and-key))))) + +(with-test (:name :bug-894202) + (assert (eq :good + (handler-case + (let ((name (gensym "FOO")) + (decl (gensym "BAR"))) + (eval `(defgeneric ,name () + (declare (,decl))))) + (warning () + :good))))) + +(with-test (:name :bug-898331) + (handler-bind ((warning #'error)) + (eval `(defgeneric bug-898331 (request type remaining-segment-requests all-requests))) + (eval `(defmethod bug-898331 ((request cons) (type (eql :cancel)) + remaining-segment-requests + all-segment-requests) + (declare (ignore all-segment-requests)) + (check-type request t))))) + +(with-test (:name :bug-1001799) + ;; compilation of the defmethod used to cause infinite recursion + (let ((pax (gensym "PAX")) + (pnr (gensym "PNR")) + (sup (gensym "SUP")) + (frob (gensym "FROB")) + (sb-ext:*evaluator-mode* :compile)) + (eval + `(progn + (declaim (optimize (speed 1) (space 1) (safety 3) (debug 3) (compilation-speed 1))) + (defclass ,pax (,sup) + ((,pnr :type (or null ,pnr)))) + (defclass ,pnr (,sup) + ((,pax :type (or null ,pax)))) + (defclass ,sup () + ()) + (defmethod ,frob ((pnr ,pnr)) + (slot-value pnr ',pax)))))) + +(with-test (:name :bug-1099708) + (defclass bug-1099708 () + ((slot-1099708 :initarg :slot-1099708))) + ;; caused infinite equal testing in function name lookup + (eval + '(progn + (defun make-1099708-1 () + (make-instance 'bug-1099708 :slot-1099708 '#1= (1 2 . #1#))) + (defun make-1099708-2 () + (make-instance 'bug-1099708 :slot-1099708 '#2= (1 2 . #2#))))) + (assert (not (eql (slot-value (make-1099708-1) 'slot-1099708) + (slot-value (make-1099708-2) 'slot-1099708))))) + +(with-test (:name :bug-1099708b-list) + (defclass bug-1099708b-list () + ((slot-1099708b-list :initarg :slot-1099708b-list))) + (eval + '(progn + (defun make-1099708b-list-1 () + (make-instance 'bug-1099708b-list :slot-1099708b-list '(some value))) + (defun make-1099708b-list-2 () + (make-instance 'bug-1099708b-list :slot-1099708b-list '(some value))))) + (assert (eql (slot-value (make-1099708b-list-1) 'slot-1099708b-list) + (slot-value (make-1099708b-list-1) 'slot-1099708b-list))) + (assert (eql (slot-value (make-1099708b-list-2) 'slot-1099708b-list) + (slot-value (make-1099708b-list-2) 'slot-1099708b-list))) + (assert (not (eql (slot-value (make-1099708b-list-1) 'slot-1099708b-list) + (slot-value (make-1099708b-list-2) 'slot-1099708b-list))))) + +(with-test (:name :bug-1099708b-string) + (defclass bug-1099708b-string () + ((slot-1099708b-string :initarg :slot-1099708b-string))) + (eval + '(progn + (defun make-1099708b-string-1 () + (make-instance 'bug-1099708b-string :slot-1099708b-string "string")) + (defun make-1099708b-string-2 () + (make-instance 'bug-1099708b-string :slot-1099708b-string "string")))) + (assert (eql (slot-value (make-1099708b-string-1) 'slot-1099708b-string) + (slot-value (make-1099708b-string-1) 'slot-1099708b-string))) + (assert (eql (slot-value (make-1099708b-string-2) 'slot-1099708b-string) + (slot-value (make-1099708b-string-2) 'slot-1099708b-string))) + (assert (not (eql (slot-value (make-1099708b-string-1) 'slot-1099708b-string) + (slot-value (make-1099708b-string-2) 'slot-1099708b-string))))) + +(with-test (:name :bug-1099708b-bitvector) + (defclass bug-1099708b-bitvector () + ((slot-1099708b-bitvector :initarg :slot-1099708b-bitvector))) + (eval + '(progn + (defun make-1099708b-bitvector-1 () + (make-instance 'bug-1099708b-bitvector :slot-1099708b-bitvector #*1011)) + (defun make-1099708b-bitvector-2 () + (make-instance 'bug-1099708b-bitvector :slot-1099708b-bitvector #*1011)))) + (assert (eql (slot-value (make-1099708b-bitvector-1) 'slot-1099708b-bitvector) + (slot-value (make-1099708b-bitvector-1) 'slot-1099708b-bitvector))) + (assert (eql (slot-value (make-1099708b-bitvector-2) 'slot-1099708b-bitvector) + (slot-value (make-1099708b-bitvector-2) 'slot-1099708b-bitvector))) + (assert (not (eql (slot-value (make-1099708b-bitvector-1) 'slot-1099708b-bitvector) + (slot-value (make-1099708b-bitvector-2) 'slot-1099708b-bitvector))))) + +(with-test (:name :bug-1099708b-pathname) + (defclass bug-1099708b-pathname () + ((slot-1099708b-pathname :initarg :slot-1099708b-pathname))) + (eval + '(progn + (defun make-1099708b-pathname-1 () + (make-instance 'bug-1099708b-pathname :slot-1099708b-pathname #p"pn")) + (defun make-1099708b-pathname-2 () + (make-instance 'bug-1099708b-pathname :slot-1099708b-pathname #p"pn")))) + (assert (eql (slot-value (make-1099708b-pathname-1) 'slot-1099708b-pathname) + (slot-value (make-1099708b-pathname-1) 'slot-1099708b-pathname))) + (assert (eql (slot-value (make-1099708b-pathname-2) 'slot-1099708b-pathname) + (slot-value (make-1099708b-pathname-2) 'slot-1099708b-pathname))) + (assert (not (eql (slot-value (make-1099708b-pathname-1) 'slot-1099708b-pathname) + (slot-value (make-1099708b-pathname-2) 'slot-1099708b-pathname))))) + +(with-test (:name :bug-1099708c-list) + (defclass bug-1099708c-list () + ((slot-1099708c-list :initarg :slot-1099708c-list))) + (eval + '(progn + (defun make-1099708c-list-1 () + (make-instance 'bug-1099708c-list :slot-1099708c-list #1='(some value))) + (defun make-1099708c-list-2 () + (make-instance 'bug-1099708c-list :slot-1099708c-list #1#)))) + (assert (eql (slot-value (make-1099708c-list-1) 'slot-1099708c-list) + (slot-value (make-1099708c-list-1) 'slot-1099708c-list))) + (assert (eql (slot-value (make-1099708c-list-2) 'slot-1099708c-list) + (slot-value (make-1099708c-list-2) 'slot-1099708c-list))) + (assert (eql (slot-value (make-1099708c-list-1) 'slot-1099708c-list) + (slot-value (make-1099708c-list-2) 'slot-1099708c-list)))) + +;;; bug-1179858 + +;;; Define a class and force the "fallback" constructor generator to be +;;; used by having a HAIRY-AROUND-OR-NONSTANDARD-PRIMARY-METHOD-P on +;;; SHARED-INITIALIZE. +(defclass bug-1179858 () + ((foo :initarg :foo :reader bug-1179858-foo)) + (:default-initargs :foo (error "Should not be evaluated"))) +(defmethod shared-initialize :around ((instance bug-1179858) (slot-names t) &key) + (call-next-method)) + +(with-test (:name (:make-instance :fallback-generator-initarg-handling :bug-1179858)) + ;; Now compile a lambda containing MAKE-INSTANCE to exercise the + ;; fallback constructor generator. Call the resulting compiled + ;; function to trigger the bug. + (funcall (compile nil '(lambda () (make-instance 'bug-1179858 :foo t))))) + +;;; Other brokenness, found while investigating: fallback-generator +;;; handling of non-keyword initialization arguments +(defclass bug-1179858b () + ((foo :initarg foo :reader bug-1179858b-foo)) + (:default-initargs foo 14)) +(defmethod shared-initialize :around ((instance bug-1179858b) (slot-names t) &key) + (call-next-method)) + +(with-test (:name (:make-instance :fallback-generator-non-keyword-initarg :bug-1179858)) + (flet ((foo= (n i) (= (bug-1179858b-foo i) n))) + (assert + (foo= 14 (funcall (compile nil '(lambda () (make-instance 'bug-1179858b)))))) + (assert + (foo= 15 (funcall (compile nil '(lambda () (make-instance 'bug-1179858b 'foo 15)))))))) + +(with-test (:name (:cpl-violation-setup :bug-309076)) + (assert (raises-error? + (progn + (defclass bug-309076-broken-class (standard-class) () + (:metaclass sb-mop:funcallable-standard-class)) + (sb-mop:finalize-inheritance (find-class 'bug-309076-broken-class)))))) + +(with-test (:name (:cpl-violation-irrelevant-class :bug-309076)) + (defclass bug-309076-class (standard-class) ()) + (defmethod sb-mop:validate-superclass ((x bug-309076-class) (y standard-class)) t) + (assert (typep (make-instance 'bug-309076-class) 'bug-309076-class))) ;;;; success -(sb-ext:quit :unix-status 104)