X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~mdw/git/clg/blobdiff_plain/7479d92c2e0ee576d0d376bbbbb72a9dcb948e4b..0c0db5e2b38bf90a44b4cfb4646102470093fed1:/glib/gboxed.lisp

diff --git a/glib/gboxed.lisp b/glib/gboxed.lisp
index e2dac5c..393cb18 100644
--- a/glib/gboxed.lisp
+++ b/glib/gboxed.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: gboxed.lisp,v 1.2 2001-04-30 11:25:25 espen Exp $
+;; $Id: gboxed.lisp,v 1.6 2001-10-21 22:02:01 espen Exp $
 
 (in-package "GLIB")
 
@@ -23,85 +23,79 @@ (in-package "GLIB")
 (eval-when (:compile-toplevel :load-toplevel :execute)
   (defclass boxed (proxy)
     ()
-    (:metaclass proxy-class)))
-
-(defmethod initialize-proxy ((boxed boxed) &rest initargs
-			     &key location weak-ref)
-  (declare (ignore initargs))
-  (setf
-   (slot-value boxed 'location)
-   (if weak-ref
-       (%boxed-copy (find-type-number (class-of boxed)) location)
-     location))
-  (call-next-method))
-
-(defmethod instance-finalizer ((boxed boxed))
-  (let ((location (proxy-location boxed))
-	(type-number (find-type-number (class-of boxed))))
-    (declare (type system-area-pointer location))
-    #'(lambda ()
-	(%boxed-free type-number location)
-	(remove-cached-instance location))))
-
-
-(deftype-method translate-to-alien boxed (type-spec boxed &optional weak-ref)
-  (if weak-ref
-      `(proxy-location ,boxed)
-    `(let ((boxed ,boxed))
-       (%boxed-copy
-	(find-type-number type-spec)
-	(proxy-location boxed)))))
-
-(deftype-method unreference-alien boxed (type-spec c-struct)
-  `(%boxed-free ,(find-type-number type-spec) ,c-struct))
-
-
-(defbinding %boxed-copy () pointer
-  (type type-number)
+    (:metaclass proxy-class)
+    (:copy %boxed-copy)
+    (:free %boxed-free)))
+
+(defbinding %boxed-copy (type location) pointer
+  ((find-type-number type) type-number)
   (location pointer))
 
-(defbinding %boxed-free () nil
-  (type type-number)
+(defbinding %boxed-free (type location) nil
+  ((find-type-number type) type-number)
   (location pointer))
 
 
 ;;;; Metaclass for boxed classes
 
 (eval-when (:compile-toplevel :load-toplevel :execute)
-  (defclass boxed-class (proxy-class)))
-
-
-(defmethod shared-initialize ((class boxed-class) names
-			      &rest initargs
-			      &key name alien-name type-init)
-  (declare (ignore initargs names))
-  (call-next-method)
-
-  (let* ((class-name (or name (class-name class)))
-	 (type-number
-	  (cond
-	   ((and alien-name type-init)
-	    (error
-	     "Specify either :type-init or :alien-name for class ~A"
-	     class-name))
-	   (alien-name (type-number-from-alien-name (first alien-name)))
-	   (type-init (funcall (mkbinding (first type-init) 'type-number)))
-	   (t
-	    (or
-	     (type-number-from-alien-name
-	      (default-alien-type-name class-name) nil)
-	     (funcall
-	      (mkbinding
-	       (default-alien-fname (format nil "~A_get_type" class-name))
-	       'type-number)))))))
-    (setf (find-type-number class) type-number)))
-
-
-(defmethod validate-superclass
+  (defclass boxed-class (proxy-class))
+
+
+  (defmethod shared-initialize ((class boxed-class) names
+				&rest initargs &key name alien-name)
+    (declare (ignore initargs names))
+    (call-next-method)
+    
+    (let* ((class-name (or name (class-name class)))
+	   (type-number
+	    (find-type-number
+	     (or (first alien-name) (default-alien-type-name class-name)))))
+      (register-type class-name type-number)))
+
+
+  (defmethod validate-superclass
     ((class boxed-class) (super pcl::standard-class))
-  (subtypep (class-name super) 'boxed))
+    (subtypep (class-name super) 'boxed)))
+
+
+;;;; 
+
+(defun expand-boxed-type (type-number &optional slots)
+  `(defclass ,(type-from-number type-number) (boxed)
+     ,slots
+     (:metaclass boxed-class)
+     (:alien-name ,(find-type-name type-number))))
+
+(register-derivable-type 'boxed "GBoxed" 'expand-boxed-type)
+
+;;;; Special boxed types
+
+(defclass gstring (boxed)
+  ()
+  (:metaclass boxed-class)
+  (:alien-name "GString"))
+
+(deftype-method translate-from-alien
+    gstring (type-spec location &optional weak-ref)
+  `(let ((location ,location))
+     (unless (null-pointer-p location)
+       (prog1
+	   (c-call::%naturalize-c-string location)
+	 ,(unless weak-ref
+	    (unreference-alien type-spec location))))))
 
+(deftype-method translate-to-alien
+    gstring (type-spec string &optional weak-ref)
+  (declare (ignore type-spec weak-ref))
+  `(let ((string ,string))
+     ;; Always copy strings to prevent seg fault due to GC
+     (funcall
+      ',(proxy-class-copy (find-class type-spec))
+      ',type-spec
+      (make-pointer (1+ (kernel:get-lisp-obj-address string))))))
 
-;;;; Initializing type numbers
+(deftype-method cleanup-alien gstring (type-spec c-string &optional weak-ref)
+  (when weak-ref
+    (unreference-alien type-spec c-string)))
 
-(setf (alien-type-name 'boxed) "GBoxed")