Bug fix

[clg] / glib / gerror.lisp

diff --git a/glib/gerror.lisp b/glib/gerror.lisp
index 5e0f56d529d05785d22d2211250341165c9d39af..13b22d50f96940f22a106e11a8a079687c4a4545 100644 (file)
--- a/glib/gerror.lisp
+++ b/glib/gerror.lisp
@@ -1,5 +1,5 @@
 ;; Common Lisp bindings for GTK+ v2.0
-;; Copyright 2005 Espen S. Johnsen <espen@users.sf.net>
+;; Copyright 2005-2006 Espen S. Johnsen <espen@users.sf.net>
 ;;
 ;; Permission is hereby granted, free of charge, to any person obtaining
 ;; a copy of this software and associated documentation files (the
@@ -20,7 +20,7 @@
 ;; TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
 ;; SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
-;; $Id: gerror.lisp,v 1.5 2006-02-26 15:30:01 espen Exp $
+;; $Id: gerror.lisp,v 1.10 2008-04-11 20:35:48 espen Exp $
 
 
 (in-package "GLIB")
@@ -29,7 +29,9 @@ (defclass gerror (struct)
   ((domain :allocation :alien :type quark :reader gerror-domain)
    (code :allocation :alien :type int :reader gerror-code)
    (message :allocation :alien :type string :reader gerror-message))
-  (:metaclass struct-class))
+  (:metaclass struct-class)
+  (:ref %gerror-copy)
+  (:unref %gerror-free))
 
 (defbinding (%gerror-copy "g_error_copy") () pointer
   (location pointer))
@@ -37,17 +39,8 @@ (defbinding (%gerror-copy "g_error_copy") () pointer
 (defbinding (%gerror-free "g_error_free") () nil
   (location pointer))
 
-(defmethod reference-foreign ((class (eql (find-class 'gerror))) location)
-  (declare (ignore class))
-  (%gerror-copy location))
-
-(defmethod unreference-foreign ((class (eql (find-class 'gerror))) location)
-  (declare (ignore class))
-  (%gerror-free location))
-
-
 (define-condition glib-error (error)
-  ((code :initarg :domain :reader gerror-code)
+  ((code :initarg :code :reader gerror-code)
    (message :initarg :message :reader gerror-message))
   (:report (lambda (condition stream)
             (write-string (gerror-message condition) stream))))
@@ -67,9 +60,13 @@ (defun signal-gerror (gerror)
 
 (deftype gerror-signal () 'gerror)
 
-(define-type-method from-alien-form ((type gerror-signal) gerror)
+(define-type-method return-type ((type gerror-signal))
   (declare (ignore type))
-  `(let ((gerror ,(from-alien-form 'gerror gerror)))
+  '(or null gerror))
+
+(define-type-method from-alien-form ((type gerror-signal) gerror &key (ref :free))
+  (declare (ignore type))
+  `(let ((gerror ,(from-alien-form 'gerror gerror :ref ref)))
      (when gerror
        (signal-gerror gerror))))
 
@@ -107,6 +104,9 @@ (define-condition critical-log-level (log-level)
 (define-condition warning-log-level (log-level)
   ())
 
+(define-condition message-log-level (log-level)
+  ())
+
 (define-condition info-log-level (log-level)
   ())
 
@@ -130,18 +130,21 @@ (define-callback log-handler nil
     (funcall (if fatal-p #'error #'warn) condition
      :domain domain :message message)))
 
-(setf (extern-alien "log_handler" system-area-pointer) 
+#-clisp
+(setf 
+ #+cmu(alien:extern-alien "log_handler" alien:system-area-pointer) 
+ #+sbcl(sb-alien:extern-alien "log_handler" sb-alien:system-area-pointer)
  (callback-address log-handler))
 
 
-#+glib2.6
+#?(pkg-exists-p "glib-2.0" :atleast-version "2.6.0")
 (progn
   ;; Unfortunately this will only work as long as we don't abort to
   ;; toplevel from within the log handler. If we do that, the next
   ;; invocation of g_log will be handled as a recursion and cause an
   ;; abort (SIGABORT being signaled). To make things even worse, SBCL
   ;; doesn't handle SIGABRT at all.
-  (defbinding %log-set-default-handler () pointer
-    ((progn log-handler) callback)
+  (defbinding %log-set-default-handler (nil) pointer
+    (log-handler callback)
     (nil null))
   (%log-set-default-handler))