(I seem to've screwed up during the checkin of 0.pre7.131 before, so
[sbcl.git] / src / pcl / walk.lisp
index b1d8865..0589bd1 100644 (file)
 
 ;;; In SBCL, as in CMU CL before it, the environment is represented
 ;;; with a structure that holds alists for the functional things,
-;;; variables, blocks, etc.
-;;; Except for SYMBOL-MACROLET, only the SB-C::LEXENV-FUNCTIONS slot
-;;; is relevant. It holds: Alist (Name . What), where What is either
-;;; a functional (a local function) or a list (MACRO . <function>) (a
-;;; local macro, with the specifier expander.) Note that Name may be a
-;;; (SETF <name>) function.
-;;; Accessors are defined below, eg (ENV-WALK-FUNCTION ENV).
+;;; variables, blocks, etc. Except for SYMBOL-MACROLET, only the
+;;; SB-C::LEXENV-FUNS slot is relevant. It holds: Alist (Name . What),
+;;; where What is either a functional (a local function) or a list
+;;; (MACRO . <function>) (a local macro, with the specifier expander.)
+;;; Note that Name may be a (SETF <name>) function. Accessors are
+;;; defined below, eg (ENV-WALK-FUNCTION ENV).
 ;;;
 ;;; If WITH-AUGMENTED-ENVIRONMENT is called from WALKER-ENVIRONMENT-BIND
 ;;; this code hides the WALKER version of an environment
 ;;; a list, which was not really an interpreted function.
 ;;; Instead this list was COERCEd to a #<FUNCTION ...>!
 ;;;
-;;; Instead, we now use a special sort of "function"-type for that information,
-;;; because the functions slot in SB-C::LEXENV is supposed to have a list of
-;;; <Name MACRO . #<function> elements.
-;;; So, now we hide our bits of interest in the walker-info slot in our new
-;;; BOGO-FUNCTION.
+;;; Instead, we now use a special sort of "function"-type for that
+;;; information, because the functions slot in SB-C::LEXENV is
+;;; supposed to have a list of <Name MACRO . #<function> elements.
+;;; So, now we hide our bits of interest in the walker-info slot in
+;;; our new BOGO-FUN.
 ;;;
 ;;; MACROEXPAND-1 is the only SBCL function that gets called with the
 ;;; constructed environment argument.
 
+(/show "walk.lisp 108")
+
 (defmacro with-augmented-environment
     ((new-env old-env &key functions macros) &body body)
   `(let ((,new-env (with-augmented-environment-internal ,old-env
                                                        ,macros)))
      ,@body))
 
-(defstruct (bogo-function
-           (:alternate-metaclass sb-kernel:funcallable-instance
-                                 sb-kernel:funcallable-structure-class
-                                 sb-kernel:make-funcallable-structure-class)
-           (:type sb-kernel:funcallable-structure)
-           (:copier nil))
-  (walker-info (required-argument) :type list))
-
-(defun walker-info-to-bogo-function (x)
-  (make-bogo-function :walker-info x))
+;;; a unique tag to show that we're the intended caller of BOGO-FUN
+(defvar *bogo-fun-magic-tag*
+  '(:bogo-fun-magic-tag))
 
-(defun bogo-function-to-walker-info (x)
-  (bogo-function-walker-info x))
+;;; The interface of BOGO-FUNs (previously implemented as
+;;; FUNCALLABLE-INSTANCEs) is just these two operations, so we can do
+;;; them with ordinary closures.
+;;;
+;;; KLUDGE: BOGO-FUNs are sorta weird, and MNA and I have both hacked
+;;; on this code without quite figuring out what they're for. (He
+;;; changed them to work after some changes in the IR1 interpreter
+;;; made functions not be built lazily, and I changed them so that
+;;; they don't need FUNCALLABLE-INSTANCE stuff, so that the F-I stuff
+;;; can become less general.) There may be further simplifications or
+;;; clarifications which could be done. -- WHN 2001-10-19
+(defun walker-info-to-bogo-fun (walker-info)
+  (lambda (magic-tag &rest rest)
+    (aver (not rest)) ; else someone is using me in an unexpected way
+    (aver (eql magic-tag *bogo-fun-magic-tag*)) ; else ditto
+    walker-info))
+(defun bogo-fun-to-walker-info (bogo-fun)
+  (declare (type function bogo-fun))
+  (funcall bogo-fun *bogo-fun-magic-tag*))
    
-(defun with-augmented-environment-internal (env functions macros)
+(defun with-augmented-environment-internal (env funs macros)
   ;; Note: In order to record the correct function definition, we
   ;; would have to create an interpreted closure, but the
   ;; WITH-NEW-DEFINITION macro down below makes no distinction between
   (let ((lexenv (sb-kernel::coerce-to-lexenv env)))
     (sb-c::make-lexenv
       :default lexenv
-      :functions
-      (append (mapcar (lambda (f)
-                       (cons (car f) (sb-c::make-functional :lexenv lexenv)))
-                     functions)
-             (mapcar (lambda (m)
-                       (list* (car m)
-                              'sb-c::macro
-                               (if (eq (car m) *key-to-walker-environment*)
-                                 (walker-info-to-bogo-function (cadr m))
-                                 (coerce (cadr m) 'function))))
-                      macros)))))
+      :funs (append (mapcar (lambda (f)
+                             (cons (car f)
+                                   (sb-c::make-functional :lexenv lexenv)))
+                           funs)
+                   (mapcar (lambda (m)
+                             (list* (car m)
+                                    'sb-c::macro
+                                    (if (eq (car m)
+                                            *key-to-walker-environment*)
+                                        (walker-info-to-bogo-fun (cadr m))
+                                        (coerce (cadr m) 'function))))
+                           macros)))))
 
 (defun environment-function (env fn)
   (when env
-    (let ((entry (assoc fn (sb-c::lexenv-functions env) :test #'equal)))
+    (let ((entry (assoc fn (sb-c::lexenv-funs env) :test #'equal)))
       (and entry
           (sb-c::functional-p (cdr entry))
           (cdr entry)))))
 
 (defun environment-macro (env macro)
   (when env
-    (let ((entry (assoc macro (sb-c::lexenv-functions env) :test #'eq)))
+    (let ((entry (assoc macro (sb-c::lexenv-funs env) :test #'eq)))
       (and entry
           (eq (cadr entry) 'sb-c::macro)
            (if (eq macro *key-to-walker-environment*)
-             (values (bogo-function-to-walker-info (cddr entry)))
-             (values (function-lambda-expression (cddr entry))))))))
+              (values (bogo-fun-to-walker-info (cddr entry)))
+              (values (function-lambda-expression (cddr entry))))))))
 \f
 ;;;; other environment hacking, not so SBCL-specific as the
 ;;;; environment hacking in the previous section
        (body (cdddr form)))
     (walk-form-internal
       `(let ()
-        (declare (special ,@(mapcar #'(lambda (x) (if (listp x) (car x) x))
+        (declare (special ,@(mapcar (lambda (x) (if (listp x) (car x) x))
                                     bindings)))
         (flet ((.let-if-dummy. () ,@body))
           (if ,test
 
 (defun walk-multiple-value-setq (form context env)
   (let ((vars (cadr form)))
-    (if (some #'(lambda (var)
-                 (variable-symbol-macro-p var env))
+    (if (some (lambda (var)
+               (variable-symbol-macro-p var env))
              vars)
-       (let* ((temps (mapcar #'(lambda (var)
-                                 (declare (ignore var))
-                                 (gensym))
+       (let* ((temps (mapcar (lambda (var)
+                               (declare (ignore var))
+                               (gensym))
                              vars))
-              (sets (mapcar #'(lambda (var temp) `(setq ,var ,temp))
+              (sets (mapcar (lambda (var temp) `(setq ,var ,temp))
                             vars
                             temps))
               (expanded `(multiple-value-bind ,temps ,(caddr form)
           (walked-body
             (walk-declarations
               body
-              #'(lambda (real-body real-env)
-                  (setq walked-bindings
-                        (walk-bindings-1 bindings
-                                         old-env
-                                         new-env
-                                         context
-                                         nil))
-                  (walk-repeat-eval real-body real-env))
+              (lambda (real-body real-env)
+                (setq walked-bindings
+                      (walk-bindings-1 bindings
+                                       old-env
+                                       new-env
+                                       context
+                                       nil))
+                (walk-repeat-eval real-body real-env))
               new-env)))
       (relist* form mvb walked-bindings mv-form walked-body))))
 
     (walker-environment-bind
        (new-env old-env
                 :lexical-variables
-                (append (mapcar #'(lambda (binding)
-                                    `(,(car binding)
-                                      :macro . ,(cadr binding)))
+                (append (mapcar (lambda (binding)
+                                  `(,(car binding)
+                                    :macro . ,(cadr binding)))
                                 bindings)
                         (env-lexical-variables old-env)))
       (relist* form 'symbol-macrolet bindings