X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~mdw/git/clg/blobdiff_plain/9ca5565a9443f9bd9f464838b7828589fe47be7f..73572c12ccd49c661d06287903bfa725f5fd93a5:/glib/proxy.lisp diff --git a/glib/proxy.lisp b/glib/proxy.lisp index f5c3f6e..6ad8b90 100644 --- a/glib/proxy.lisp +++ b/glib/proxy.lisp @@ -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))