aboutsummaryrefslogtreecommitdiffstats
path: root/widgets/table
diff options
context:
space:
mode:
authorFederico Mena Quintero <federico@ximian.com>2001-09-25 05:31:50 +0800
committerFederico Mena Quintero <federico@src.gnome.org>2001-09-25 05:31:50 +0800
commit40226dadd54daf0955407e40673a895fc2f7445c (patch)
treec269da50e7693451ef48b2c1e074cc0d17920262 /widgets/table
parent8ab52c05d984af14cf5583ebd19883a9d0ec1b98 (diff)
downloadgsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar.gz
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar.bz2
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar.lz
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar.xz
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.tar.zst
gsoc2013-evolution-40226dadd54daf0955407e40673a895fc2f7445c.zip
Assert that the virtual method is implemented instead of being forgiving.
2001-09-24 Federico Mena Quintero <federico@ximian.com> * e-table-group.c (e_table_group_add): Assert that the virtual method is implemented instead of being forgiving. (e_table_group_add_array): Likewise. (e_table_group_add_all): Likewise. (e_table_group_remove): Likewise. (e_table_group_increment): Likewise. (e_table_group_decrement): Likewise. (e_table_group_row_count): Likewise. (e_table_group_set_focus): Likewise. (e_table_group_get_focus): Likewise. (e_table_group_get_focus_column): Likewise. (e_table_group_get_printable): Likewise. (e_table_group_compute_location): Likewise. (e_table_group_get_cell_geometry): Likewise. (etg_destroy): Clear the fields after we unref them. * e-table-group-leaf.c (etgl_destroy): Clear the fields after we unref them. (e_table_group_leaf_construct): Subsets should not be sunk. Do not ref/sink the subset. * e-table-header.c (e_table_header_add_column): Document the fact that the header assumes ownership of the column. svn path=/trunk/; revision=13102
Diffstat (limited to 'widgets/table')
-rw-r--r--widgets/table/e-table-group-leaf.c153
-rw-r--r--widgets/table/e-table-group.c132
-rw-r--r--widgets/table/e-table-header.c3
-rw-r--r--widgets/table/e-table.c2
4 files changed, 179 insertions, 111 deletions
diff --git a/widgets/table/e-table-group-leaf.c b/widgets/table/e-table-group-leaf.c
index e4c46747dd..c721b7d49a 100644
--- a/widgets/table/e-table-group-leaf.c
+++ b/widgets/table/e-table-group-leaf.c
@@ -45,8 +45,12 @@ static void
etgl_destroy (GtkObject *object)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF(object);
- if (etgl->ets)
+
+ if (etgl->ets) {
gtk_object_unref (GTK_OBJECT(etgl->ets));
+ etgl->ets = NULL;
+ }
+
if (etgl->item) {
if (etgl->etgl_cursor_change_id != 0)
gtk_signal_disconnect (GTK_OBJECT (etgl->item),
@@ -63,13 +67,12 @@ etgl_destroy (GtkObject *object)
if (etgl->etgl_click_id != 0)
gtk_signal_disconnect (GTK_OBJECT (etgl->item),
etgl->etgl_click_id);
- if (etgl->etgl_key_press_id != 0)
+ if (etgl->etgl_key_press_id != 0)
gtk_signal_disconnect (GTK_OBJECT (etgl->item),
etgl->etgl_key_press_id);
- if (etgl->etgl_start_drag_id != 0)
+ if (etgl->etgl_start_drag_id != 0)
gtk_signal_disconnect (GTK_OBJECT (etgl->item),
etgl->etgl_start_drag_id);
- gtk_object_destroy (GTK_OBJECT(etgl->item));
etgl->etgl_cursor_change_id = 0;
etgl->etgl_cursor_activated_id = 0;
@@ -78,10 +81,16 @@ etgl_destroy (GtkObject *object)
etgl->etgl_click_id = 0;
etgl->etgl_key_press_id = 0;
etgl->etgl_start_drag_id = 0;
+
+ gtk_object_destroy (GTK_OBJECT(etgl->item));
etgl->item = NULL;
}
- if (etgl->selection_model)
+
+ if (etgl->selection_model) {
gtk_object_unref (GTK_OBJECT(etgl->selection_model));
+ etgl->selection_model = NULL;
+ }
+
if (GTK_OBJECT_CLASS (etgl_parent_class)->destroy)
GTK_OBJECT_CLASS (etgl_parent_class)->destroy (object);
}
@@ -94,17 +103,21 @@ e_table_group_leaf_construct (GnomeCanvasGroup *parent,
ETableModel *model,
ETableSortInfo *sort_info)
{
- etgl->is_grouped = e_table_sort_info_grouping_get_count(sort_info);
+ etgl->is_grouped = e_table_sort_info_grouping_get_count(sort_info) > 0 ? TRUE : FALSE;
+
if (etgl->is_grouped)
- etgl->ets = E_TABLE_SUBSET(e_table_sorted_variable_new (model, full_header, sort_info));
+ etgl->ets = E_TABLE_SUBSET(e_table_sorted_variable_new (model,
+ full_header,
+ sort_info));
else
- etgl->ets = E_TABLE_SUBSET(e_table_sorted_new (model, full_header, sort_info));
- gtk_object_ref(GTK_OBJECT(etgl->ets));
- gtk_object_sink(GTK_OBJECT(etgl->ets));
+ etgl->ets = E_TABLE_SUBSET(e_table_sorted_new (model,
+ full_header,
+ sort_info));
+
e_table_group_construct (parent, E_TABLE_GROUP (etgl), full_header, header, model);
}
-/**
+/**
* e_table_group_leaf_new
* @parent: The %GnomeCanvasGroup to create a child of.
* @full_header: The full header of the %ETable.
@@ -127,7 +140,7 @@ e_table_group_leaf_new (GnomeCanvasGroup *parent,
ETableGroupLeaf *etgl;
g_return_val_if_fail (parent != NULL, NULL);
-
+
etgl = gtk_type_new (e_table_group_leaf_get_type ());
e_table_group_leaf_construct (parent, etgl, full_header,
@@ -139,18 +152,21 @@ static void
etgl_cursor_change (GtkObject *object, gint row, ETableGroupLeaf *etgl)
{
if (row < E_TABLE_SUBSET(etgl->ets)->n_map)
- e_table_group_cursor_change (E_TABLE_GROUP(etgl), E_TABLE_SUBSET(etgl->ets)->map_table[row]);
+ e_table_group_cursor_change (E_TABLE_GROUP(etgl),
+ E_TABLE_SUBSET(etgl->ets)->map_table[row]);
}
static void
etgl_cursor_activated (GtkObject *object, gint view_row, ETableGroupLeaf *etgl)
{
if (view_row < E_TABLE_SUBSET(etgl->ets)->n_map)
- e_table_group_cursor_activated (E_TABLE_GROUP(etgl), E_TABLE_SUBSET(etgl->ets)->map_table[view_row]);
+ e_table_group_cursor_activated (E_TABLE_GROUP(etgl),
+ E_TABLE_SUBSET(etgl->ets)->map_table[view_row]);
}
static void
-etgl_double_click (GtkObject *object, gint model_row, gint model_col, GdkEvent *event, ETableGroupLeaf *etgl)
+etgl_double_click (GtkObject *object, gint model_row, gint model_col, GdkEvent *event,
+ ETableGroupLeaf *etgl)
{
e_table_group_double_click (E_TABLE_GROUP(etgl), model_row, model_col, event);
}
@@ -159,22 +175,30 @@ static gint
etgl_key_press (GtkObject *object, gint row, gint col, GdkEvent *event, ETableGroupLeaf *etgl)
{
if (row < E_TABLE_SUBSET(etgl->ets)->n_map && row >= 0)
- return e_table_group_key_press (E_TABLE_GROUP(etgl), E_TABLE_SUBSET(etgl->ets)->map_table[row], col, event);
+ return e_table_group_key_press (E_TABLE_GROUP(etgl),
+ E_TABLE_SUBSET(etgl->ets)->map_table[row],
+ col,
+ event);
else
return 0;
}
static gint
-etgl_start_drag (GtkObject *object, gint model_row, gint model_col, GdkEvent *event, ETableGroupLeaf *etgl)
+etgl_start_drag (GtkObject *object, gint model_row, gint model_col, GdkEvent *event,
+ ETableGroupLeaf *etgl)
{
return e_table_group_start_drag (E_TABLE_GROUP(etgl), model_row, model_col, event);
}
static gint
-etgl_right_click (GtkObject *object, gint view_row, gint model_col, GdkEvent *event, ETableGroupLeaf *etgl)
+etgl_right_click (GtkObject *object, gint view_row, gint model_col, GdkEvent *event,
+ ETableGroupLeaf *etgl)
{
if (view_row < E_TABLE_SUBSET(etgl->ets)->n_map)
- return e_table_group_right_click (E_TABLE_GROUP(etgl), E_TABLE_SUBSET(etgl->ets)->map_table[view_row], model_col, event);
+ return e_table_group_right_click (E_TABLE_GROUP(etgl),
+ E_TABLE_SUBSET(etgl->ets)->map_table[view_row],
+ model_col,
+ event);
else
return 0;
}
@@ -183,7 +207,10 @@ static gint
etgl_click (GtkObject *object, gint row, gint col, GdkEvent *event, ETableGroupLeaf *etgl)
{
if (row < E_TABLE_SUBSET(etgl->ets)->n_map)
- return e_table_group_click (E_TABLE_GROUP(etgl), E_TABLE_SUBSET(etgl->ets)->map_table[row], col, event);
+ return e_table_group_click (E_TABLE_GROUP(etgl),
+ E_TABLE_SUBSET(etgl->ets)->map_table[row],
+ col,
+ event);
else
return 0;
}
@@ -192,12 +219,14 @@ static void
etgl_reflow (GnomeCanvasItem *item, gint flags)
{
ETableGroupLeaf *leaf = E_TABLE_GROUP_LEAF(item);
+
gtk_object_get(GTK_OBJECT(leaf->item),
"height", &leaf->height,
NULL);
gtk_object_get(GTK_OBJECT(leaf->item),
"width", &leaf->width,
NULL);
+
e_canvas_item_request_parent_reflow (item);
}
@@ -209,27 +238,51 @@ etgl_realize (GnomeCanvasItem *item)
if (GNOME_CANVAS_ITEM_CLASS (etgl_parent_class)->realize)
GNOME_CANVAS_ITEM_CLASS (etgl_parent_class)->realize (item);
- etgl->item = E_TABLE_ITEM(gnome_canvas_item_new (GNOME_CANVAS_GROUP(etgl),
- e_table_item_get_type (),
- "ETableHeader", E_TABLE_GROUP(etgl)->header,
- "ETableModel", etgl->ets,
- "alternating_row_colors", etgl->alternating_row_colors,
- "horizontal_draw_grid", etgl->horizontal_draw_grid,
- "vertical_draw_grid", etgl->vertical_draw_grid,
- "drawfocus", etgl->draw_focus,
- "cursor_mode", etgl->cursor_mode,
- "minimum_width", etgl->minimum_width,
- "length_threshold", etgl->length_threshold,
- "selection_model", etgl->selection_model,
- NULL));
-
- etgl->etgl_cursor_change_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "cursor_change", GTK_SIGNAL_FUNC(etgl_cursor_change), etgl);
- etgl->etgl_cursor_activated_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "cursor_activated", GTK_SIGNAL_FUNC(etgl_cursor_activated), etgl);
- etgl->etgl_double_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "double_click", GTK_SIGNAL_FUNC(etgl_double_click), etgl);
- etgl->etgl_right_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "right_click", GTK_SIGNAL_FUNC(etgl_right_click), etgl);
- etgl->etgl_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "click", GTK_SIGNAL_FUNC(etgl_click), etgl);
- etgl->etgl_key_press_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "key_press", GTK_SIGNAL_FUNC(etgl_key_press), etgl);
- etgl->etgl_start_drag_id = gtk_signal_connect (GTK_OBJECT(etgl->item), "start_drag", GTK_SIGNAL_FUNC(etgl_start_drag), etgl);
+ etgl->item = E_TABLE_ITEM(gnome_canvas_item_new (
+ GNOME_CANVAS_GROUP(etgl),
+ e_table_item_get_type (),
+ "ETableHeader", E_TABLE_GROUP(etgl)->header,
+ "ETableModel", etgl->ets,
+ "alternating_row_colors", etgl->alternating_row_colors,
+ "horizontal_draw_grid", etgl->horizontal_draw_grid,
+ "vertical_draw_grid", etgl->vertical_draw_grid,
+ "drawfocus", etgl->draw_focus,
+ "cursor_mode", etgl->cursor_mode,
+ "minimum_width", etgl->minimum_width,
+ "length_threshold", etgl->length_threshold,
+ "selection_model", etgl->selection_model,
+ NULL));
+
+ etgl->etgl_cursor_change_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "cursor_change",
+ GTK_SIGNAL_FUNC(etgl_cursor_change),
+ etgl);
+ etgl->etgl_cursor_activated_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "cursor_activated",
+ GTK_SIGNAL_FUNC(etgl_cursor_activated),
+ etgl);
+ etgl->etgl_double_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "double_click",
+ GTK_SIGNAL_FUNC(etgl_double_click),
+ etgl);
+
+ etgl->etgl_right_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "right_click",
+ GTK_SIGNAL_FUNC(etgl_right_click),
+ etgl);
+ etgl->etgl_click_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "click",
+ GTK_SIGNAL_FUNC(etgl_click),
+ etgl);
+ etgl->etgl_key_press_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "key_press",
+ GTK_SIGNAL_FUNC(etgl_key_press),
+ etgl);
+ etgl->etgl_start_drag_id = gtk_signal_connect (GTK_OBJECT(etgl->item),
+ "start_drag",
+ GTK_SIGNAL_FUNC(etgl_start_drag),
+ etgl);
+
e_canvas_item_request_reflow(item);
}
@@ -237,6 +290,7 @@ static void
etgl_add (ETableGroup *etg, gint row)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
e_table_subset_variable_add (E_TABLE_SUBSET_VARIABLE(etgl->ets), row);
}
@@ -246,6 +300,7 @@ static void
etgl_add_array (ETableGroup *etg, const gint *array, gint count)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
e_table_subset_variable_add_array (E_TABLE_SUBSET_VARIABLE(etgl->ets), array, count);
}
@@ -255,6 +310,7 @@ static void
etgl_add_all (ETableGroup *etg)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
e_table_subset_variable_add_all (E_TABLE_SUBSET_VARIABLE(etgl->ets));
}
@@ -264,6 +320,7 @@ static gboolean
etgl_remove (ETableGroup *etg, gint row)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
return e_table_subset_variable_remove (E_TABLE_SUBSET_VARIABLE(etgl->ets), row);
}
@@ -274,6 +331,7 @@ static void
etgl_increment (ETableGroup *etg, gint position, gint amount)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
e_table_subset_variable_increment (E_TABLE_SUBSET_VARIABLE(etgl->ets), position, amount);
}
@@ -283,6 +341,7 @@ static void
etgl_decrement (ETableGroup *etg, gint position, gint amount)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (E_IS_TABLE_SUBSET_VARIABLE(etgl->ets)) {
e_table_subset_variable_decrement (E_TABLE_SUBSET_VARIABLE(etgl->ets), position, amount);
}
@@ -292,6 +351,7 @@ static int
etgl_row_count (ETableGroup *etg)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
return e_table_model_row_count(E_TABLE_MODEL(etgl->ets));
}
@@ -299,6 +359,7 @@ static void
etgl_set_focus (ETableGroup *etg, EFocus direction, gint view_col)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
if (direction == E_FOCUS_END) {
e_table_item_set_cursor (etgl->item, view_col, e_table_model_row_count(E_TABLE_MODEL(etgl->ets)) - 1);
} else {
@@ -310,6 +371,7 @@ static gint
etgl_get_focus_column (ETableGroup *etg)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
return e_table_item_get_focused_column (etgl->item);
}
@@ -317,6 +379,7 @@ static EPrintable *
etgl_get_printable (ETableGroup *etg)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
return e_table_item_get_printable (etgl->item);
}
@@ -324,6 +387,7 @@ static void
etgl_compute_location (ETableGroup *etg, int *x, int *y, int *row, int *col)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
e_table_item_compute_location (etgl->item, x, y, row, col);
}
@@ -331,6 +395,7 @@ static void
etgl_get_cell_geometry (ETableGroup *etg, int *row, int *col, int *x, int *y, int *width, int *height)
{
ETableGroupLeaf *etgl = E_TABLE_GROUP_LEAF (etg);
+
e_table_item_get_cell_geometry (etgl->item, row, col, x, y, width, height);
}
@@ -463,7 +528,7 @@ etgl_class_init (GtkObjectClass *object_class)
object_class->get_arg = etgl_get_arg;
item_class->realize = etgl_realize;
-
+
etgl_parent_class = gtk_type_class (PARENT_TYPE);
e_group_class->add = etgl_add;
@@ -494,9 +559,9 @@ etgl_class_init (GtkObjectClass *object_class)
gtk_object_add_arg_type ("ETableGroupLeaf::selection_model", E_SELECTION_MODEL_TYPE,
GTK_ARG_WRITABLE, ARG_SELECTION_MODEL);
- gtk_object_add_arg_type ("ETableGroupLeaf::height", GTK_TYPE_DOUBLE,
+ gtk_object_add_arg_type ("ETableGroupLeaf::height", GTK_TYPE_DOUBLE,
GTK_ARG_READABLE, ARG_HEIGHT);
- gtk_object_add_arg_type ("ETableGroupLeaf::width", GTK_TYPE_DOUBLE,
+ gtk_object_add_arg_type ("ETableGroupLeaf::width", GTK_TYPE_DOUBLE,
GTK_ARG_READWRITE, ARG_WIDTH);
gtk_object_add_arg_type ("ETableGroupLeaf::minimum_width", GTK_TYPE_DOUBLE,
GTK_ARG_READWRITE, ARG_MINIMUM_WIDTH);
diff --git a/widgets/table/e-table-group.c b/widgets/table/e-table-group.c
index d9174452be..32c972ed40 100644
--- a/widgets/table/e-table-group.c
+++ b/widgets/table/e-table-group.c
@@ -44,17 +44,27 @@ static void
etg_destroy (GtkObject *object)
{
ETableGroup *etg = E_TABLE_GROUP(object);
- if (etg->header)
+
+ if (etg->header) {
gtk_object_unref (GTK_OBJECT(etg->header));
- if (etg->full_header)
+ etg->header = NULL;
+ }
+
+ if (etg->full_header) {
gtk_object_unref (GTK_OBJECT(etg->full_header));
- if (etg->model)
+ etg->full_header = NULL;
+ }
+
+ if (etg->model) {
gtk_object_unref (GTK_OBJECT(etg->model));
+ etg->model = NULL;
+ }
+
if (GTK_OBJECT_CLASS (etg_parent_class)->destroy)
GTK_OBJECT_CLASS (etg_parent_class)->destroy (object);
}
-/**
+/**
* e_table_group_new
* @parent: The %GnomeCanvasGroup to create a child of.
* @full_header: The full header of the %ETable.
@@ -81,7 +91,7 @@ e_table_group_new (GnomeCanvasGroup *parent,
int n)
{
g_return_val_if_fail (model != NULL, NULL);
-
+
if (n < e_table_sort_info_grouping_get_count(sort_info)) {
return e_table_group_container_new (parent, full_header, header, model, sort_info, n);
} else {
@@ -90,7 +100,7 @@ e_table_group_new (GnomeCanvasGroup *parent,
return NULL;
}
-/**
+/**
* e_table_group_construct
* @parent: The %GnomeCanvasGroup to create a child of.
* @etg: The %ETableGroup to construct.
@@ -116,7 +126,7 @@ e_table_group_construct (GnomeCanvasGroup *parent,
gnome_canvas_item_constructv (GNOME_CANVAS_ITEM (etg), parent, 0, NULL);
}
-/**
+/**
* e_table_group_add
* @etg: The %ETableGroup to add a row to
* @row: The row to add.
@@ -131,11 +141,11 @@ e_table_group_add (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->add)
- ETG_CLASS (etg)->add (etg, row);
+ g_assert (ETG_CLASS (etg)->add != NULL);
+ ETG_CLASS (etg)->add (etg, row);
}
-/**
+/**
* e_table_group_add_array
* @etg: The %ETableGroup to add to
* @array: The array to add.
@@ -152,11 +162,11 @@ e_table_group_add_array (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->add_array)
- ETG_CLASS (etg)->add_array (etg, array, count);
+ g_assert (ETG_CLASS (etg)->add_array != NULL);
+ ETG_CLASS (etg)->add_array (etg, array, count);
}
-/**
+/**
* e_table_group_add_all
* @etg: The %ETableGroup to add to
*
@@ -169,11 +179,11 @@ e_table_group_add_all (ETableGroup *etg)
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->add_all)
- ETG_CLASS (etg)->add_all (etg);
+ g_assert (ETG_CLASS (etg)->add_all != NULL);
+ ETG_CLASS (etg)->add_all (etg);
}
-/**
+/**
* e_table_group_remove
* @etg: The %ETableGroup to remove a row from
* @row: The row to remove.
@@ -191,13 +201,11 @@ e_table_group_remove (ETableGroup *etg,
g_return_val_if_fail (etg != NULL, FALSE);
g_return_val_if_fail (E_IS_TABLE_GROUP (etg), FALSE);
- if (ETG_CLASS (etg)->remove)
- return ETG_CLASS (etg)->remove (etg, row);
- else
- return FALSE;
+ g_assert (ETG_CLASS (etg)->remove != NULL);
+ return ETG_CLASS (etg)->remove (etg, row);
}
-/**
+/**
* e_table_group_increment
* @etg: The %ETableGroup to increment
* @position: The position to increment from
@@ -215,11 +223,11 @@ e_table_group_increment (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->increment)
- ETG_CLASS (etg)->increment (etg, position, amount);
+ g_assert (ETG_CLASS (etg)->increment != NULL);
+ ETG_CLASS (etg)->increment (etg, position, amount);
}
-/**
+/**
* e_table_group_increment
* @etg: The %ETableGroup to decrement
* @position: The position to decrement from
@@ -237,11 +245,11 @@ e_table_group_decrement (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->decrement)
- ETG_CLASS (etg)->decrement (etg, position, amount);
+ g_assert (ETG_CLASS (etg)->decrement != NULL);
+ ETG_CLASS (etg)->decrement (etg, position, amount);
}
-/**
+/**
* e_table_group_increment
* @etg: The %ETableGroup to count
*
@@ -253,15 +261,13 @@ gint
e_table_group_row_count (ETableGroup *etg)
{
g_return_val_if_fail (etg != NULL, 0);
- g_return_val_if_fail (E_IS_TABLE_GROUP (etg), 0);
+ g_return_val_if_fail (E_IS_TABLE_GROUP (etg), -1);
- if (ETG_CLASS (etg)->row_count)
- return ETG_CLASS (etg)->row_count (etg);
- else
- return 0;
+ g_assert (ETG_CLASS (etg)->row_count != NULL);
+ return ETG_CLASS (etg)->row_count (etg);
}
-/**
+/**
* e_table_group_set_focus
* @etg: The %ETableGroup to set
* @direction: The direction the focus is coming from.
@@ -278,11 +284,11 @@ e_table_group_set_focus (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->set_focus)
- ETG_CLASS (etg)->set_focus (etg, direction, view_col);
+ g_assert (ETG_CLASS (etg)->set_focus != NULL);
+ ETG_CLASS (etg)->set_focus (etg, direction, view_col);
}
-/**
+/**
* e_table_group_get_focus
* @etg: The %ETableGroup to check
*
@@ -296,13 +302,11 @@ e_table_group_get_focus (ETableGroup *etg)
g_return_val_if_fail (etg != NULL, FALSE);
g_return_val_if_fail (E_IS_TABLE_GROUP (etg), FALSE);
- if (ETG_CLASS (etg)->get_focus)
- return ETG_CLASS (etg)->get_focus (etg);
- else
- return FALSE;
+ g_assert (ETG_CLASS (etg)->get_focus != NULL);
+ return ETG_CLASS (etg)->get_focus (etg);
}
-/**
+/**
* e_table_group_get_focus_column
* @etg: The %ETableGroup to check
*
@@ -314,15 +318,13 @@ gint
e_table_group_get_focus_column (ETableGroup *etg)
{
g_return_val_if_fail (etg != NULL, FALSE);
- g_return_val_if_fail (E_IS_TABLE_GROUP (etg), FALSE);
+ g_return_val_if_fail (E_IS_TABLE_GROUP (etg), -1);
- if (ETG_CLASS (etg)->get_focus_column)
- return ETG_CLASS (etg)->get_focus_column (etg);
- else
- return -1;
+ g_assert (ETG_CLASS (etg)->get_focus_column != NULL);
+ return ETG_CLASS (etg)->get_focus_column (etg);
}
-/**
+/**
* e_table_group_get_printable
* @etg: %ETableGroup which will be printed
*
@@ -337,13 +339,11 @@ e_table_group_get_printable (ETableGroup *etg)
g_return_val_if_fail (etg != NULL, NULL);
g_return_val_if_fail (E_IS_TABLE_GROUP (etg), NULL);
- if (ETG_CLASS (etg)->get_printable)
- return ETG_CLASS (etg)->get_printable (etg);
- else
- return NULL;
+ g_assert (ETG_CLASS (etg)->get_printable != NULL);
+ return ETG_CLASS (etg)->get_printable (etg);
}
-/**
+/**
* e_table_group_compute_location
* @eti: %ETableGroup to look in.
* @x: A pointer to the x location to find in the %ETableGroup.
@@ -363,11 +363,11 @@ e_table_group_compute_location (ETableGroup *etg, int *x, int *y, int *row, int
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->compute_location)
- ETG_CLASS (etg)->compute_location (etg, x, y, row, col);
+ g_assert (ETG_CLASS (etg)->compute_location != NULL);
+ ETG_CLASS (etg)->compute_location (etg, x, y, row, col);
}
-/**
+/**
* e_table_group_get_position
* @eti: %ETableGroup to look in.
* @x: A pointer to the location to store the found x location in.
@@ -393,11 +393,11 @@ e_table_group_get_cell_geometry (ETableGroup *etg,
g_return_if_fail (etg != NULL);
g_return_if_fail (E_IS_TABLE_GROUP (etg));
- if (ETG_CLASS (etg)->get_cell_geometry)
- ETG_CLASS (etg)->get_cell_geometry (etg, row, col, x, y, width, height);
+ g_assert (ETG_CLASS (etg)->get_cell_geometry != NULL);
+ ETG_CLASS (etg)->get_cell_geometry (etg, row, col, x, y, width, height);
}
-/**
+/**
* e_table_group_cursor_change
* @eti: %ETableGroup to emit the signal on
* @row: The new cursor row (model row)
@@ -415,7 +415,7 @@ e_table_group_cursor_change (ETableGroup *e_table_group, gint row)
row);
}
-/**
+/**
* e_table_group_cursor_activated
* @eti: %ETableGroup to emit the signal on
* @row: The cursor row (model row)
@@ -433,7 +433,7 @@ e_table_group_cursor_activated (ETableGroup *e_table_group, gint row)
row);
}
-/**
+/**
* e_table_group_double_click
* @eti: %ETableGroup to emit the signal on
* @row: The row clicked on (model row)
@@ -453,7 +453,7 @@ e_table_group_double_click (ETableGroup *e_table_group, gint row, gint col, GdkE
row, col, event);
}
-/**
+/**
* e_table_group_right_click
* @eti: %ETableGroup to emit the signal on
* @row: The row clicked on (model row)
@@ -477,7 +477,7 @@ e_table_group_right_click (ETableGroup *e_table_group, gint row, gint col, GdkEv
return return_val;
}
-/**
+/**
* e_table_group_click
* @eti: %ETableGroup to emit the signal on
* @row: The row clicked on (model row)
@@ -501,7 +501,7 @@ e_table_group_click (ETableGroup *e_table_group, gint row, gint col, GdkEvent *e
return return_val;
}
-/**
+/**
* e_table_group_key_press
* @eti: %ETableGroup to emit the signal on
* @row: The cursor row (model row)
@@ -525,7 +525,7 @@ e_table_group_key_press (ETableGroup *e_table_group, gint row, gint col, GdkEven
return return_val;
}
-/**
+/**
* e_table_group_start_drag
* @eti: %ETableGroup to emit the signal on
* @row: The cursor row (model row)
@@ -549,7 +549,7 @@ e_table_group_start_drag (ETableGroup *e_table_group, gint row, gint col, GdkEve
return return_val;
}
-/**
+/**
* e_table_group_get_header
* @eti: %ETableGroup to check
*
@@ -612,7 +612,7 @@ etg_class_init (GtkObjectClass *object_class)
klass->click = NULL;
klass->key_press = NULL;
klass->start_drag = NULL;
-
+
klass->add = NULL;
klass->add_array = NULL;
klass->add_all = NULL;
diff --git a/widgets/table/e-table-header.c b/widgets/table/e-table-header.c
index 56c7bed164..dd5625df99 100644
--- a/widgets/table/e-table-header.c
+++ b/widgets/table/e-table-header.c
@@ -321,7 +321,8 @@ eth_do_insert (ETableHeader *eth, int pos, ETableCol *val)
*
* This function adds the @tc ETableCol definition into the @eth ETableHeader
* at position @pos. This is the way you add new ETableCols to the
- * ETableHeader.
+ * ETableHeader. The header will assume ownership of the @tc; you should not
+ * unref it after you add it.
*
* This function will emit the "structure_change" signal on the @eth object.
* The ETableCol is assumed
diff --git a/widgets/table/e-table.c b/widgets/table/e-table.c
index 5ac8e45a61..bf2f884c45 100644
--- a/widgets/table/e-table.c
+++ b/widgets/table/e-table.c
@@ -700,6 +700,8 @@ et_canvas_realize (GtkWidget *canvas, ETable *e_table)
static void
et_canvas_button_press (GtkWidget *canvas, GdkEvent *event, ETable *e_table)
{
+ fprintf (stderr, "et_canvas_button_press(): PRESSED\n");
+
if (GTK_WIDGET_HAS_FOCUS(canvas)) {
GnomeCanvasItem *item = GNOME_CANVAS(canvas)->focused_item;