1.0.18.10: Record filenames in DEBUG-SOURCEs during EVAL-WHEN, LOAD.
[sbcl.git] / src / code / target-load.lisp
index ed607ff..219aba0 100644 (file)
 \f
 ;;;; LOAD-AS-SOURCE
 
-;;; Load a text file.  (Note that load-as-fasl is in another file.)
+;;; Load a text stream.  (Note that load-as-fasl is in another file.)
 (defun load-as-source (stream verbose print)
   (maybe-announce-load stream verbose)
-  (do ((sexpr (read stream nil *eof-object*)
-              (read stream nil *eof-object*)))
-      ((eq sexpr *eof-object*)
-       t)
-    (if print
-        (let ((results (multiple-value-list (eval sexpr))))
-          (load-fresh-line)
-          (format t "~{~S~^, ~}~%" results))
-      (eval sexpr))))
+  (macrolet ((do-sexprs ((sexpr stream) &body body)
+               (aver (symbolp sexpr))
+               (with-unique-names (source-info)
+                 (once-only ((stream stream))
+                   `(if (handler-case (pathname stream)
+                          (error () nil))
+                        (let ((,source-info (sb!c::make-file-source-info
+                                            (pathname ,stream)
+                                            (stream-external-format ,stream))))
+                          (setf (sb!c::source-info-stream ,source-info) ,stream)
+                          (sb!c::do-forms-from-info ((,sexpr) ,source-info)
+                            ,@body))
+                        (do ((,sexpr (read ,stream nil *eof-object*)
+                                     (read ,stream nil *eof-object*)))
+                            ((eq ,sexpr *eof-object*))
+                          ,@body))))))
+    (do-sexprs (sexpr stream)
+      (if print
+          (let ((results (multiple-value-list (eval sexpr))))
+            (load-fresh-line)
+            (format t "~{~S~^, ~}~%" results))
+          (eval sexpr)))
+    t))
 \f
 ;;;; LOAD itself
 
              (invalid-fasl-expected condition)
              (invalid-fasl-fhsss condition)))))
 
-;; Pretty well any way of doing LOAD will expose race conditions: for
-;; example, a file might get deleted or renamed after we open it but
-;; before we find its truename.  It seems useful to say that
-;; detectible ways the file system can fail to be static are good
-;; enough reason to stop loading, but to stop in a way that
-;; distinguishes errors that occur mid-way through LOAD from the
-;; initial failure to OPEN the file, so that handlers can try do
-;; defaulting only when the file didn't exist at the start of LOAD,
-;; while allowing race conditions to get through.
-(define-condition load-race-condition (error)
-  ((pathname :reader load-race-condition-pathname :initarg :pathname))
-  (:report (lambda (condition stream)
-             (format stream "~@<File ~S was deleted or renamed during LOAD.~:>"
-                     (load-race-condition-pathname condition)))))
-
-(defmacro resignal-race-condition (&body body)
-  `(handler-case (progn ,@body)
-     (file-error (error)
-       (error 'load-race-condition :pathname (file-error-pathname error)))))
 
 ;;; The following comment preceded the pre 1.0.12.36 definition of
 ;;; LOAD; it may no longer be accurate:
 ;; LOADing a non-compiled file. Check whether this bug exists in SBCL
 ;; and fix it if so.
 
-;;; This is our real LOAD.  The LOAD below is just a wrapper that does
-;;; some defaulting in case the user asks us to load a file that
-;;; doesn't exist at the time we start.
-(defun %load (pathspec &key (verbose *load-verbose*) (print *load-print*)
-              (if-does-not-exist t) (external-format :default))
-  (when (streamp pathspec)
-    (let* ( ;; Bindings required by ANSI.
-           (*readtable* *readtable*)
-           (*package* (sane-package))
-           ;; FIXME: we should probably document the circumstances
-           ;; where *LOAD-PATHNAME* and *LOAD-TRUENAME* aren't
-           ;; pathnames during LOAD.  ANSI makes no exceptions here.
-           (*load-pathname* (handler-case (pathname pathspec)
-                              ;; FIXME: it should probably be a type
-                              ;; error to try to get a pathname for a
-                              ;; stream that doesn't have one, but I
-                              ;; don't know if we guarantee that.
-                              (error () nil)))
-           (*load-truename* (when *load-pathname*
-                              (handler-case (truename *load-pathname*)
-                                (file-error () nil))))
-           ;; Bindings used internally.
-           (*load-depth* (1+ *load-depth*))
-           ;; KLUDGE: I can't find in the ANSI spec where it says
-           ;; that DECLAIM/PROCLAIM of optimization policy should
-           ;; have file scope. CMU CL did this, and it seems
-           ;; reasonable, but it might not be right; after all,
-           ;; things like (PROCLAIM '(TYPE ..)) don't have file
-           ;; scope, and I can't find anything under PROCLAIM or
-           ;; COMPILE-FILE or LOAD or OPTIMIZE which justifies this
-           ;; behavior. Hmm. -- WHN 2001-04-06
-           (sb!c::*policy* sb!c::*policy*))
-      (return-from %load
-        (if (equal (stream-element-type pathspec) '(unsigned-byte 8))
-            (load-as-fasl pathspec verbose print)
-            (load-as-source pathspec verbose print)))))
-  ;; If we're here, PATHSPEC isn't a stream, so must be some other
-  ;; kind of pathname designator.
-  (with-open-file (stream pathspec
-                          :element-type '(unsigned-byte 8)
-                          :if-does-not-exist
-                          (if if-does-not-exist :error nil))
-    (unless stream
-      (return-from %load nil))
-    (let* ((header-line (make-array
-                         (length *fasl-header-string-start-string*)
-                         :element-type '(unsigned-byte 8))))
-      (read-sequence header-line stream)
-      (if (mismatch header-line *fasl-header-string-start-string*
-                    :test #'(lambda (code char) (= code (char-code char))))
-          (let ((truename (resignal-race-condition (probe-file stream))))
-            (when (and truename
-                       (string= (pathname-type truename) *fasl-file-type*))
-              (error 'fasl-header-missing
-                     :stream (namestring truename)
-                     :fhsss header-line
-                     :expected *fasl-header-string-start-string*)))
-          (progn
-            (file-position stream :start)
-            (return-from %load
-              (%load stream :verbose verbose :print print))))))
-  ;; Because we're just opening for input, we don't need
-  ;; WITH-OPEN-FILE's abort handling semantics, and we want to say
-  ;; it's an error for PATHSPEC to have existed before but not now, so
-  ;; WITH-OPEN-STREAM it is.
-  (with-open-stream (stream (resignal-race-condition
-                              (open pathspec
-                                    :external-format external-format)))
-    (%load stream :verbose verbose :print print)))
-
-;; Given a simple %LOAD like the above, one can implement any
-;; particular defaulting strategy with a wrapper like this one:
 (defun load (pathspec &key (verbose *load-verbose*) (print *load-print*)
-            (if-does-not-exist :error) (external-format :default))
+             (if-does-not-exist t) (external-format :default))
   #!+sb-doc
   "Load the file given by FILESPEC into the Lisp environment, returning
    T on success."
-  (handler-bind ((file-error
-                  #'(lambda (error)
-                      ;; This handler will run if %LOAD failed to OPEN
-                      ;; the file to look for a fasl header.
-                      (let ((pathname (file-error-pathname error)))
-                        ;; As PROBE-FILE returned NIL, the file
-                        ;; doesn't exist.  If the filename we tried to
-                        ;; open lacked a type, try loading a filename
-                        ;; determined by our defaulting.
-                        (when (null (handler-case (probe-file pathname)
-                                      (file-error (error) error)))
-                          (when (null (pathname-type pathname))
-                            (let ((default (probe-load-defaults pathname)))
-                              (when default
-                                (return-from load
-                                  (resignal-race-condition
-                                    (%load default
-                                           :verbose verbose
-                                           :print print
-                                           :external-format
-                                           external-format
+  (flet ((load-stream (stream)
+           (let* (;; Bindings required by ANSI.
+                  (*readtable* *readtable*)
+                  (*package* (sane-package))
+                  ;; FIXME: we should probably document the circumstances
+                  ;; where *LOAD-PATHNAME* and *LOAD-TRUENAME* aren't
+                  ;; pathnames during LOAD.  ANSI makes no exceptions here.
+                  (*load-pathname* (handler-case (pathname stream)
+                                     ;; FIXME: it should probably be a type
+                                     ;; error to try to get a pathname for a
+                                     ;; stream that doesn't have one, but I
+                                     ;; don't know if we guarantee that.
+                                     (error () nil)))
+                  (*load-truename* (when *load-pathname*
+                                     (handler-case (truename stream)
+                                       (file-error () nil))))
+                  ;; Bindings used internally.
+                  (*load-depth* (1+ *load-depth*))
+                  ;; KLUDGE: I can't find in the ANSI spec where it says
+                  ;; that DECLAIM/PROCLAIM of optimization policy should
+                  ;; have file scope. CMU CL did this, and it seems
+                  ;; reasonable, but it might not be right; after all,
+                  ;; things like (PROCLAIM '(TYPE ..)) don't have file
+                  ;; scope, and I can't find anything under PROCLAIM or
+                  ;; COMPILE-FILE or LOAD or OPTIMIZE which justifies this
+                  ;; behavior. Hmm. -- WHN 2001-04-06
+                  (sb!c::*policy* sb!c::*policy*))
+             (return-from load
+               (if (equal (stream-element-type stream) '(unsigned-byte 8))
+                   (load-as-fasl stream verbose print)
+                   (load-as-source stream verbose print))))))
+    (when (streamp pathspec)
+      (return-from load (load-stream pathspec)))
+    (let ((pathname (pathname pathspec)))
+      (with-open-stream
+          (stream (or (open pathspec :element-type '(unsigned-byte 8)
+                            :if-does-not-exist nil)
+                      (when (null (pathname-type pathspec))
+                        (let ((defaulted-pathname
+                               (probe-load-defaults pathspec)))
+                          (if defaulted-pathname
+                              (progn (setq pathname defaulted-pathname)
+                                     (open pathname
                                            :if-does-not-exist
-                                           if-does-not-exist))))))))
-                      ;; If we're here, one of three things happened:
-                      ;; (1) %LOAD errored and PROBE-FILE succeeded,
-                      ;; in which case the file must be a bad symlink,
-                      ;; unreadable, or it was created between %LOAD
-                      ;; and PROBE-FILE; (2) %LOAD errored and
-                      ;; PROBE-FILE errored, and so things are amiss
-                      ;; in the file system (albeit possibly
-                      ;; differently now than when OPEN errored); (3)
-                      ;; our defaulting did not find a file.  In any
-                      ;; of these cases, decline to handle the
-                      ;; original error or return NIL, depending on
-                      ;; IF-DOES-NOT-EXIST.
+                                           (if if-does-not-exist :error nil)
+                                           :element-type '(unsigned-byte 8))))))
                       (if if-does-not-exist
-                          nil
-                          (return-from load nil)))))
-    (%load pathspec :verbose verbose :print print
-           :external-format external-format)))
+                          (error 'simple-file-error
+                                 :pathname pathspec
+                                 :format-control
+                                 "~@<Couldn't load ~S: file does not exist.~@:>"
+                                 :format-arguments (list pathspec)))))
+        (unless stream
+          (return-from load nil))
+
+        (let* ((header-line (make-array
+                             (length *fasl-header-string-start-string*)
+                             :element-type '(unsigned-byte 8))))
+          (read-sequence header-line stream)
+          (if (mismatch header-line *fasl-header-string-start-string*
+                        :test #'(lambda (code char) (= code (char-code char))))
+              (let ((truename (probe-file stream)))
+                (when (and truename
+                           (string= (pathname-type truename) *fasl-file-type*))
+                  (error 'fasl-header-missing
+                         :stream (namestring truename)
+                         :fhsss header-line
+                         :expected *fasl-header-string-start-string*)))
+              (progn
+                (file-position stream :start)
+                (return-from load
+                  (load-stream stream))))))
+      (with-open-file (stream pathname :external-format external-format)
+        (load-stream stream)))))
 
 ;; This implements the defaulting SBCL seems to have inherited from
 ;; CMU.  This routine does not try to perform any loading; all it does
                       (file-error () nil)))
     (cond ((and defaulted-fasl-truename
                 defaulted-source-truename
-                (> (resignal-race-condition
-                     (file-write-date defaulted-source-truename))
-                   (resignal-race-condition
-                     (file-write-date defaulted-fasl-truename))))
+                (> (file-write-date defaulted-source-truename)
+                   (file-write-date defaulted-fasl-truename)))
            (restart-case
                (error "The object file ~A is~@
                        older than the presumed source:~%  ~A."