X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=tests%2Finterface.pure.lisp;h=b3f8949ac87ab6d54712e207d2cbbaac86769712;hb=3bbbfec26f90bc005c2ff5c8de0ceb0f95f1a52b;hp=1986498a761890c5325d41d6d244d59f73f2c73e;hpb=f17811c866c0412da17d3ee94f11cf38783301f7;p=sbcl.git

diff --git a/tests/interface.pure.lisp b/tests/interface.pure.lisp
index 1986498..b3f8949 100644
--- a/tests/interface.pure.lisp
+++ b/tests/interface.pure.lisp
@@ -1,3 +1,5 @@
+;;;; tests for problems in the interface presented to the user/programmer
+
 ;;;; This software is part of the SBCL system. See the README file for
 ;;;; more information.
 ;;;;
@@ -10,70 +12,18 @@
 ;;;; more information.
 
 (in-package :cl-user)
-
-;;; Check for fbound external symbols in public packages that have no
-;;; argument list information. (This can happen if we get carried away
-;;; with byte compilation, since at least in sbcl-0.6.12 the byte
-;;; compiler can't record argument list information.)
-(defvar *public-package-names*
-  '("SB-ALIEN" "SB-C-CALL" "SB-DEBUG" "SB-EXT" "SB-GRAY" "SB-MP"
-    "SB-PROFILE" "SB-PCL" "COMMON-LISP"))
-(defun has-arglist-info-p (function)
-  (declare (type function function))
-  ;; The Lisp-level type FUNCTION can conceal a multitude of sins..
-  (case (sb-kernel:get-type function)
-    (#.sb-vm:function-header-type (sb-kernel:%function-arglist function))
-    (#.sb-vm:closure-function-header-type (has-arglist-info-p
-					   (sb-kernel:%closure-function
-					    function)))
-    ;; (There might be other cases with arglist info also.
-    ;; FUNCTION-HEADER-TYPE and CLOSURE-FUNCTION-HEADER-TYPE just
-    ;; happen to be the two case that I had my nose rubbed in when
-    ;; debugging a GC problem caused by applying %FUNCTION-ARGLIST to
-    ;; a closure. -- WHN 2001-06-05)
-    (t nil)))
-(defun check-ext-symbols-arglist (package)
-  (format t "~% looking at package: ~A" package)
-  (do-external-symbols (ext-sym package)
-    (when (fboundp ext-sym)
-      (let ((fun (symbol-function ext-sym)))
-	(cond ((macro-function ext-sym)
-	       ;; FIXME: Macro functions should have their argument list
-	       ;; information checked separately. Just feeding them into
-	       ;; the ordinary-function logic below doesn't work right,
-	       ;; though, and I haven't figured out what does work
-	       ;; right. For now we just punt.
-	       (values))
-	      #+nil 
-	      ((sb-int:info :function :accessor-for ext-sym)
-	       (values))
-	      ((typep fun 'generic-function)
-	       ;; FIXME: Check the argument lists of generic functions,
-	       ;; instead of just punting like this. (DESCRIBE seems
-	       ;; to know how to do it, at least for #'DOCUMENTATION.)
-	       (values))
-	      (;; FIXME: CONDITION slot accessors (e.g.
-	       ;; PRINT-NOT-READABLE-OBJECT) fall into this category.
-	       ;; They seem to have argument lists -- since at least
-	       ;; DESCRIBE knows how to find them -- but I have
-	       ;; neither reverse engineered how it's finding them,
-	       ;; nor factored that into a function which can be
-	       ;; shared with the logic here..
-	       (= (sb-kernel:get-type fun) sb-vm:closure-header-type)
-	       (values)) ; ..so for now we just punt.
-	      (t
-	       (let ((fun (symbol-function ext-sym)))
-		 (unless (has-arglist-info-p fun)
-		   (error "Function ~A has no arg-list information available."
-			  ext-sym)))))))))
-(dolist (public-package *public-package-names*)
-  (when (find-package public-package)
-    (check-ext-symbols-arglist public-package)))
-(terpri)
+
+;;;; properties of symbols, e.g. presence of doc strings for public symbols
 
 ;;; FIXME: It would probably be good to require here that every
 ;;; external symbol either has a doc string or has some good excuse
 ;;; (like being an accessor for a structure which has a doc string).
+
+;;;; tests of interface machinery
 
-(print "done with interface.pure.lisp")
-
+;;; APROPOS should accept a package designator, not just a package, and
+;;; furthermore do the right thing when it gets a package designator.
+;;; (bug reported and fixed by Alexey Dejneka sbcl-devel 2001-10-17)
+(assert (< 0
+	   (length (apropos-list "PRINT" :cl))
+	   (length (apropos-list "PRINT"))))