Do not dist gir_DATA
[platform/upstream/atk.git] / atk / atktable.c
index 1210d66..5cecaf6 100755 (executable)
@@ -27,37 +27,31 @@ enum {
   COLUMN_DELETED,
   ROW_REORDERED,
   COLUMN_REORDERED,
+  MODEL_CHANGED,
   LAST_SIGNAL
 };
 
-struct _AtkTableIfaceClass
-{
-  GObjectClass parent;
-};
-
-typedef struct _AtkTableIfaceClass AtkTableIfaceClass;
-
-static void atk_table_base_init (gpointer *g_class);
+static void  atk_table_base_init (gpointer *g_class);
 
 static guint atk_table_signals[LAST_SIGNAL] = { 0 };
 
 GType
-atk_table_get_type ()
+atk_table_get_type (void)
 {
   static GType type = 0;
-
+  
   if (!type) {
     GTypeInfo tinfo =
     {
       sizeof (AtkTableIface),
       (GBaseInitFunc) atk_table_base_init,
       (GBaseFinalizeFunc) NULL,
-
+      
     };
-
+    
     type = g_type_register_static (G_TYPE_INTERFACE, "AtkTable", &tinfo, 0);
   }
-
+  
   return type;
 }
 
@@ -66,65 +60,74 @@ static void
 atk_table_base_init (gpointer *g_class)
 {
   static gboolean initialized = FALSE;
-
-  if (! initialized)
-  {
-    atk_table_signals[ROW_INSERTED] =
-      g_signal_newc ("row_inserted",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     atk_marshal_VOID__INT_INT,
-                     G_TYPE_NONE,
-                     2, G_TYPE_INT, G_TYPE_INT);
-    atk_table_signals[COLUMN_INSERTED] =
-      g_signal_newc ("column_inserted",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     atk_marshal_VOID__INT_INT,
-                     G_TYPE_NONE,
-                     2, G_TYPE_INT, G_TYPE_INT);
-    atk_table_signals[ROW_DELETED] =
-      g_signal_newc ("row_deleted",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     atk_marshal_VOID__INT_INT,
-                     G_TYPE_NONE,
-                     2, G_TYPE_INT, G_TYPE_INT);
-    atk_table_signals[COLUMN_DELETED] =
-      g_signal_newc ("column_deleted",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     atk_marshal_VOID__INT_INT,
-                     G_TYPE_NONE,
-                     2, G_TYPE_INT, G_TYPE_INT);
-    atk_table_signals[ROW_REORDERED] =
-      g_signal_newc ("row_reordered",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     g_cclosure_marshal_VOID__VOID,
-                     G_TYPE_NONE,
-                     0);
-    atk_table_signals[COLUMN_REORDERED] =
-      g_signal_newc ("column_reordered",
-                     ATK_TYPE_TABLE,
-                     G_SIGNAL_RUN_LAST,
-                     G_STRUCT_OFFSET (AtkTableIface, model_changed),
-                     (GSignalAccumulator) NULL, NULL,
-                     g_cclosure_marshal_VOID__VOID,
-                     G_TYPE_NONE,
-                     0);
-    initialized = TRUE;
-  }
+  
+  if (!initialized)
+    {
+      atk_table_signals[ROW_INSERTED] =
+       g_signal_new ("row_inserted",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, row_inserted),
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT,
+                     G_TYPE_NONE,
+                     2, G_TYPE_INT, G_TYPE_INT);
+      atk_table_signals[COLUMN_INSERTED] =
+       g_signal_new ("column_inserted",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, column_inserted),
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT,
+                     G_TYPE_NONE,
+                     2, G_TYPE_INT, G_TYPE_INT);
+      atk_table_signals[ROW_DELETED] =
+       g_signal_new ("row_deleted",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, row_deleted),
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT,
+                     G_TYPE_NONE,
+                     2, G_TYPE_INT, G_TYPE_INT);
+      atk_table_signals[COLUMN_DELETED] =
+       g_signal_new ("column_deleted",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, column_deleted),
+                     (GSignalAccumulator) NULL, NULL,
+                     atk_marshal_VOID__INT_INT,
+                     G_TYPE_NONE,
+                     2, G_TYPE_INT, G_TYPE_INT);
+      atk_table_signals[ROW_REORDERED] =
+       g_signal_new ("row_reordered",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, row_reordered),
+                     (GSignalAccumulator) NULL, NULL,
+                     g_cclosure_marshal_VOID__VOID,
+                     G_TYPE_NONE,
+                     0);
+      atk_table_signals[COLUMN_REORDERED] =
+       g_signal_new ("column_reordered",
+                     ATK_TYPE_TABLE,
+                     G_SIGNAL_RUN_LAST,
+                     G_STRUCT_OFFSET (AtkTableIface, column_reordered),
+                     (GSignalAccumulator) NULL, NULL,
+                     g_cclosure_marshal_VOID__VOID,
+                     G_TYPE_NONE,
+                     0);
+      atk_table_signals[MODEL_CHANGED] =
+        g_signal_new ("model_changed",
+                      ATK_TYPE_TABLE,
+                      G_SIGNAL_RUN_LAST,
+                      G_STRUCT_OFFSET (AtkTableIface, model_changed),
+                      (GSignalAccumulator) NULL, NULL,
+                      g_cclosure_marshal_VOID__VOID,
+                      G_TYPE_NONE, 0);
+
+      initialized = TRUE;
+    }
 }
 
 /**
@@ -133,7 +136,7 @@ atk_table_base_init (gpointer *g_class)
  * @row: a #gint representing a row in @table
  * @column: a #gint representing a column in @table
  *
- * Get a reference to the table cell at @row, @column
+ * Get a reference to the table cell at @row, @column.
  *
  * Returns: a AtkObject* representing the referred to accessible
  **/
@@ -144,8 +147,9 @@ atk_table_ref_at (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
+  g_return_val_if_fail (row >= 0, NULL);
+  g_return_val_if_fail (column >= 0, NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
 
@@ -161,15 +165,11 @@ atk_table_ref_at (AtkTable *table,
  * @row: a #gint representing a row in @table
  * @column: a #gint representing a column in @table
  *
- * Gets a #gint representing the index at the specified @row and @column,
- * or 0 if value does not implement this interface.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
+ * Gets a #gint representing the index at the specified @row and @column.
  *
- * Returns: a #gint representing the index at specified position, or 0
- * if value does not implement this interface.
+ * Returns: a #gint representing the index at specified position.
+ * The value -1 is returned if the object at row,column is not a child
+ * of table or table does not implement this interface.
  **/
 gint
 atk_table_get_index_at (AtkTable *table,
@@ -178,31 +178,27 @@ atk_table_get_index_at (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
-  g_return_val_if_fail (ATK_IS_TABLE (table), 0);
+  g_return_val_if_fail (ATK_IS_TABLE (table), -1);
+  g_return_val_if_fail (row >= 0, -1);
+  g_return_val_if_fail (column >= 0, -1);
 
   iface = ATK_TABLE_GET_IFACE (table);
 
   if (iface->get_index_at)
     return (iface->get_index_at) (table, row, column);
   else
-    return 0;
+    return -1;
 }
 
 /**
  * atk_table_get_row_at_index:
  * @table: a GObject instance that implements AtkTableInterface
- * @index: a #gint representing an index in @table
+ * @index_: a #gint representing an index in @table
  *
- * Gets a #gint representing the row at the specified @index, or 0
- * if the value does not implement this interface
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
+ * Gets a #gint representing the row at the specified @index_.
  *
- * Returns: a gint representing the row at the specified  index, or 0
- * if value does not implement this interface.
+ * Returns: a gint representing the row at the specified index,
+ * or -1 if the table does not implement this interface
  **/
 gint
 atk_table_get_row_at_index (AtkTable *table,
@@ -210,31 +206,25 @@ atk_table_get_row_at_index (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
-  g_return_val_if_fail (ATK_IS_TABLE (table), 0);
+  g_return_val_if_fail (ATK_IS_TABLE (table), -1);
 
   iface = ATK_TABLE_GET_IFACE (table);
 
   if (iface->get_row_at_index)
     return (iface->get_row_at_index) (table, index);
   else
-    return 0;
+    return -1;
 }
 
 /**
  * atk_table_get_column_at_index:
  * @table: a GObject instance that implements AtkTableInterface
- * @index: a #gint representing an index in @table
+ * @index_: a #gint representing an index in @table
  *
- * Gets a #gint representing the column at the specified @index, or 0
- * if the value does not implement this interface
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
+ * Gets a #gint representing the column at the specified @index_. 
  *
- * Returns: a gint representing the column at the specified  index, or 0
- * if value does not implement this interface.
+ * Returns: a gint representing the column at the specified index,
+ * or -1 if the table does not implement this interface
  **/
 gint
 atk_table_get_column_at_index (AtkTable *table,
@@ -242,7 +232,6 @@ atk_table_get_column_at_index (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -250,7 +239,7 @@ atk_table_get_column_at_index (AtkTable *table,
   if (iface->get_column_at_index)
     return (iface->get_column_at_index) (table, index);
   else
-    return 0;
+    return -1;
 }
 
 /**
@@ -258,20 +247,15 @@ atk_table_get_column_at_index (AtkTable *table,
  * @table: a GObject instance that implements AtkTableInterface
  *
  * Gets the caption for the @table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
- * Returns: a gchar* representing the table caption, or %NULL
+ * Returns: a AtkObject* representing the table caption, or %NULL
  * if value does not implement this interface.
  **/
-gchar*
+AtkObject*
 atk_table_get_caption (AtkTable *table)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -287,10 +271,6 @@ atk_table_get_caption (AtkTable *table)
  * @table: a GObject instance that implements AtkTableIface
  *
  * Gets the number of columns in the table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the number of columns, or 0
  * if value does not implement this interface.
@@ -300,7 +280,6 @@ atk_table_get_n_columns (AtkTable *table)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -317,21 +296,16 @@ atk_table_get_n_columns (AtkTable *table)
  * @column: a #gint representing a column in @table
  *
  * Gets the description text of the specified @column in the table
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gchar* representing the column description, or %NULL
  * if value does not implement this interface.
  **/
-gchar*
+G_CONST_RETURN gchar*
 atk_table_get_column_description (AtkTable *table,
                                   gint     column)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -350,10 +324,6 @@ atk_table_get_column_description (AtkTable *table,
  *
  * Gets the number of columns occupied by the accessible object
  * at the specified @row and @column in the @table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the column extent at specified position, or 0
  * if value does not implement this interface.
@@ -365,7 +335,6 @@ atk_table_get_column_extent_at (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -382,10 +351,6 @@ atk_table_get_column_extent_at (AtkTable *table,
  * @column: a #gint representing a column in the table
  *
  * Gets the column header of a specified column in an accessible table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a AtkObject* representing the specified column header, or
  * %NULL if value does not implement this interface.
@@ -395,7 +360,6 @@ atk_table_get_column_header (AtkTable *table, gint column)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -411,10 +375,6 @@ atk_table_get_column_header (AtkTable *table, gint column)
  * @table: a GObject instance that implements AtkTableIface
  *
  * Gets the number of rows in the table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the number of rows, or 0
  * if value does not implement this interface.
@@ -424,7 +384,6 @@ atk_table_get_n_rows (AtkTable *table)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -438,24 +397,19 @@ atk_table_get_n_rows (AtkTable *table)
 /**
  * atk_table_get_row_description:
  * @table: a GObject instance that implements AtkTableIface
- * @r: a #gint representing a row in @table
+ * @row: a #gint representing a row in @table
  *
  * Gets the description text of the specified row in the table
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gchar* representing the row description, or %NULL
  * if value does not implement this interface.
  **/
-gchar*
+G_CONST_RETURN gchar*
 atk_table_get_row_description (AtkTable *table,
                                gint      row)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -474,10 +428,6 @@ atk_table_get_row_description (AtkTable *table,
  *
  * Gets the number of rows occupied by the accessible object
  * at a specified @row and @column in the @table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the row extent at specified position, or 0
  * if value does not implement this interface.
@@ -489,7 +439,6 @@ atk_table_get_row_extent_at (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -506,10 +455,6 @@ atk_table_get_row_extent_at (AtkTable *table,
  * @row: a #gint representing a row in the table
  *
  * Gets the row header of a specified row in an accessible table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a AtkObject* representing the specified row header, or
  * %NULL if value does not implement this interface.
@@ -519,7 +464,6 @@ atk_table_get_row_header (AtkTable *table, gint row)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -535,12 +479,8 @@ atk_table_get_row_header (AtkTable *table, gint row)
  * @table: a GObject instance that implements AtkTableIface
  *
  * Gets the summary description of the table.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
- * Returns: a AtkObject* representing a sumary description of the table,
+ * Returns: a AtkObject* representing a summary description of the table,
  * or zero if value does not implement this interface.
  **/
 AtkObject*
@@ -548,7 +488,6 @@ atk_table_get_summary (AtkTable *table)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, NULL);
   g_return_val_if_fail (ATK_IS_TABLE (table), NULL);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -562,13 +501,10 @@ atk_table_get_summary (AtkTable *table)
 /**
  * atk_table_get_selected_rows:
  * @table: a GObject instance that implements AtkTableIface
+ * @selected: a #gint** that is to contain the selected row numbers
  *
  * Gets the selected rows of the table by initializing **selected with 
  * the selected row numbers. This array should be freed by the caller.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the number of selected rows,
  * or zero if value does not implement this interface.
@@ -578,7 +514,6 @@ atk_table_get_selected_rows (AtkTable *table, gint **selected)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -592,24 +527,19 @@ atk_table_get_selected_rows (AtkTable *table, gint **selected)
 /**
  * atk_table_get_selected_columns:
  * @table: a GObject instance that implements AtkTableIface
+ * @selected: a #gint** that is to contain the selected columns numbers
  *
  * Gets the selected columns of the table by initializing **selected with 
  * the selected column numbers. This array should be freed by the caller.
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gint representing the number of selected columns,
  * or %0 if value does not implement this interface.
- *
  **/
 gint 
 atk_table_get_selected_columns (AtkTable *table, gint **selected)
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, 0);
   g_return_val_if_fail (ATK_IS_TABLE (table), 0);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -627,10 +557,6 @@ atk_table_get_selected_columns (AtkTable *table, gint **selected)
  *
  * Gets a boolean value indicating whether the specified @column
  * is selected
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the column is selected, or 0
  * if value does not implement this interface.
@@ -641,7 +567,6 @@ atk_table_is_column_selected (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -659,10 +584,6 @@ atk_table_is_column_selected (AtkTable *table,
  *
  * Gets a boolean value indicating whether the specified @row
  * is selected
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the row is selected, or 0
  * if value does not implement this interface.
@@ -673,7 +594,6 @@ atk_table_is_row_selected (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -690,12 +610,8 @@ atk_table_is_row_selected (AtkTable *table,
  * @row: a #gint representing a row in @table
  * @column: a #gint representing a column in @table
  *
- * Gets a boolean value indicating whether the acessible object
+ * Gets a boolean value indicating whether the accessible object
  * at the specified @row and @column is selected
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the cell is selected, or 0
  * if value does not implement this interface.
@@ -707,7 +623,6 @@ atk_table_is_selected (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -724,10 +639,6 @@ atk_table_is_selected (AtkTable *table,
  * @row: a #gint representing a row in @table
  *
  * Adds the specified @row to the selection. 
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if row was successfully added to selection,
  * or 0 if value does not implement this interface.
@@ -738,7 +649,6 @@ atk_table_add_row_selection (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -754,10 +664,6 @@ atk_table_add_row_selection (AtkTable *table,
  * @row: a #gint representing a row in @table
  *
  * Removes the specified @row from the selection. 
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the row was successfully removed from
  * the selection, or 0 if value does not implement this interface.
@@ -768,7 +674,6 @@ atk_table_remove_row_selection (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -784,10 +689,6 @@ atk_table_remove_row_selection (AtkTable *table,
  * @column: a #gint representing a column in @table
  *
  * Adds the specified @column to the selection. 
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the column was successfully added to 
  * the selection, or 0 if value does not implement this interface.
@@ -798,7 +699,6 @@ atk_table_add_column_selection (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -814,10 +714,6 @@ atk_table_add_column_selection (AtkTable *table,
  * @column: a #gint representing a column in @table
  *
  * Adds the specified @column to the selection. 
- * Note: callers should not rely on %NULL or on a zero value for
- * indication of whether AtkSelectionIface is implemented, they should
- * use type checking/interface checking macros or the
- * atk_get_accessible_table() convenience method.
  *
  * Returns: a gboolean representing if the column was successfully removed from
  * the selection, or 0 if value does not implement this interface.
@@ -828,7 +724,6 @@ atk_table_remove_column_selection (AtkTable *table,
 {
   AtkTableIface *iface;
 
-  g_return_val_if_fail (table != NULL, FALSE);
   g_return_val_if_fail (ATK_IS_TABLE (table), FALSE);
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -842,17 +737,16 @@ atk_table_remove_column_selection (AtkTable *table,
 /**
  * atk_table_set_caption:
  * @table: a GObject instance that implements AtkTableIface
- * @caption: a #gchar representing the caption to set for @table
+ * @caption: a #AtkObject representing the caption to set for @table
  *
  * Sets the caption for the table.
  **/
 void
 atk_table_set_caption (AtkTable       *table,
-                       gchar          *caption)
+                       AtkObject      *caption)
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -873,11 +767,10 @@ atk_table_set_caption (AtkTable       *table,
 void
 atk_table_set_column_description (AtkTable       *table,
                                   gint           column,
-                                  gchar          *description)
+                                  const gchar    *description)
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -892,7 +785,7 @@ atk_table_set_column_description (AtkTable       *table,
  * @column: a #gint representing a column in @table
  * @header: an #AtkTable
  *
- * Sets the specified column header to @header
+ * Sets the specified column header to @header.
  **/
 void
 atk_table_set_column_header (AtkTable  *table,
@@ -901,7 +794,6 @@ atk_table_set_column_header (AtkTable  *table,
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -922,11 +814,10 @@ atk_table_set_column_header (AtkTable  *table,
 void
 atk_table_set_row_description (AtkTable       *table,
                                gint           row,
-                               gchar          *description)
+                               const gchar    *description)
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -941,7 +832,7 @@ atk_table_set_row_description (AtkTable       *table,
  * @row: a #gint representing a row in @table
  * @header: an #AtkTable 
  *
- * Sets the specified row header to @header
+ * Sets the specified row header to @header.
  **/
 void
 atk_table_set_row_header (AtkTable  *table,
@@ -950,7 +841,6 @@ atk_table_set_row_header (AtkTable  *table,
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);
@@ -965,7 +855,7 @@ atk_table_set_row_header (AtkTable  *table,
  * @accessible: an #AtkObject representing the summary description
  * to set for @table
  *
- * Sets the summary description of the table
+ * Sets the summary description of the table.
  **/
 void
 atk_table_set_summary (AtkTable       *table,
@@ -973,7 +863,6 @@ atk_table_set_summary (AtkTable       *table,
 {
   AtkTableIface *iface;
 
-  g_return_if_fail (table != NULL);
   g_return_if_fail (ATK_IS_TABLE (table));
 
   iface = ATK_TABLE_GET_IFACE (table);