chiark / gitweb /
Changes required by SBCL
[clg] / glib / proxy.lisp
index f5c3f6e4330097b0ce195b39b05e2b2871226bb4..6ad8b907363828c28ad0e4cfac74fa2fb02b30dd 100644 (file)
@@ -15,7 +15,7 @@
 ;; License along with this library; if not, write to the Free Software
 ;; Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
-;; $Id: proxy.lisp,v 1.14 2004-11-19 13:02:51 espen Exp $
+;; $Id: proxy.lisp,v 1.19 2005-02-03 23:09:04 espen Exp $
 
 (in-package "GLIB")
 
@@ -28,21 +28,26 @@   (defclass virtual-slots-class (standard-class)
   (defclass direct-virtual-slot-definition (standard-direct-slot-definition)
     ((setter :reader slot-definition-setter :initarg :setter)
      (getter :reader slot-definition-getter :initarg :getter)
+     (unbound :reader slot-definition-unbound :initarg :unbound)
      (boundp :reader slot-definition-boundp :initarg :boundp)))
   
   (defclass effective-virtual-slot-definition (standard-effective-slot-definition)
     ((setter :reader slot-definition-setter :initarg :setter)
      (getter :reader slot-definition-getter :initarg :getter)
-     (boundp :reader slot-definition-boundp :initarg :boundp)))
+     (unbound :reader slot-definition-unbound :initarg :unbound)
+     (boundp :reader slot-definition-boundp :initarg :boundp))))
+  
+  (defvar *unbound-marker* (gensym "UNBOUND-MARKER-"))
 
-  (defun most-specific-slot-value (instances slot &optional default)
+  (defun most-specific-slot-value (instances slot &optional 
+                                  (default *unbound-marker*))
     (let ((object (find-if
                   #'(lambda (ob)
                       (and (slot-exists-p ob slot) (slot-boundp ob slot)))
                   instances)))
       (if object
          (slot-value object slot)
-         default))))
+         default)));)
 
   
 
@@ -58,72 +63,133 @@ (defmethod effective-slot-definition-class ((class virtual-slots-class) &rest in
 
 
 (defmethod initialize-internal-slot-functions ((slotd effective-virtual-slot-definition))
-  (with-slots (getter setter boundp) slotd
-    (unless (slot-boundp slotd 'reader-function)
-      (setf 
+  (if (not (slot-boundp slotd 'getter))
+      (setf
        (slot-value slotd 'reader-function)
-       (etypecase getter
-        (function getter)
-        (null #'(lambda (object)
-                  (declare (ignore object))
-                  (error "Can't read slot: ~A" (slot-definition-name slotd))))
-        (symbol #'(lambda (object)
-                    (funcall getter object)))
-        (string ;(let ()(reader  (mkbinding getter 
-;;                             (slot-definition-type slotd) 'pointer)))
-                  (setf (slot-value slotd 'reader-function)
-                        #'(lambda (object)
-                            (let ((reader
-                               (mkbinding getter 
-                                (slot-definition-type slotd) 'pointer)))
-                            (funcall reader (proxy-location object)))))))))
-
-    (unless (slot-boundp slotd 'writer-function)
+       #'(lambda (object)
+          (declare (ignore object))
+          (error "Can't read slot: ~A" (slot-definition-name slotd)))
+       (slot-value slotd 'boundp-function)
+       #'(lambda (object) (declare (ignore object)) nil))
+
+    (let ((getter-function
+          (let ((getter (slot-value slotd 'getter)))
+            (etypecase getter
+              (function getter)
+              (symbol 
+               #'(lambda (object)
+                   (funcall getter object)))
+              (string 
+               (let ((reader nil))
+                 (setf (slot-value slotd 'reader-function)
+                       #'(lambda (object)
+                           (unless reader
+                             (setq reader
+                              (mkbinding getter 
+                               (slot-definition-type slotd) 'pointer)))
+                           (funcall reader (proxy-location object))))))))))
+
       (setf 
-       (slot-value slotd 'writer-function)
+       (slot-value slotd 'boundp-function)
+       (cond
+       ((slot-boundp slotd 'unbound)
+        (let ((unbound-value (slot-value slotd 'unbound)))
+          #'(lambda (object)
+              (not (eq (funcall getter-function object) unbound-value)))))
+       ((slot-boundp slotd 'boundp)
+        (let ((boundp (slot-value slotd 'boundp)))
+          (etypecase boundp
+            (function boundp)
+            (symbol #'(lambda (object)
+                        (funcall boundp object)))
+            (string (let ((reader ()))
+                      #'(lambda (object)
+                          (unless reader
+                            (setq reader
+                             (mkbinding boundp
+                              (slot-definition-type slotd) 'pointer)))
+                          (funcall reader (proxy-location object))))))))
+       ((multiple-value-bind (unbound-p unbound-value)
+            (unbound-value (slot-definition-type slotd))
+          (when unbound-p
+            #'(lambda (object)
+                (not (eq (funcall getter-function object) unbound-value))))))
+       (#'(lambda (object) (declare (ignore object)) t))))
+
+      (setf
+       (slot-value slotd 'reader-function)
+       (cond
+       ((slot-boundp slotd 'unbound)
+        (let ((unbound (slot-value slotd 'unbound))
+              (slot-name (slot-definition-name slotd)))
+          #'(lambda (object)
+              (let ((value (funcall getter-function object)))
+                (if (eq value unbound)
+                    (slot-unbound (class-of object) object slot-name)
+                  value)))))
+       ((slot-boundp slotd 'boundp)
+        (let ((boundp-function (slot-value slotd 'boundp-function)))
+          #'(lambda (object)
+              (and
+               (funcall boundp-function object)
+               (funcall getter-function object)))))
+       ((multiple-value-bind (unbound-p unbound-value)
+            (unbound-value (slot-definition-type slotd))
+          (let ((slot-name (slot-definition-name slotd)))
+            (when unbound-p
+              #'(lambda (object)
+                  (let ((value (funcall getter-function object)))
+                    (if (eq value unbound-value)
+                        (slot-unbound (class-of object) object slot-name)
+                        value)))))))
+       (getter-function)))))
+
+  (setf 
+   (slot-value slotd 'writer-function)
+   (if (not (slot-boundp slotd 'setter))
+       #'(lambda (object)
+          (declare (ignore object))
+          (error "Can't set slot: ~A" (slot-definition-name slotd)))
+     (with-slots (setter) slotd
        (etypecase setter
         (function setter)
-        (null #'(lambda (object)
-                  (declare (ignore object))
-                  (error "Can't set slot: ~A" (slot-definition-name slotd))))
-        ((or symbol cons) #'(lambda (value object)
-                              (funcall (fdefinition setter) value object)))
+        ((or symbol cons) 
+         #'(lambda (value object)
+             (funcall (fdefinition setter) value object)))
         (string
-         (let ((writer ()));; (mkbinding setter 'nil 'pointer 
-;;                      (slot-definition-type slotd))))
-           (setf (slot-value slotd 'writer-function)
-                 #'(lambda (value object)
-                     (unless writer
-                       (setq writer
-                        (mkbinding setter 'nil 'pointer 
-                         (slot-definition-type slotd))))
-                     (funcall writer (proxy-location object) value))))))))
-
-    (unless (slot-boundp slotd 'boundp-function)
-      (setf 
-       (slot-value slotd 'boundp-function)
-       (etypecase boundp
-        (function boundp)
-        (null #'(lambda (object)
-                  (declare (ignore object))
-                  t))
-        (symbol #'(lambda (object)
-                    (funcall boundp object)))))))
+         (let ((writer ()))
+           (setf
+            (slot-value slotd 'writer-function)
+            #'(lambda (value object)
+                (unless writer
+                  (setq writer
+                   (mkbinding setter 'nil 'pointer 
+                    (slot-definition-type slotd))))
+                (funcall writer (proxy-location object) value)))))))))
+
   (initialize-internal-slot-gfs (slot-definition-name slotd)))
 
 
 
-(defmethod compute-slot-accessor-info ((slotd effective-virtual-slot-definition)
-                                           type gf)
+(defmethod compute-slot-accessor-info ((slotd effective-virtual-slot-definition) type gf)
   nil)
 
 (defmethod compute-effective-slot-definition-initargs ((class virtual-slots-class) direct-slotds)
-  (if (eq (most-specific-slot-value direct-slotds 'allocation) :virtual)
-      (nconc 
-       (list :getter (most-specific-slot-value direct-slotds 'getter)
-            :setter (most-specific-slot-value direct-slotds 'setter)
-            :boundp (most-specific-slot-value direct-slotds 'boundp))
-       (call-next-method))
+  (if (typep (first direct-slotds) 'direct-virtual-slot-definition)
+      (let ((initargs ()))
+       (let ((getter (most-specific-slot-value direct-slotds 'getter)))
+         (unless (eq getter *unbound-marker*)
+           (setf (getf initargs :getter) getter)))
+       (let ((setter (most-specific-slot-value direct-slotds 'setter)))
+         (unless (eq setter *unbound-marker*)
+           (setf (getf initargs :setter) setter)))
+       (let ((unbound (most-specific-slot-value direct-slotds 'unbound)))
+         (unless (eq unbound *unbound-marker*)
+           (setf (getf initargs :unbound) unbound)))
+       (let ((boundp (most-specific-slot-value direct-slotds 'boundp)))
+         (unless (eq boundp *unbound-marker*)
+           (setf (getf initargs :boundp) boundp)))
+       (nconc initargs (call-next-method)))
     (call-next-method)))
 
 
@@ -157,26 +223,26 @@ (defvar *instance-cache* (make-hash-table :test #'eql))
 
 (defun cache-instance (instance)
   (setf
-   (gethash (system:sap-int (proxy-location instance)) *instance-cache*)
-   (ext:make-weak-pointer instance)))
+   (gethash (sap-int (proxy-location instance)) *instance-cache*)
+   (make-weak-pointer instance)))
 
 (defun find-cached-instance (location)
-  (let ((ref (gethash (system:sap-int location) *instance-cache*)))
+  (let ((ref (gethash (sap-int location) *instance-cache*)))
     (when ref
-      (ext:weak-pointer-value ref))))
+      (weak-pointer-value ref))))
 
 (defun instance-cached-p (location)
-  (gethash (system:sap-int location) *instance-cache*))
+  (gethash (sap-int location) *instance-cache*))
 
 (defun remove-cached-instance (location)
-  (remhash (system:sap-int location) *instance-cache*))
+  (remhash (sap-int location) *instance-cache*))
 
 ;; For debuging
 (defun cached-instances ()
   (let ((instances ()))
     (maphash #'(lambda (location ref)
                 (declare (ignore location))
-                (push (ext:weak-pointer-value ref) instances))
+                (push (weak-pointer-value ref) instances))
             *instance-cache*)
     instances))
                        
@@ -192,6 +258,12 @@ (defgeneric instance-finalizer (object))
 (defgeneric reference-foreign (class location))
 (defgeneric unreference-foreign (class location))
 
+(defmethod reference-foreign ((name symbol) location)
+  (reference-foreign (find-class name) location))
+
+(defmethod unreference-foreign ((name symbol) location)
+  (unreference-foreign (find-class name) location))
+
 (defmethod unreference-foreign :around ((class class) location)
   (unless (null-pointer-p location)
 ;;     (format t "Unreferencing ~A at ~A" (class-name class) location)
@@ -203,19 +275,15 @@ (defmethod unreference-foreign :around ((class class) location)
 
 (defmethod print-object ((instance proxy) stream)
   (print-unreadable-object (instance stream :type t :identity nil)
-    (format stream "at 0x~X" (sap-int (proxy-location instance)))))
-
-(defmethod print-object ((instance proxy) stream)
-  (print-unreadable-object (instance stream :type t :identity nil)
-    (format stream "at 0x~X" (sap-int (proxy-location instance)))))
-
+    (when (slot-boundp instance 'location)
+      (format stream "at 0x~X" (sap-int (proxy-location instance))))))
 
 (defmethod initialize-instance :around ((instance proxy) &key location)
   (if location
       (setf (slot-value instance 'location) location)      
     (call-next-method))
   (cache-instance instance)
-  (ext:finalize instance (instance-finalizer instance))
+  (finalize instance (instance-finalizer instance))
   instance)
 
 (defmethod instance-finalizer ((instance proxy))
@@ -230,6 +298,10 @@ (defmethod instance-finalizer ((instance proxy))
 
 ;;;; Metaclass used for subclasses of proxy
 
+(defgeneric most-specific-proxy-superclass (class))
+(defgeneric direct-proxy-superclass (class))
+  
+
 (eval-when (:compile-toplevel :load-toplevel :execute)
   (defclass proxy-class (virtual-slots-class)
     ((size :reader proxy-instance-size)))
@@ -241,22 +313,19 @@   (defclass direct-alien-slot-definition (direct-virtual-slot-definition)
   (defclass effective-alien-slot-definition (effective-virtual-slot-definition)
     ((offset :reader slot-definition-offset :initarg :offset)))
 
-
   (defmethod most-specific-proxy-superclass ((class proxy-class))
     (find-if
      #'(lambda (class)
         (subtypep (class-name class) 'proxy))
      (cdr (compute-class-precedence-list class))))
-  
+
   (defmethod direct-proxy-superclass ((class proxy-class))
     (find-if
      #'(lambda (class)
         (subtypep (class-name class) 'proxy))
      (class-direct-superclasses class)))
   
-  (defmethod shared-initialize ((class proxy-class) names
-                               &rest initargs &key size)
-    (declare (ignore initargs))
+  (defmethod shared-initialize ((class proxy-class) names &key size)
     (call-next-method)
     (cond
       (size (setf (slot-value class 'size) (first size)))
@@ -284,29 +353,23 @@   (defmethod compute-effective-slot-definition-initargs ((class proxy-class) dir
   (defmethod initialize-internal-slot-functions ((slotd effective-alien-slot-definition))
     (with-slots (offset) slotd
       (let ((type (slot-definition-type slotd)))
-       (unless (slot-boundp slotd 'reader-function)
+       (unless (slot-boundp slotd 'getter)
          (let ((reader (reader-function type)))
            (setf 
-            (slot-value slotd 'reader-function)
+            (slot-value slotd 'getter)
             #'(lambda (object)
                 (funcall reader (proxy-location object) offset)))))
 
-       (unless (slot-boundp slotd 'writer-function)
+       (unless (slot-boundp slotd 'setter)
          (let ((writer (writer-function type))
                (destroy (destroy-function type)))
            (setf 
-            (slot-value slotd 'writer-function)
+            (slot-value slotd 'setter)
             #'(lambda (value object)
                 (let ((location (proxy-location object)))
                   (funcall destroy location offset) ; destroy old value
-                  (funcall writer value location offset))))))
+                  (funcall writer value location offset))))))))
 
-       (unless (slot-boundp slotd 'boundp-function)
-         (setf 
-          (slot-value slotd 'boundp-function)
-          #'(lambda (object)
-              (declare (ignore object))
-              t)))))
     (call-next-method))
   
 
@@ -315,7 +378,11 @@   (defconstant +struct-alignmen+ 4)
 
   (defmethod compute-slots ((class proxy-class))
     (loop 
-     with offset = (proxy-instance-size (most-specific-proxy-superclass class))
+     with offset = (let ((size-of-super-classes
+                         (proxy-instance-size 
+                          (most-specific-proxy-superclass class))))
+                    (+ size-of-super-classes 
+                       (mod size-of-super-classes +struct-alignmen+)))
      with size = offset
      for slotd in (class-direct-slots class)
      when (eq (slot-definition-allocation slotd) :alien)
@@ -338,6 +405,9 @@   (defmethod validate-superclass ((class proxy-class) (super standard-class))
   (defmethod proxy-instance-size (class)
     (declare (ignore class))
     0)
+
+  (defmethod proxy-instance-size ((class-name symbol))
+    (proxy-instance-size (find-class class-name)))
 )
   
 (defmethod alien-type ((class proxy-class) &rest args)
@@ -381,7 +451,7 @@ (defmethod copy-to-alien-form (instance (class proxy-class) &rest args)
   `(reference-foreign ',(class-name class) (proxy-location ,instance)))
 
 (defmethod copy-to-alien-function ((class proxy-class) &rest args)
-  (declare (ignore class args))
+  (declare (ignore args))
   #'(lambda (instance)
       (reference-foreign class (proxy-location instance))))
 
@@ -405,6 +475,9 @@ (defmethod destroy-function ((class proxy-class) &rest args)
   #'(lambda (location &optional (offset 0))
       (unreference-foreign class (sap-ref-sap location offset))))
 
+(defmethod unbound-value ((class proxy-class) &rest args)
+  (declare (ignore args))
+  (values t nil))
 
 (defgeneric ensure-proxy-instance (class location)
   (:documentation "Returns a proxy object representing the foreign object at the give location."))
@@ -434,7 +507,7 @@ (defmethod initialize-instance ((struct struct) &rest initargs)
     (let ((size (proxy-instance-size (class-of struct))))
       (if (zerop size)
          (error "~A has zero size" (class-of struct))
-         (setf (slot-value struct 'location) (allocate-memory size)))))
+       (setf (slot-value struct 'location) (allocate-memory size)))))
   (call-next-method))