X-Git-Url: https://git.distorted.org.uk/~mdw/clg/blobdiff_plain/167450a314d73a284308da7bd4117302ded4f0dc..adceaa0a6e981cb88f70493c9eafeb79dc73c7a6:/gtk/gtktree.lisp

diff --git a/gtk/gtktree.lisp b/gtk/gtktree.lisp
index caae594..4404530 100644
--- a/gtk/gtktree.lisp
+++ b/gtk/gtktree.lisp
@@ -1,21 +1,26 @@
-;; Common Lisp bindings for GTK+ v2.0
-;; Copyright (C) 1999-2001 Espen S. Johnsen <esj@stud.cs.uit.no>
+;; Common Lisp bindings for GTK+ v2.x
+;; Copyright 2004-2005 Espen S. Johnsen <espen@users.sf.net>
 ;;
-;; This library is free software; you can redistribute it and/or
-;; modify it under the terms of the GNU Lesser General Public
-;; License as published by the Free Software Foundation; either
-;; version 2 of the License, or (at your option) any later version.
+;; Permission is hereby granted, free of charge, to any person obtaining
+;; a copy of this software and associated documentation files (the
+;; "Software"), to deal in the Software without restriction, including
+;; without limitation the rights to use, copy, modify, merge, publish,
+;; distribute, sublicense, and/or sell copies of the Software, and to
+;; permit persons to whom the Software is furnished to do so, subject to
+;; the following conditions:
 ;;
-;; This library is distributed in the hope that it will be useful,
-;; but WITHOUT ANY WARRANTY; without even the implied warranty of
-;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-;; Lesser General Public License for more details.
+;; The above copyright notice and this permission notice shall be
+;; included in all copies or substantial portions of the Software.
 ;;
-;; You should have received a copy of the GNU Lesser General Public
-;; License along with this library; if not, write to the Free Software
-;; Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+;; THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+;; EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+;; MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+;; IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+;; CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+;; TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+;; SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
-;; $Id: gtktree.lisp,v 1.1 2004-11-07 17:22:06 espen Exp $
+;; $Id: gtktree.lisp,v 1.34 2008-04-11 20:53:32 espen Exp $
 
 
 (in-package "GTK")
@@ -38,6 +43,7 @@
       (cell-layout-pack-end layout cell expand)
     (cell-layout-pack-start layout cell expand)))
 
+
 (defbinding cell-layout-reorder () nil
   (cell-layout cell-layout)
   (cell cell-renderer)
@@ -52,73 +58,159 @@
   ((string-downcase attribute) string)
   (column int))
 
-(def-callback-marshal %cell-layout-data-func 
-    (nil cell-layout cell-renderer tree-model tree-iter))
+(define-callback-marshal %cell-layout-data-callback nil 
+  (cell-layout cell-renderer tree-model tree-iter))
 
 (defbinding cell-layout-set-cell-data-func (cell-layout cell function) nil
   (cell-layout cell-layout)
   (cell cell-renderer)
-  ((callback %cell-layout-data-func) pointer)
+  (%cell-layout-data-callback callback)
   ((register-callback-function function) unsigned-int)
-  ((callback %destroy-user-data) pointer))
+  (user-data-destroy-callback callback))
 
 (defbinding cell-layout-clear-attributes () nil
   (cell-layout cell-layout)
   (cell cell-renderer))
 
 
+;;;; Cell Renderer
+
+(defmethod compute-signal-function ((gobject cell-renderer-toggle) (signal (eql 'toggled)) function object args)
+  (declare (ignore gobject signal function object args))
+  (let ((function (call-next-method)))
+    #'(lambda (object path)
+	(funcall function object (ensure-tree-path path)))))
+
 
 ;;;; List Store
 
-(defmethod initialize-instance ((list-store list-store) &key columns)
+(defmethod initialize-instance ((list-store list-store) &key column-types
+				column-names initial-content)
   (call-next-method)
-  (%list-store-set-column-types list-store (length columns)
-   (map 'vector #'find-type-number columns)))
+  (%list-store-set-column-types list-store column-types)
+  (when column-names
+    (setf 
+     (user-data list-store 'column-names) 
+     (coerce column-names 'vector)))
+  (when initial-content
+    (loop
+     with iter = (make-instance 'tree-iter)
+     for row in initial-content
+     do (list-store-append list-store row iter))))
 
 
 (defbinding %list-store-set-column-types () nil
   (list-store list-store)
-  (n-columns unsigned-int)
-  (columns (vector type-number)))
+  ((length columns) unsigned-int)
+  (columns (vector gtype)))
+
+(defbinding list-store-remove (store row) boolean
+  (store list-store)
+  ((ensure-tree-iter store row) tree-iter))
 
 (defbinding %list-store-set-value () nil
   (list-store list-store)
   (tree-iter tree-iter)
   (column int)
-  (value gvalue))
+  (gvalue gvalue))
 
-(defun list-store-set-value (list-store tree-iter column type value)
-  (let ((gvalue (gvalue-new type value)))
-    (unwind-protect
-	 (%list-store-set-value list-store tree-iter column gvalue)
-      (gvalue-free gvalue))))
+(defmethod (setf tree-model-value) (value (store list-store) row column)
+  (let* ((index (tree-model-column-index store column))
+	 (type (tree-model-get-column-type store index)))
+    (with-gvalue (gvalue type value)
+      (%list-store-set-value store (ensure-tree-iter store row) index gvalue)))
+  value)
 
-(defbinding list-store-remove () boolean
+(defbinding %list-store-insert-with-valuesv () nil
   (list-store list-store)
-  (tree-iter tree-iter))
+  (tree-iter tree-iter)
+  (position int)
+  (columns (vector int))
+  (gvalues pointer)
+  ((length columns) int))
 
-(defbinding list-store-insert () nil
+(defbinding %list-store-insert () nil
   (list-store list-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (position int))
 
-(defbinding list-store-insert-before (list-store &optional sibling) nil
+(defun list-store-insert (store position &optional data (iter (make-instance 'tree-iter)))
+  (etypecase data
+    (null (%list-store-insert store iter position))
+
+    (list (with-memory (gvalues (* (/ (length data) 2) +gvalue-size+))
+	    (let ((columns
+		   (loop
+		    for (column value) on data by #'cddr
+		    as index = (tree-model-column-index store column)
+		    as type = (tree-model-get-column-type store index)
+		    as gvalue = gvalues then (pointer+ gvalue  +gvalue-size+)
+		    do (gvalue-init gvalue type value)
+		    collect index)))
+		(unwind-protect
+		    (%list-store-insert-with-valuesv store iter position columns gvalues)
+		  (loop
+		   repeat (length columns)
+		   as gvalue = gvalues then (pointer+ gvalue  +gvalue-size+)
+		   do (gvalue-unset gvalue))))))
+
+    (vector (with-memory (gvalues (* (length data) +gvalue-size+))
+	      (let ((columns
+		     (loop
+		      for index below (length data)
+		      as type = (tree-model-get-column-type store index)
+		      as gvalue = gvalues then (pointer+ gvalue  +gvalue-size+)
+		      do (gvalue-init gvalue type (aref data index))
+		      collect index)))
+		(unwind-protect
+		    (%list-store-insert-with-valuesv store iter position columns gvalues))
+		(loop
+		 repeat (length data)
+		 as gvalue = gvalues then (pointer+ gvalue  +gvalue-size+)
+		 do (gvalue-unset gvalue))))))
+  iter)
+
+(defbinding %list-store-insert-before () nil
   (list-store list-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (sibling (or null tree-iter)))
 
-(defbinding list-store-insert-after (list-store &optional sibling) nil
+(defun list-store-insert-before
+    (store sibling &optional data (iter (make-instance 'tree-iter)))
+  (%list-store-insert-before store iter sibling)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %list-store-insert-after () nil
   (list-store list-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (sibling (or null tree-iter)))
 
-(defbinding list-store-prepend () nil
+(defun list-store-insert-after
+    (store sibling &optional data (iter (make-instance 'tree-iter)))
+  (%list-store-insert-after store iter sibling)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %list-store-prepend () nil
   (list-store list-store)
-  ((make-instance 'tree-iter) tree-iter :in-out))
+  (tree-iter tree-iter))
+
+(defun list-store-prepend 
+    (store &optional data (iter (make-instance 'tree-iter)))
+  (%list-store-prepend store iter)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
 
-(defbinding list-store-append () nil
+(defbinding %list-store-append () nil
   (list-store list-store)
-  ((make-instance 'tree-iter) tree-iter :in-out))
+  (tree-iter tree-iter))
+
+(defun list-store-append 
+    (store &optional data (iter (make-instance 'tree-iter)))
+  (%list-store-append store iter)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
 
 (defbinding list-store-clear () nil
   (list-store list-store))
@@ -137,68 +229,553 @@
   (iter tree-iter)
   (psoition (or null tree-iter)))
 
-
 (defbinding list-store-move-after () nil
   (list-store list-store)
   (iter tree-iter)
   (psoition tree-iter))
 
 
+;;; Tree Path
+
+(defbinding %tree-path-free () nil
+  (location pointer))
+
+(defbinding %tree-path-get-indices () pointer
+  (location pointer))
+
+(defbinding %tree-path-get-depth () int
+  (location pointer))
+
+(defun %make-tree-path (path)
+  (let* ((c-vector (make-c-vector 'int (length path) :content path))
+	 (pointer-offset (adjust-offset (size-of 'int) 'pointer))
+	 (location (allocate-memory (+ pointer-offset (size-of 'pointer)))))
+    (funcall (writer-function 'int) (length path) location)
+    (funcall (writer-function 'pointer) c-vector location pointer-offset)
+    location))
+
+(defun %tree-path-to-vector (location)
+  (unless (null-pointer-p location)
+    (let ((indices (%tree-path-get-indices location))
+	  (depth (%tree-path-get-depth location)))
+      (if (null-pointer-p indices)
+	  #()
+	(map-c-vector 'vector #'identity indices 'int depth)))))
+
+(defmacro %with-tree-path ((var path) &body body)
+  (let* ((pointer-offset (adjust-offset (size-of 'int) 'pointer))
+	 (vector-offset (adjust-offset (+ pointer-offset (size-of 'pointer)) 'int)))
+    `(with-memory (,var (+ ,vector-offset (* ,(size-of 'int) (length ,path))))
+      (funcall (writer-function 'int) (length ,path) ,var)
+      (setf (ref-pointer ,var ,pointer-offset) (pointer+ ,var ,vector-offset))
+      (make-c-vector 'int (length ,path) :content ,path :location (pointer+ ,var ,vector-offset))
+      ,@body)))
+
+(eval-when (:compile-toplevel :load-toplevel :execute)
+  (define-type-method alien-type ((type tree-path))
+    (declare (ignore type))
+    (alien-type 'pointer))
+  
+  (define-type-method size-of ((type tree-path) &key inlined)
+    (assert-not-inlined type inlined)
+    (size-of 'pointer))
+  
+  (define-type-method alien-arg-wrapper ((type tree-path) var path style form &optional copy-in-p)
+    (declare (ignore type))
+    (cond
+     ((and (in-arg-p style) copy-in-p)
+      `(with-pointer (,var (%make-tree-path ,path))
+         ,form))
+     ((and (in-arg-p style) (not (out-arg-p style)))
+      `(%with-tree-path (,var ,path)
+         ,form))
+     ((and (in-arg-p style) (out-arg-p style))
+      (let ((tree-path (make-symbol "SYMBOL")))
+	`(%with-tree-path (,tree-path ,path)
+	   (with-pointer (,var ,tree-path)
+             ,form))))
+     ((and (out-arg-p style) (not (in-arg-p style)))
+      `(with-pointer (,var)
+         ,form))))
+
+  (define-type-method to-alien-form ((type tree-path) path &optional copy-p)
+    (declare (ignore type copy-p))
+    `(%make-tree-path ,path))
+  
+  (define-type-method from-alien-form ((type tree-path) location &key (ref :free))
+    (declare (ignore type))
+    `(prog1
+	 (%tree-path-to-vector ,location)
+       ,(when (eq ref :free)
+	  `(%tree-path-free ,location)))))
+  
+(define-type-method to-alien-function ((type tree-path) &optional copy-p)
+  (declare (ignore type))
+  #'%make-tree-path
+  (unless copy-p
+    #'(lambda (tree-path location)
+	(declare (ignore tree-path))
+	(%tree-path-free location))))
+  
+(define-type-method from-alien-function ((type tree-path) &key (ref :free))
+  (declare (ignore type))
+  (if (eq ref :free)
+      #'(lambda (location)
+	  (prog1
+	      (%tree-path-to-vector location)
+	    (%tree-path-free location)))
+    #'(lambda (location)
+	(%tree-path-to-vector location))))
+  
+(define-type-method writer-function ((type tree-path) &key temp inlined)
+  (declare (ignore temp))
+  (assert-not-inlined type inlined)
+  (let ((writer (writer-function 'pointer)))
+    #'(lambda (path location &optional (offset 0))
+	(funcall writer (%make-tree-path path) location offset))))
+
+(define-type-method reader-function ((type tree-path) &key ref inlined)
+  (declare (ignore ref))
+  (assert-not-inlined type inlined)
+  #'(lambda (location &optional (offset 0))
+      (%tree-path-to-vector (ref-pointer location offset))))
+
+(define-type-method destroy-function ((type tree-path) &key temp inlined)
+  (declare (ignore temp))
+  (assert-not-inlined type inlined)
+  #'(lambda (location &optional (offset 0))
+      (%tree-path-free (ref-pointer location offset))))
+
+(defun ensure-tree-path (path)
+  (etypecase path
+    (string (coerce (clg-utils:split-string path :delimiter #\:) 'vector))
+    (vector path)))
+
+
 ;;; Tree Model
 
+(defgeneric tree-model-value (model row column))
+(defgeneric (setf tree-model-value) (value model row column))
+(defgeneric tree-model-row-data (model row))
+(defgeneric (setf tree-model-row-data) (data model row))
+(defgeneric tree-model-column-index (model column))
+(defgeneric tree-model-column-name (model index))
+
+
+(defbinding %tree-row-reference-new () pointer
+  (model tree-model)
+  (path tree-path))
+
+(defmethod allocate-foreign ((reference tree-row-reference) &key model path)
+  (%tree-row-reference-new model path))
+
+(defbinding tree-row-reference-get-path () (or null tree-path)
+  (reference tree-row-reference))
+
+(defbinding (tree-row-reference-valid-p "gtk_tree_row_reference_valid") () boolean
+  (reference tree-row-reference))
+
+
+(defbinding tree-model-get-column-type () gtype
+  (tree-model tree-model)
+  (index int))
+
+(defbinding tree-model-get-iter (model path &optional (iter (make-instance 'tree-iter))) boolean
+  (model tree-model)
+  (iter tree-iter :in/return)
+  (path tree-path))
+
+#?(pkg-exists-p "gtk+-2.0" :atleast-version "2.12.0")
+(defmethod allocate-foreign ((tree-iter tree-iter) &rest initargs)
+  (declare (ignore initargs))
+  (let ((size (foreign-size (class-of tree-iter))))
+    (slice-alloc size)))
+ 
+(defun ensure-tree-iter (model row)
+  (etypecase row
+    (tree-iter row)
+    (tree-path 
+     (multiple-value-bind (valid-p iter) (tree-model-get-iter model row)
+       (if valid-p
+	   iter
+	 (error "Invalid tree path for ~A: ~A" model row))))
+    (tree-row-reference 
+     (let ((path (tree-row-reference-get-path row)))
+       (if path
+	   (ensure-tree-iter model path)
+	 (error "~A not valid" row))))))
+
+(defbinding tree-model-get-path () tree-path
+  (tree-model tree-model)
+  (iter tree-iter))
+
+(defbinding %tree-model-get-value () nil
+  (tree-model tree-model)
+  (iter tree-iter)
+  (column int)
+  (gvalue gvalue))
+
+(defmethod tree-model-value ((model tree-model) row column)
+  (let ((index (tree-model-column-index model column)))
+    (with-gvalue (gvalue)
+      (%tree-model-get-value model (ensure-tree-iter model row) index gvalue))))
+
+(defmethod tree-model-row-data ((model tree-model) row)
+  (coerce
+   (loop
+    with iter = (ensure-tree-iter model row)
+    for index from 0 below (tree-model-n-columns model)
+    collect (tree-model-value model iter index))
+   'vector))
+
+
+(defbinding tree-model-iter-next () boolean
+  (tree-model tree-model)
+  (iter tree-iter :in/return))
+
+(defbinding tree-model-iter-children 
+    (tree-model parent &optional (iter (make-instance 'tree-iter))) boolean
+  (tree-model tree-model)
+  (iter tree-iter :in/return)
+  (parent (or null tree-iter)))
+
+(defbinding (tree-model-iter-has-child-p "gtk_tree_model_iter_has_child") 
+    () boolean
+  (tree-model tree-model)
+  (iter tree-iter))
+
+(defbinding tree-model-iter-n-children (tree-model &optional iter) int
+  (tree-model tree-model)
+  (iter (or null tree-iter)))
+
+(defbinding tree-model-iter-nth-child
+    (tree-model parent n &optional (iter (make-instance 'tree-iter))) boolean
+  (tree-model tree-model)
+  (iter tree-iter :in/return)
+  (parent (or null tree-iter))
+  (n int))
+
+(defbinding tree-model-iter-parent
+    (tree-model child &optional (iter (make-instance 'tree-iter))) boolean
+  (tree-model tree-model)
+  (iter tree-iter :in/return)
+  (child tree-iter))
+
+(define-callback-marshal %tree-model-foreach-callback boolean 
+  (tree-model tree-path tree-iter))
+
+(defbinding %tree-model-foreach (tree-model callback-id) nil
+  (tree-model tree-model)
+  (%tree-model-foreach-callback callback)
+  (callback-id unsigned-int))
+
+(defun tree-model-foreach (model function)
+  (with-callback-function (id function)
+    (%tree-model-foreach model id)))
+
+(defbinding tree-model-row-changed () nil
+  (tree-model tree-model)
+  (path tree-path)
+  (iter tree-iter))
+
+(defbinding tree-model-row-inserted () nil
+  (tree-model tree-model)
+  (path tree-path)
+  (iter tree-iter))
+
+(defbinding tree-model-row-has-child-toggled () nil
+  (tree-model tree-model)
+  (path tree-path)
+  (iter tree-iter))
+
+(defbinding tree-model-row-deleted () nil
+  (tree-model tree-model)
+  (path tree-path)
+  (iter tree-iter))
+
+(defbinding tree-model-rows-reordered () nil
+  (tree-model tree-model)
+  (path tree-path)
+  (iter tree-iter)
+  (new-order int))
+
+(defmethod tree-model-column-index ((model tree-model) column)
+  (or
+   (etypecase column
+     (number column)
+     (string (position column (user-data model 'column-names) :test #'string=))
+     (symbol (position column (user-data model 'column-names))))
+   (error "~A has no column ~S" model column)))
+
+(defmethod tree-model-column-name ((model tree-model) index)
+  (svref (user-data model 'column-names) index))
+
+
+(defmethod (setf tree-model-row-data) ((data list) (model tree-model) (iter tree-iter))
+  (loop
+   for (column value) on data by #'cddr
+   do (setf (tree-model-value model iter column) value))
+  data)
+
+(defmethod (setf tree-model-row-data) ((data vector) (model tree-model) row)
+  (loop
+   with iter = (ensure-tree-iter model row)
+   for index from 0
+   for value across data
+   do (setf (tree-model-value model iter index) value))
+  data)
+
+
+;;; Tree Selection
+
+(define-callback-marshal %tree-selection-callback boolean 
+  (tree-selection tree-model tree-path (path-currently-selected boolean)))
+
+(defbinding tree-selection-set-select-function (selection function) nil
+  (selection tree-selection)
+  (%tree-selection-callback callback)
+  ((register-callback-function function) unsigned-int)
+  (user-data-destroy-callback callback))
+
+(defbinding tree-selection-get-selected 
+    (selection &optional (iter (make-instance 'tree-iter))) boolean
+  (selection tree-selection)
+  (nil null) 
+  (iter tree-iter :in/return))
+
+(define-callback-marshal %tree-selection-foreach-callback nil (tree-model tree-path tree-iter))
+
+(defbinding %tree-selection-selected-foreach (tree-selection callback-id) nil
+  (tree-selection tree-selection)
+  (%tree-selection-foreach-callback callback)
+  (callback-id unsigned-int))
+
+(defun tree-selection-selected-foreach (selection function)
+  (with-callback-function (id function)
+    (%tree-selection-selected-foreach selection id)))
+
+(defbinding tree-selection-get-selected-rows () (glist tree-path)
+  (tree-selection tree-selection)
+  (nil null))
+
+(defbinding tree-selection-count-selected-rows () int
+  (tree-selection tree-selection))
+
+(defbinding %tree-selection-select-path () nil
+  (tree-selection tree-selection)
+  (tree-path tree-path))
+
+(defbinding %tree-selection-unselect-path () nil
+  (tree-selection tree-selection)
+  (tree-path tree-path))
+
+(defbinding %tree-selection-path-is-selected () boolean
+  (tree-selection tree-selection)
+  (tree-path tree-path))
+
+(defbinding %tree-selection-select-iter () nil
+  (tree-selection tree-selection)
+  (tree-iter tree-iter))
+
+(defbinding %tree-selection-unselect-iter () nil
+  (tree-selection tree-selection)
+  (tree-iter tree-iter))
+
+(defbinding %tree-selection-iter-is-selected () boolean
+  (tree-selection tree-selection)
+  (tree-iter tree-iter))
+
+(defun tree-selection-select (selection row)
+  (etypecase row
+    (tree-path (%tree-selection-select-path selection row))
+    (tree-iter (%tree-selection-select-iter selection row))))
+
+(defun tree-selection-unselect (selection row)
+  (etypecase row
+    (tree-path (%tree-selection-unselect-path selection row))
+    (tree-iter (%tree-selection-unselect-iter selection row))))
+
+(defun tree-selection-is-selected-p (selection row)
+  (etypecase row
+    (tree-path (%tree-selection-path-is-selected selection row))
+    (tree-iter (%tree-selection-iter-is-selected selection row))))
+
+(defbinding tree-selection-select-all () nil
+  (tree-selection tree-selection))
+
+(defbinding tree-selection-unselect-all () nil
+  (tree-selection tree-selection))
+
+(defbinding tree-selection-select-range () nil
+  (tree-selection tree-selection)
+  (start tree-path)
+  (end tree-path))
+
+(defbinding tree-selection-unselect-range () nil
+  (tree-selection tree-selection)
+  (start tree-path)
+  (end tree-path))
+
+
+;;; Tree Sortable
+
+(eval-when (:compile-toplevel :load-toplevel :execute)
+  (define-enum-type sort-column (:default -1) (:unsorted -2))
+  (define-enum-type sort-order (:before -1) (:equal 0) (:after 1)))
+
+
+(define-callback-marshal %tree-iter-compare-callback (or int sort-order)
+  (tree-model (a tree-iter) (b tree-iter)))
+
+(defbinding tree-sortable-sort-column-changed () nil
+  (sortable tree-sortable))
+
+(defbinding %tree-sortable-get-sort-column-id () boolean
+  (sortable tree-sortable)
+  (column int :out)
+  (order sort-type :out))
+
+(defun tree-sortable-get-sort-column (sortable)
+  (multiple-value-bind (special-p column order) 
+      (%tree-sortable-get-sort-column-id sortable)
+    (values
+     (if special-p
+	 (int-to-sort-order column)
+       (tree-model-column-name sortable column))
+     order)))
+
+(defbinding (tree-sortable-set-sort-column 
+	     "gtk_tree_sortable_set_sort_column_id") 
+    (sortable column order) nil
+  (sortable tree-sortable)
+  ((etypecase column
+     ((or integer sort-column) column)
+     (symbol (tree-model-column-index sortable column))) 
+   (or sort-column int))
+  (order sort-type))
+
+(defbinding %tree-sortable-set-sort-func (sortable column function) nil
+  (sortable tree-sortable)
+  ((tree-model-column-index sortable column) int)
+  (%tree-iter-compare-callback callback)
+  ((register-callback-function function) unsigned-int)
+  (user-data-destroy-callback callback))
+
+(defbinding %tree-sortable-set-default-sort-func () nil
+  (sortable tree-sortable)
+  (compare-func (or null callback))
+  (callback-id unsigned-int)
+  (destroy-func (or null callback)))
+
+(defun tree-sortable-set-sort-func (sortable column function)
+  "Sets the comparison function used when sorting to be FUNCTION. If
+the current sort column of SORTABLE is the same as COLUMN,
+then the model will sort using this function."
+  (cond
+   ((and (eq column :default) (not function))
+    (%tree-sortable-set-default-sort-func sortable nil 0 nil))
+   ((eq column :default) 
+    (%tree-sortable-set-default-sort-func sortable 
+     %tree-iter-compare-callback
+     (register-callback-function function)
+     user-data-destroy-callback))
+   ((%tree-sortable-set-sort-func sortable column function))))
+
+(defbinding tree-sortable-has-default-sort-func-p () boolean
+  (sortable tree-sortable))
 
 
 ;;; Tree Store
 
 (defbinding %tree-store-set-column-types () nil
   (tree-store tree-store)
-  (n-columns unsigned-int)
-  (columns (vector type-number)))
+  ((length columns) unsigned-int)
+  (columns (vector gtype)))
 
-(defmethod initialize-instance ((tree-store tree-store) &key columns)
+(defmethod initialize-instance ((tree-store tree-store) &key column-types
+				column-names)
   (call-next-method)
-  (%tree-store-set-column-types tree-store (length columns)
-   (map 'vector #'find-type-number columns)))
+  (%tree-store-set-column-types tree-store column-types)
+  (when column-names
+    (setf (user-data tree-store 'column-names) column-names)))
 
 
 (defbinding %tree-store-set-value () nil
   (tree-store tree-store)
   (tree-iter tree-iter)
   (column int)
-  (value gvalue))
+  (gvalue gvalue))
+
+(defmethod (setf tree-model-value) (value (store tree-store) row column)
+  (let* ((index (tree-model-column-index store column))
+	 (type (tree-model-get-column-type store index)))
+    (with-gvalue (gvalue type value)
+      (%tree-store-set-value store (ensure-tree-iter store row) index gvalue)))
+  value)
+
 
 (defbinding tree-store-remove () boolean
   (tree-store tree-store)
   (tree-iter tree-iter))
 
-(defbinding tree-store-insert (tree-store position &optional parent) nil
+(defbinding %tree-store-insert () nil
   (tree-store tree-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (parent (or null tree-iter))
   (position int))
 
-(defbinding tree-store-insert-before (tree-store &optional parent sibling) nil
+(defun tree-store-insert 
+    (store parent position &optional data (iter (make-instance 'tree-iter)))
+  (%tree-store-insert store iter parent position)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %tree-store-insert-before () nil
   (tree-store tree-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (parent (or null tree-iter))
   (sibling (or null tree-iter)))
 
-(defbinding tree-store-insert-after (tree-store &optional parent sibling) nil
+(defun tree-store-insert-before 
+    (store parent sibling &optional data (iter (make-instance 'tree-iter)))
+  (%tree-store-insert-before store iter parent sibling)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %tree-store-insert-after () nil
   (tree-store tree-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (parent (or null tree-iter))
   (sibling (or null tree-iter)))
 
-(defbinding tree-store-prepend (tree-store &optional parent) nil
+(defun tree-store-insert-after 
+    (store parent sibling &optional data (iter (make-instance 'tree-iter)))
+  (%tree-store-insert-after store iter parent sibling)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %tree-store-prepend () nil
   (tree-store tree-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (parent (or null tree-iter)))
 
-(defbinding tree-store-append (tree-store &optional parent) nil
+(defun tree-store-prepend 
+    (store parent &optional data (iter (make-instance 'tree-iter)))
+  (%tree-store-prepend store iter parent)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
+(defbinding %tree-store-append () nil
   (tree-store tree-store)
-  ((make-instance 'tree-iter) tree-iter :in-out)
+  (tree-iter tree-iter)
   (parent (or null tree-iter)))
 
+(defun tree-store-append 
+    (store parent &optional data (iter (make-instance 'tree-iter)))
+  (%tree-store-append store iter parent)
+  (when data (setf (tree-model-row-data store iter) data))
+  iter)
+
 (defbinding (tree-store-is-ancestor-p "gtk_tree_store_is_ancestor") () boolean
   (tree-store tree-store)
   (tree-iter tree-iter)
@@ -236,8 +813,14 @@
 
 ;;; Tree View
 
-(defbinding tree-view-get-selection () tree-selection
-  (tree-view tree-view))
+(defmethod initialize-instance ((tree-view tree-view) &rest initargs 
+				&key column)
+  (declare (ignore column))
+  (call-next-method)
+  (mapc #'(lambda (column)
+	    (tree-view-append-column tree-view column))
+	(get-all initargs :column)))
+
 
 (defbinding tree-view-columns-autosize () nil
   (tree-view tree-view))
@@ -250,7 +833,7 @@
   (tree-view tree-view)
   (tree-view-column tree-view-column))
 
-(defbinding tree-view-insert-column (view columnd position) int
+(defbinding tree-view-insert-column (view column position) int
   (view tree-view)
   (column tree-view-column)
   ((if (eq position :end) -1 position) int))
@@ -264,7 +847,11 @@
   (column tree-view-column)
   (base-column (or null tree-view-column)))
 
-;;(defbinding tree-view-set-column drag-function ...)
+#+(or)
+(define-callback-setter tree-view-set-column-drag-function tree-view boolean
+  (column tree-view-column) 
+  (prev tree-view-column) 
+  (next tree-view-column))
 
 (defbinding tree-view-scroll-to-point () nil
   (tree-view tree-view)
@@ -279,13 +866,8 @@
   (row-align single-float)
   (col-align single-float))
 
-(defbinding tree-view-set-cursor () nil
-  (tree-view tree-view)
-  (path tree-path)
-  (focus-column tree-view-column)
-  (start-editing boolean))
-
-(defbinding tree-view-set-cursor-on-cell () nil
+(defbinding (tree-view-set-cursor "gtk_tree_view_set_cursor_on_cell") 
+    (tree-view path &key focus-column focus-cell start-editing) nil
   (tree-view tree-view)
   (path tree-path)
   (focus-column (or null tree-view-column))
@@ -321,11 +903,11 @@
   (tree-view tree-view)
   (path tree-path))
 
-(def-callback-marshal %tree-view-mapping-func (nil tree-view tree-path))
+(define-callback-marshal %tree-view-mapping-callback nil (tree-view tree-path))
 
-(defbinding %tree-view-map-expanded-rows () nil
+(defbinding %tree-view-map-expanded-rows (tree-view callback-id) nil
   (tree-view tree-view)
-  ((callback %tree-view-mapping-func) pointer)
+  (%tree-view-mapping-callback callback)
   (callback-id unsigned-int))
 
 (defun map-expanded-rows (function tree-view)
@@ -350,17 +932,177 @@
   (tree-view tree-view)
   (path (or null tree-path))
   (column (or null tree-view-column))
-  ((make-instance 'gdk:rectangle) gdk:rectangle :in-out))
+  ((make-instance 'gdk:rectangle) gdk:rectangle :in/return))
 
 (defbinding tree-view-get-background-area () nil
   (tree-view tree-view)
   (path (or null tree-path))
   (column (or null tree-view-column))
-  ((make-instance 'gdk:rectangle) gdk:rectangle :in-out))
+  ((make-instance 'gdk:rectangle) gdk:rectangle :in/return))
 
 (defbinding tree-view-get-visible-rect () nil
   (tree-view tree-view)
-  ((make-instance 'gdk:rectangle) gdk:rectangle :in-out))
+  ((make-instance 'gdk:rectangle) gdk:rectangle :in/return))
 
 ;; and many more functions which we'll add later
 
+
+;;;; Icon View
+
+#?(pkg-exists-p "gtk+-2.0" :atleast-version "2.6.0")
+(progn
+  (defbinding icon-view-get-path-at-pos () tree-path
+    (icon-view icon-view)
+    (x int) (y int))
+
+  (define-callback-marshal %icon-view-foreach-callback nil (icon-view tree-path))
+
+  (defbinding %icon-view-selected-foreach (icon-view callback-id) tree-path
+    (icon-view icon-view)
+    (%icon-view-foreach-callback callback)
+    (callback-id unsigned-int))
+  
+  (defun icon-view-foreach (icon-view function)
+    (with-callback-function (id function)
+      (%icon-view-selected-foreach icon-view id)))
+
+  (defbinding icon-view-select-path () nil
+    (icon-view icon-view)
+    (path tree-path))
+
+  (defbinding icon-view-unselect-path () nil
+    (icon-view icon-view)
+    (path tree-path))
+
+  (defbinding icon-view-path-is-selected-p () boolean
+    (icon-view icon-view)
+    (path tree-path))
+
+  (defbinding icon-view-get-selected-items () (glist tree-path)
+    (icon-view icon-view))
+
+  (defbinding icon-view-select-all () nil
+    (icon-view icon-view))
+
+  (defbinding icon-view-unselect-all () nil
+    (icon-view icon-view))
+  
+  (defbinding icon-view-item-activated () nil
+    (icon-view icon-view)
+    (path tree-path))
+
+  (defbinding %icon-view-set-text-column (column icon-view) nil
+    (icon-view icon-view)
+    ((if (integerp column) 
+	 column 
+       (tree-model-column-index (icon-view-model icon-view) column)) int))
+
+  (defbinding %%icon-view-get-text-column () int
+    (icon-view icon-view))
+
+  (defun %icon-view-get-text-column (icon-view)
+    (tree-model-column-index 
+     (icon-view-model icon-view) 
+     (%%icon-view-get-text-column icon-view)))
+
+  (defun %icon-view-text-column-boundp (icon-view)
+    (not (eql (%%icon-view-get-text-column icon-view) -1)))
+
+
+  (defbinding %icon-view-set-markup-column (column icon-view) nil
+    (icon-view icon-view)
+    ((if (integerp column) 
+	 column 
+       (tree-model-column-index (icon-view-model icon-view) column)) int))
+
+  (defbinding %%icon-view-get-markup-column () int
+    (icon-view icon-view))
+
+  (defun %icon-view-get-markup-column (icon-view)
+    (tree-model-column-index 
+     (icon-view-model icon-view) 
+     (%%icon-view-get-markup-column icon-view)))
+
+  (defun %icon-view-markup-column-boundp (icon-view)
+    (not (eql (%%icon-view-get-markup-column icon-view) -1)))
+
+
+  (defbinding %icon-view-set-pixbuf-column (column icon-view) nil
+    (icon-view icon-view)
+    ((if (integerp column) 
+	 column 
+       (tree-model-column-index (icon-view-model icon-view) column)) int)))
+
+  (defbinding %%icon-view-get-pixbuf-column () int
+    (icon-view icon-view))
+
+  (defun %icon-view-get-pixbuf-column (icon-view)
+    (tree-model-column-index 
+     (icon-view-model icon-view) 
+     (%%icon-view-get-pixbuf-column icon-view)))
+
+  (defun %icon-view-pixbuf-column-boundp (icon-view)
+    (not (eql (%%icon-view-get-pixbuf-column icon-view) -1)))
+
+
+#?(pkg-exists-p "gtk+-2.0" :atleast-version "2.8.0")
+(progn
+  (defbinding icon-view-get-item-at-pos () boolean
+    (icon-view icon-view)
+    (x int)
+    (y int)
+    (tree-path tree-path :out)
+    (cell cell-renderer :out))
+
+  (defbinding icon-view-set-cursor (icon-view path &key cell start-editing) nil
+    (icon-view icon-view)
+    (path tree-path)
+    (cell (or null cell-renderer))
+    (start-editing boolean))
+  
+  (defbinding icon-view-get-cursor () boolean
+    (icon-view icon-view)
+    (path tree-path :out)
+    (cell cell-renderer :out))
+
+  (defbinding icon-view-get-dest-item-at-pos () boolean
+    (icon-view icon-view)
+    (drag-x int)
+    (drag-y int)
+    (tree-path tree-path :out)
+    (pos drop-position :out))
+
+  (defbinding icon-view-create-drag-icon () gdk:pixmap
+    (icon-view icon-view)
+    (tree-path tree-path))
+
+  (defbinding icon-view-scroll-to-path (icon-view tree-path &key row-align column-align) nil
+    (icon-view icon-view)
+    (tree-path tree-path)
+    ((or row-align column-align) boolean)
+    (row-align single-float)
+    (column-align single-float))
+
+  (defbinding icon-view-get-visible-range () boolean
+    (icon-view icon-view)
+    (start-path tree-path :out)
+    (end-path tree-path :out))
+
+  (defbinding icon-view-enable-model-drag-source () nil
+    (icon-view icon-view)
+    (start-button-mask gdk:modifier-type)
+    (targets (vector (inlined target-entry)))
+    ((length targets) unsigned-int)
+    (actions gdk:drag-action))
+
+  (defbinding icon-view-enable-model-drag-dest () nil
+    (icon-view icon-view)
+    (targets (vector (inlined target-entry)))
+    ((length targets) unsigned-int)
+    (actions gdk:drag-action))
+
+  (defbinding icon-view-unset-model-drag-source () nil
+    (icon-view icon-view))
+
+  (defbinding icon-view-unset-model-drag-dest () nil
+    (icon-view icon-view)))