X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fcode%2Fcondition.lisp;h=ee00b04325d42faa6dba35968f8885b760a716b9;hb=c40c56eedec692acb930ec0e358a933ab33a3560;hp=594b3ac33eb2874517ad399d5c82a9058df2de43;hpb=ef43bf713ac67d3acf4727a5111567e70675df44;p=sbcl.git diff --git a/src/code/condition.lisp b/src/code/condition.lisp index 594b3ac..ee00b04 100644 --- a/src/code/condition.lisp +++ b/src/code/condition.lisp @@ -171,7 +171,10 @@ ;;; The current code doesn't seem to quite match that. (def!method print-object ((x condition) stream) (if *print-escape* - (print-unreadable-object (x stream :type t :identity t)) + (if (and (typep x 'simple-condition) (slot-boundp x 'format-control)) + (print-unreadable-object (x stream :type t :identity t) + (format stream "~S" (simple-condition-format-control x))) + (print-unreadable-object (x stream :type t :identity t))) ;; KLUDGE: A comment from CMU CL here said ;; 7/13/98 BUG? CPL is not sorted and results here depend on order of ;; superclasses in define-condition call! @@ -576,20 +579,6 @@ ',(all-writers) (sb!c:source-location))))))) -;;;; DESCRIBE on CONDITIONs - -;;; a function to be used as the guts of DESCRIBE-OBJECT (CONDITION T) -;;; eventually (once we get CLOS up and running so that we can define -;;; methods) -(defun describe-condition (condition stream) - (format stream - "~&~@<~S ~_is a ~S. ~_Its slot values are ~_~S.~:>~%" - condition - (type-of condition) - (concatenate 'list - (condition-actual-initargs condition) - (condition-assigned-slots condition)))) - ;;;; various CONDITIONs specified by ANSI (define-condition serious-condition (condition) ()) @@ -648,6 +637,11 @@ "end of file on ~S" (stream-error-stream condition))))) +(define-condition closed-stream-error (stream-error) () + (:report + (lambda (condition stream) + (format stream "~S is closed" (stream-error-stream condition))))) + (define-condition file-error (error) ((pathname :reader file-error-pathname :initarg :pathname)) (:report @@ -903,6 +897,9 @@ (define-condition simple-reference-error (reference-condition simple-error) ()) +(define-condition simple-reference-warning (reference-condition simple-warning) + ()) + (define-condition duplicate-definition (reference-condition warning) ((name :initarg :name :reader duplicate-definition-name)) (:report (lambda (c s) @@ -946,15 +943,12 @@ '(:ansi-cl :function make-array) '(:ansi-cl :function sb!xc:upgraded-array-element-type)))) -(define-condition displaced-to-array-too-small-error - (reference-condition simple-error) - () - (:default-initargs - :references (list '(:ansi-cl :function adjust-array)))) - (define-condition type-warning (reference-condition simple-warning) () (:default-initargs :references (list '(:sbcl :node "Handling of Types")))) +(define-condition type-style-warning (reference-condition simple-style-warning) + () + (:default-initargs :references (list '(:sbcl :node "Handling of Types")))) (define-condition local-argument-mismatch (reference-condition simple-warning) () @@ -1093,6 +1087,36 @@ SB-EXT:PACKAGE-LOCKED-ERROR-SYMBOL.")) "No traps are enabled? How can this be?" stream)))))) +(define-condition invalid-array-index-error (type-error) + ((array :initarg :array :reader invalid-array-index-error-array) + (axis :initarg :axis :reader invalid-array-index-error-axis)) + (:report + (lambda (condition stream) + (let ((array (invalid-array-index-error-array condition))) + (format stream "Index ~W out of bounds for ~@[axis ~W of ~]~S, ~ + should be nonnegative and <~W." + (type-error-datum condition) + (when (> (array-rank array) 1) + (invalid-array-index-error-axis condition)) + (type-of array) + ;; Extract the bound from (INTEGER 0 (BOUND)) + (caaddr (type-error-expected-type condition))))))) + +(define-condition invalid-array-error (reference-condition type-error) () + (:report + (lambda (condition stream) + (let ((*print-array* nil)) + (format stream + "~@" + (type-error-expected-type condition) + (array-displacement (type-error-datum condition)))))) + (:default-initargs + :references + (list '(:ansi-cl :function adjust-array)))) + (define-condition index-too-large-error (type-error) () (:report @@ -1177,6 +1201,25 @@ SB-EXT:PACKAGE-LOCKED-ERROR-SYMBOL.")) (simple-condition-format-arguments condition) (reader-impossible-number-error-error condition)))))) +(define-condition standard-readtable-modified-error (reference-condition error) + ((operation :initarg :operation :reader standard-readtable-modified-operation)) + (:report (lambda (condition stream) + (format stream "~S would modify the standard readtable." + (standard-readtable-modified-operation condition)))) + (:default-initargs :references `((:ansi-cl :section (2 1 1 2)) + (:ansi-cl :glossary "standard readtable")))) + +(define-condition standard-pprint-dispatch-table-modified-error + (reference-condition error) + ((operation :initarg :operation + :reader standard-pprint-dispatch-table-modified-operation)) + (:report (lambda (condition stream) + (format stream "~S would modify the standard pprint dispatch table." + (standard-pprint-dispatch-table-modified-operation + condition)))) + (:default-initargs + :references `((:ansi-cl :glossary "standard pprint dispatch table")))) + (define-condition timeout (serious-condition) ((seconds :initarg :seconds :initform nil :reader timeout-seconds)) (:report (lambda (condition stream) @@ -1241,14 +1284,6 @@ holds the source-path to the original form within that file or NIL. Associated with this condition are always the restarts STEP-INTO, STEP-NEXT, and STEP-CONTINUE.")) -#!+sb-doc -(setf (fdocumentation 'step-condition-source-path 'function) - "Source-path of the original form associated with the -STEP-FORM-CONDITION or NIL." - (fdocumentation 'step-condition-pathname 'function) - "Pathname of the original source-file associated with the -STEP-FORM-CONDITION or NIL.") - (define-condition step-result-condition (step-condition) ((result :initarg :result :reader step-condition-result))) @@ -1273,7 +1308,308 @@ the values returned by the form as a list. No associated restarts.")) (format stream "Returning from STEP"))) #!+sb-doc (:documentation "Condition signaled when STEP returns.")) + +;;; A knob for muffling warnings, mostly for use while loading files. +(defvar *muffled-warnings* 'uninteresting-redefinition + "A type that ought to specify a subtype of WARNING. Whenever a +warning is signaled, if the warning if of this type and is not +handled by any other handler, it will be muffled.") + +;;; Various STYLE-WARNING signaled in the system. +;; For the moment, we're only getting into the details for function +;; redefinitions, but other redefinitions could be done later +;; (e.g. methods). +(define-condition redefinition-warning (style-warning) + ()) +(define-condition function-redefinition-warning (redefinition-warning) + ((name :initarg :name :reader function-redefinition-warning-name) + (old :initarg :old :reader function-redefinition-warning-old-fdefinition) + ;; For DEFGENERIC and perhaps others, the redefinition + ;; destructively modifies the original, rather than storing a new + ;; object, so there's no NEW here, but only in subclasses. + )) + +(define-condition redefinition-with-defun (function-redefinition-warning) + ((new :initarg :new :reader redefinition-with-defun-new-fdefinition) + ;; KLUDGE: it would be nice to fix the unreasonably late + ;; back-patching of DEBUG-SOURCEs in the DEBUG-INFO during + ;; fasloading and just use the new fdefinition, but for the moment + ;; we'll compare the SOURCE-LOCATION created during DEFUN with the + ;; previous DEBUG-SOURCE. + (new-location :initarg :new-location + :reader redefinition-with-defun-new-location)) + (:report (lambda (warning stream) + (format stream "redefining ~S in DEFUN" + (function-redefinition-warning-name warning))))) + +(define-condition redefinition-with-defgeneric (function-redefinition-warning) + ((new-location :initarg :new-location + :reader redefinition-with-defgeneric-new-location)) + (:report (lambda (warning stream) + (format stream "redefining ~S in DEFGENERIC" + (function-redefinition-warning-name warning))))) + +(define-condition redefinition-with-defmethod (redefinition-warning) + ((gf :initarg :generic-function + :reader redefinition-with-defmethod-generic-function) + (qualifiers :initarg :qualifiers + :reader redefinition-with-defmethod-qualifiers) + (specializers :initarg :specializers + :reader redefinition-with-defmethod-specializers) + (new-location :initarg :new-location + :reader redefinition-with-defmethod-new-location) + (old-method :initarg :old-method + :reader redefinition-with-defmethod-old-method)) + (:report (lambda (warning stream) + (format stream "redefining ~S~{ ~S~} ~S in DEFMETHOD" + (redefinition-with-defmethod-generic-function warning) + (redefinition-with-defmethod-qualifiers warning) + (redefinition-with-defmethod-specializers warning))))) + +;; FIXME: see the FIXMEs in defmacro.lisp, then maybe instantiate this. +(define-condition redefinition-with-defmacro (function-redefinition-warning) + ()) + +;; Here are a few predicates for what people might find interesting +;; about redefinitions. + +;; DEFUN can replace a generic function with an ordinary function. +;; (Attempting to replace an ordinary function with a generic one +;; causes an error, though.) +(defun redefinition-replaces-generic-function-p (warning) + (and (typep warning 'redefinition-with-defun) + (typep (function-redefinition-warning-old-fdefinition warning) + 'generic-function))) + +(defun redefinition-replaces-compiled-function-with-interpreted-p (warning) + (and (typep warning 'redefinition-with-defun) + (compiled-function-p + (function-redefinition-warning-old-fdefinition warning)) + (not (compiled-function-p + (redefinition-with-defun-new-fdefinition warning))))) + +;; Most people seem to agree that re-running a DEFUN in a file is +;; completely uninteresting. +(defun uninteresting-ordinary-function-redefinition-p (warning) + ;; OAOO violation: this duplicates code in SB-INTROSPECT. + ;; Additionally, there are some functions that aren't + ;; funcallable-instances for which finding the source location is + ;; complicated (e.g. DEFSTRUCT-defined predicates and accessors), + ;; but I don't think they're defined with %DEFUN, so the warning + ;; isn't raised. + (flet ((fdefinition-file-namestring (fdefn) + #!+sb-eval + (when (typep fdefn 'sb!eval:interpreted-function) + (return-from fdefinition-file-namestring + (sb!c:definition-source-location-namestring + (sb!eval:interpreted-function-source-location fdefn)))) + ;; All the following accesses are guarded with conditionals + ;; because it's not clear whether any of the slots we're + ;; chasing down are guaranteed to be filled in. + (let* ((fdefn + ;; KLUDGE: although this looks like it only works + ;; for %SIMPLE-FUNs, in fact there's a pun such + ;; that %SIMPLE-FUN-SELF returns the simple-fun + ;; object for closures and + ;; funcallable-instances. -- CSR, circa 2005 + (sb!kernel:%simple-fun-self fdefn)) + (code (if fdefn (sb!kernel:fun-code-header fdefn))) + (debug-info (if code (sb!kernel:%code-debug-info code))) + (debug-source (if debug-info + (sb!c::debug-info-source debug-info))) + (namestring (if debug-source + (sb!c::debug-source-namestring debug-source)))) + namestring))) + (and + ;; There's garbage in various places when the first DEFUN runs in + ;; cold-init. + sb!kernel::*cold-init-complete-p* + (typep warning 'redefinition-with-defun) + (let ((old-fdefn + (function-redefinition-warning-old-fdefinition warning)) + (new-fdefn + (redefinition-with-defun-new-fdefinition warning))) + ;; Replacing a compiled function with a compiled function is + ;; clearly uninteresting, and we'll say arbitrarily that + ;; replacing an interpreted function with an interpreted + ;; function is uninteresting, too, but leave out the + ;; compiled-to-interpreted case. + (when (or (typep + old-fdefn + '(or #!+sb-eval sb!eval:interpreted-function)) + (and (typep old-fdefn + '(and compiled-function + (not funcallable-instance))) + ;; Since this is a REDEFINITION-WITH-DEFUN, + ;; NEW-FDEFN can't be a FUNCALLABLE-INSTANCE. + (typep new-fdefn 'compiled-function))) + (let* ((old-namestring (fdefinition-file-namestring old-fdefn)) + (new-namestring + (or (fdefinition-file-namestring new-fdefn) + (let ((srcloc + (redefinition-with-defun-new-location warning))) + (if srcloc + (sb!c::definition-source-location-namestring + srcloc)))))) + (and old-namestring + new-namestring + (equal old-namestring new-namestring)))))))) + +(defun uninteresting-generic-function-redefinition-p (warning) + (and (typep warning 'redefinition-with-defgeneric) + (let* ((old-fdefn + (function-redefinition-warning-old-fdefinition warning)) + (old-location + (if (typep old-fdefn 'generic-function) + (sb!pcl::definition-source old-fdefn))) + (old-namestring + (if old-location + (sb!c:definition-source-location-namestring old-location))) + (new-location + (redefinition-with-defgeneric-new-location warning)) + (new-namestring + (if new-location + (sb!c:definition-source-location-namestring new-location)))) + (and old-namestring + new-namestring + (equal old-namestring new-namestring))))) + +(defun uninteresting-method-redefinition-p (warning) + (and (typep warning 'redefinition-with-defmethod) + (let* ((old-method (redefinition-with-defmethod-old-method warning)) + (old-location (sb!pcl::definition-source old-method)) + (old-namestring (if old-location + (sb!c:definition-source-location-namestring + old-location))) + (new-location (redefinition-with-defmethod-new-location warning)) + (new-namestring (if new-location + (sb!c:definition-source-location-namestring + new-location)))) + (and new-namestring + old-namestring + (equal new-namestring old-namestring))))) + +(deftype uninteresting-redefinition () + '(or (satisfies uninteresting-ordinary-function-redefinition-p) + (satisfies uninteresting-generic-function-redefinition-p) + (satisfies uninteresting-method-redefinition-p))) + +(define-condition redefinition-with-deftransform (redefinition-warning) + ((transform :initarg :transform + :reader redefinition-with-deftransform-transform)) + (:report (lambda (warning stream) + (format stream "Overwriting ~S" + (redefinition-with-deftransform-transform warning))))) + +;;; Various other STYLE-WARNINGS +(define-condition dubious-asterisks-around-variable-name + (style-warning simple-condition) + () + (:report (lambda (warning stream) + (format stream "~@?, even though the name follows~@ +the usual naming convention (names like *FOO*) for special variables" + (simple-condition-format-control warning) + (simple-condition-format-arguments warning))))) + +(define-condition asterisks-around-lexical-variable-name + (dubious-asterisks-around-variable-name) + ()) + +(define-condition asterisks-around-constant-variable-name + (dubious-asterisks-around-variable-name) + ()) + +;; We call this UNDEFINED-ALIEN-STYLE-WARNING because there are some +;; subclasses of ERROR above having to do with undefined aliens. +(define-condition undefined-alien-style-warning (style-warning) + ((symbol :initarg :symbol :reader undefined-alien-symbol)) + (:report (lambda (warning stream) + (format stream "Undefined alien: ~S" + (undefined-alien-symbol warning))))) + +#!+sb-eval +(define-condition lexical-environment-too-complex (style-warning) + ((form :initarg :form :reader lexical-environment-too-complex-form) + (lexenv :initarg :lexenv :reader lexical-environment-too-complex-lexenv)) + (:report (lambda (warning stream) + (format stream + "~@" + (lexical-environment-too-complex-form warning) + (lexical-environment-too-complex-lexenv warning))))) + +;; Although this has -ERROR- in the name, it's just a STYLE-WARNING. +(define-condition character-decoding-error-in-comment (style-warning) + ((stream :initarg :stream :reader decoding-error-in-comment-stream) + (position :initarg :position :reader decoding-error-in-comment-position)) + (:report (lambda (warning stream) + (format stream + "Character decoding error in a ~A-comment at ~ + position ~A reading source stream ~A, ~ + resyncing." + (decoding-error-in-comment-macro warning) + (decoding-error-in-comment-position warning) + (decoding-error-in-comment-stream warning))))) + +(define-condition character-decoding-error-in-macro-char-comment + (character-decoding-error-in-comment) + ((char :initform #\; :initarg :char + :reader character-decoding-error-in-macro-char-comment-char))) + +(define-condition character-decoding-error-in-dispatch-macro-char-comment + (character-decoding-error-in-comment) + ;; ANSI doesn't give a way for a reader function invoked by a + ;; dispatch macro character to determine which dispatch character + ;; was used, so if a user wants to signal one of these from a custom + ;; comment reader, he'll have to supply the :DISP-CHAR himself. + ((disp-char :initform #\# :initarg :disp-char + :reader character-decoding-error-in-macro-char-comment-disp-char) + (sub-char :initarg :sub-char + :reader character-decoding-error-in-macro-char-comment-sub-char))) + +(defun decoding-error-in-comment-macro (warning) + (etypecase warning + (character-decoding-error-in-macro-char-comment + (character-decoding-error-in-macro-char-comment-char warning)) + (character-decoding-error-in-dispatch-macro-char-comment + (format + nil "~C~C" + (character-decoding-error-in-macro-char-comment-disp-char warning) + (character-decoding-error-in-macro-char-comment-sub-char warning))))) + +(define-condition deprecated-eval-when-situations (style-warning) + ((situations :initarg :situations + :reader deprecated-eval-when-situations-situations)) + (:report (lambda (warning stream) + (format stream "using deprecated EVAL-WHEN situation names~{ ~S~}" + (deprecated-eval-when-situations-situations warning))))) + +(define-condition proclamation-mismatch (style-warning) + ((name :initarg :name :reader proclamation-mismatch-name) + (old :initarg :old :reader proclamation-mismatch-old) + (new :initarg :new :reader proclamation-mismatch-new))) + +(define-condition type-proclamation-mismatch (proclamation-mismatch) + () + (:report (lambda (warning stream) + (format stream + "The new TYPE proclamation~% ~S for ~S does not ~ + match the old TYPE proclamation ~S" + (proclamation-mismatch-new warning) + (proclamation-mismatch-name warning) + (proclamation-mismatch-old warning))))) + +(define-condition ftype-proclamation-mismatch (proclamation-mismatch) + () + (:report (lambda (warning stream) + (format stream + "The new FTYPE proclamation~% ~S for ~S does not ~ + match the old FTYPE proclamation ~S" + (proclamation-mismatch-new warning) + (proclamation-mismatch-name warning) + (proclamation-mismatch-old warning))))) ;;;; restart definitions