# HG changeset patch
# User Sadrul Habib Chowdhury <imadil@gmail.com>
# Date 1184376704 0
# Node ID da550279d39014d92a8b8e519549920ed2cd2f40
# Parent  09db6fec9dcec5cbae0643033977f061e7f7b535
Use gobject properties instead of hacks when creating columns in a tree.

diff -r 09db6fec9dce -r da550279d390 finch/libgnt/gntmenu.c
--- a/finch/libgnt/gntmenu.c	Fri Jul 13 16:50:03 2007 +0000
+++ b/finch/libgnt/gntmenu.c	Sat Jul 14 01:31:44 2007 +0000
@@ -423,7 +423,7 @@
 		widget->priv.y = 0;
 	} else {
 		GNT_TREE(widget)->show_separator = FALSE;
-		_gnt_tree_init_internals(GNT_TREE(widget), NUM_COLUMNS);
+		g_object_set(G_OBJECT(widget), "columns", NUM_COLUMNS, NULL);
 		gnt_tree_set_col_width(GNT_TREE(widget), ITEM_TRIGGER, 3);
 		gnt_tree_set_column_resizable(GNT_TREE(widget), ITEM_TRIGGER, FALSE);
 		gnt_tree_set_col_width(GNT_TREE(widget), ITEM_SUBMENU, 1);
diff -r 09db6fec9dce -r da550279d390 finch/libgnt/gnttree.c
--- a/finch/libgnt/gnttree.c	Fri Jul 13 16:50:03 2007 +0000
+++ b/finch/libgnt/gnttree.c	Sat Jul 14 01:31:44 2007 +0000
@@ -37,6 +37,12 @@
 
 enum
 {
+	PROP_0,
+	PROP_COLUMNS,
+};
+
+enum
+{
 	SIG_SELECTION_CHANGED,
 	SIG_SCROLLED,
 	SIG_TOGGLED,
@@ -76,6 +82,7 @@
 };
 
 static void tree_selection_changed(GntTree *, GntTreeRow *, GntTreeRow *);
+static void _gnt_tree_init_internals(GntTree *tree, int col);
 
 static GntWidgetClass *parent_class = NULL;
 static guint signals[SIGS] = { 0 };
@@ -906,9 +913,39 @@
 }
 
 static void
+gnt_tree_set_property(GObject *obj, guint prop_id, const GValue *value,
+		GParamSpec *spec)
+{
+	GntTree *tree = GNT_TREE(obj);
+	switch (prop_id) {
+		case PROP_COLUMNS:
+			_gnt_tree_init_internals(tree, g_value_get_int(value));
+			break;
+		default:
+			break;
+	}
+}
+
+static void
+gnt_tree_get_property(GObject *obj, guint prop_id, GValue *value,
+		GParamSpec *spec)
+{
+	GntTree *tree = GNT_TREE(obj);
+	switch (prop_id) {
+		case PROP_COLUMNS:
+			g_value_set_int(value, tree->ncol);
+			break;
+		default:
+			break;
+	}
+}
+
+static void
 gnt_tree_class_init(GntTreeClass *klass)
 {
 	GntBindableClass *bindable = GNT_BINDABLE_CLASS(klass);
+	GObjectClass *gclass = G_OBJECT_CLASS(klass);
+
 	parent_class = GNT_WIDGET_CLASS(klass);
 	parent_class->destroy = gnt_tree_destroy;
 	parent_class->draw = gnt_tree_draw;
@@ -918,6 +955,17 @@
 	parent_class->clicked = gnt_tree_clicked;
 	parent_class->size_changed = gnt_tree_size_changed;
 
+	gclass->set_property = gnt_tree_set_property;
+	gclass->get_property = gnt_tree_get_property;
+	g_object_class_install_property(gclass,
+			PROP_COLUMNS,
+			g_param_spec_int("columns", "Columns",
+				"Number of columns in the tree.",
+				1, G_MAXINT, 1,
+				G_PARAM_READWRITE|G_PARAM_STATIC_NAME|G_PARAM_STATIC_NICK|G_PARAM_STATIC_BLURB
+			)
+		);
+
 	signals[SIG_SELECTION_CHANGED] = 
 		g_signal_new("selection-changed",
 					 G_TYPE_FROM_CLASS(klass),
@@ -1501,8 +1549,10 @@
 	tree_selection_changed(tree, row, tree->current);
 }
 
-void _gnt_tree_init_internals(GntTree *tree, int col)
+static void _gnt_tree_init_internals(GntTree *tree, int col)
 {
+	gnt_tree_destroy(GNT_WIDGET(tree));
+
 	tree->ncol = col;
 	tree->hash = g_hash_table_new_full(g_direct_hash, g_direct_equal, NULL, free_tree_row);
 	tree->columns = g_new0(struct _GntTreeColInfo, col);
@@ -1513,15 +1563,15 @@
 	}
 	tree->list = NULL;
 	tree->show_title = FALSE;
+	g_object_notify(G_OBJECT(tree), "columns");
 }
 
 GntWidget *gnt_tree_new_with_columns(int col)
 {
-	GntWidget *widget = g_object_new(GNT_TYPE_TREE, NULL);
-	GntTree *tree = GNT_TREE(widget);
+	GntWidget *widget = g_object_new(GNT_TYPE_TREE,
+			"columns", col,
+			NULL);
 
-	_gnt_tree_init_internals(tree, col);
-	
 	GNT_WIDGET_SET_FLAGS(widget, GNT_WIDGET_NO_SHADOW);
 	gnt_widget_set_take_focus(widget, TRUE);
 
diff -r 09db6fec9dce -r da550279d390 finch/libgnt/gnttree.h
--- a/finch/libgnt/gnttree.h	Fri Jul 13 16:50:03 2007 +0000
+++ b/finch/libgnt/gnttree.h	Sat Jul 14 01:31:44 2007 +0000
@@ -506,9 +506,4 @@
 
 G_END_DECLS
 
-/* The following functions should NOT be used by applications. */
-
-/* This should be called by the subclasses of GntTree's in their _new function */
-void _gnt_tree_init_internals(GntTree *tree, int col);
-
 #endif /* GNT_TREE_H */