reviewed by: Harish Krishnaswamy <kharish@novell.com>

2006-01-27  Kjartan Maraas  <kmaraas@gnome.org>

	reviewed by: Harish Krishnaswamy <kharish@novell.com>

	* addressbook/ea-addressbook.c: Remove extra semi-colons.
	* calendar/ea-cal-view.c: (ea_cal_view_event_changed_cb),
	(ea_cal_view_event_added_cb), (action_interface_get_keybinding):
	Remove unused code and variables.
	* calendar/ea-day-view-main-item.c:
	(table_interface_get_row_description): More removal of unused bits
	* calendar/ea-week-view-main-item.c:
	(ea_week_view_main_item_get_row_at_index),
	(ea_week_view_main_item_get_column_at_index),
	(ea_week_view_main_item_get_row_label),
	(table_interface_get_n_columns),
	(table_interface_get_column_description): Same again.
	* e-table/gal-a11y-e-table-item.c: (gal_a11y_e_table_item_new):
	And again.
	* e-table/gal-a11y-e-table.c: (et_ref_child): Remove unused var.
	* e-table/gal-a11y-e-tree.c: (gal_a11y_e_tree_new): Again.
	* e-text/gal-a11y-e-text.c: (et_set_selection),
	(et_real_initialize): Same here too.
	* widgets/ea-calendar-item.c:
	(table_interface_get_column_description),
	(table_interface_get_row_description),
	(selection_interface_is_child_selected),
	(ea_calendar_item_get_cell_data): And here too.
	* widgets/ea-widgets.c: More redundant semi-colons.

svn path=/trunk/; revision=31324

11 files changed, 34 insertions, 66 deletions

diff --git a/a11y/ChangeLog b/a11y/ChangeLog
index 6aa9bfaf24..5ddaffd47d 100644
--- a/a11y/ChangeLog
+++ b/a11y/ChangeLog
@@ -1,3 +1,32 @@
+2006-01-27  Kjartan Maraas  <kmaraas@gnome.org>
+
+	reviewed by: Harish Krishnaswamy <kharish@novell.com> 
+
+	* addressbook/ea-addressbook.c: Remove extra semi-colons.
+	* calendar/ea-cal-view.c: (ea_cal_view_event_changed_cb),
+	(ea_cal_view_event_added_cb), (action_interface_get_keybinding):
+	Remove unused code and variables.
+	* calendar/ea-day-view-main-item.c:
+	(table_interface_get_row_description): More removal of unused bits
+	* calendar/ea-week-view-main-item.c:
+	(ea_week_view_main_item_get_row_at_index),
+	(ea_week_view_main_item_get_column_at_index),
+	(ea_week_view_main_item_get_row_label),
+	(table_interface_get_n_columns),
+	(table_interface_get_column_description): Same again.
+	* e-table/gal-a11y-e-table-item.c: (gal_a11y_e_table_item_new):
+	And again.
+	* e-table/gal-a11y-e-table.c: (et_ref_child): Remove unused var.
+	* e-table/gal-a11y-e-tree.c: (gal_a11y_e_tree_new): Again.
+	* e-text/gal-a11y-e-text.c: (et_set_selection),
+	(et_real_initialize): Same here too.
+	* widgets/ea-calendar-item.c:
+	(table_interface_get_column_description),
+	(table_interface_get_row_description),
+	(selection_interface_is_child_selected),
+	(ea_calendar_item_get_cell_data): And here too.
+	* widgets/ea-widgets.c: More redundant semi-colons.
+
 2006-01-10  Simon Zheng  <simon.zheng@sun.com>
 
 	* widgets/ea-calendar-item.c: use e_utf8_strftime() in
diff --git a/a11y/addressbook/ea-addressbook.c b/a11y/addressbook/ea-addressbook.c
index 51ae26ba97..f4a5ebfae1 100644
--- a/a11y/addressbook/ea-addressbook.c
+++ b/a11y/addressbook/ea-addressbook.c
@@ -30,9 +30,9 @@
 #include "ea-minicard-view.h"
 #include "ea-addressbook-view.h"
 
-EA_FACTORY_GOBJECT (EA_TYPE_MINICARD, ea_minicard, ea_minicard_new);
-EA_FACTORY_GOBJECT (EA_TYPE_MINICARD_VIEW, ea_minicard_view, ea_minicard_view_new);
-EA_FACTORY_GOBJECT (EA_TYPE_AB_VIEW, ea_ab_view, ea_ab_view_new);
+EA_FACTORY_GOBJECT (EA_TYPE_MINICARD, ea_minicard, ea_minicard_new)
+EA_FACTORY_GOBJECT (EA_TYPE_MINICARD_VIEW, ea_minicard_view, ea_minicard_view_new)
+EA_FACTORY_GOBJECT (EA_TYPE_AB_VIEW, ea_ab_view, ea_ab_view_new)
 
 static gboolean ea_addressbook_focus_watcher (GSignalInvocationHint *ihint,
 					      guint n_param_values,
diff --git a/a11y/calendar/ea-cal-view.c b/a11y/calendar/ea-cal-view.c
index 31400ca2da..6a25455410 100644
--- a/a11y/calendar/ea-cal-view.c
+++ b/a11y/calendar/ea-cal-view.c
@@ -191,7 +191,6 @@ ea_cal_view_event_changed_cb (ECalendarView *cal_view, ECalendarViewEvent *event
                               gpointer data)
 {
 	AtkObject *atk_obj;
-	EaCalView *ea_cal_view;
 	AtkObject *event_atk_obj = NULL;
 
 	g_return_if_fail (E_IS_CALENDAR_VIEW (cal_view));
@@ -199,7 +198,6 @@ ea_cal_view_event_changed_cb (ECalendarView *cal_view, ECalendarViewEvent *event
 	atk_obj = gtk_widget_get_accessible (GTK_WIDGET(cal_view));
 	if (!EA_IS_CAL_VIEW (atk_obj))
 		return;
-	ea_cal_view = EA_CAL_VIEW (atk_obj);
 
 	if ((E_IS_DAY_VIEW (cal_view)) && event && event->canvas_item) {
 		event_atk_obj =
@@ -230,7 +228,6 @@ ea_cal_view_event_added_cb (ECalendarView *cal_view, ECalendarViewEvent *event,
                             gpointer data)
 {
 	AtkObject *atk_obj;
-	EaCalView *ea_cal_view;
 	AtkObject *event_atk_obj = NULL;
 	gint index;
 
@@ -239,7 +236,6 @@ ea_cal_view_event_added_cb (ECalendarView *cal_view, ECalendarViewEvent *event,
 	atk_obj = gtk_widget_get_accessible (GTK_WIDGET(cal_view));
 	if (!EA_IS_CAL_VIEW (atk_obj))
 		return;
-	ea_cal_view = EA_CAL_VIEW (atk_obj);
 
 	if ((E_IS_DAY_VIEW (cal_view)) && event && event->canvas_item) {
 		event_atk_obj =
@@ -391,7 +387,6 @@ static G_CONST_RETURN gchar*
 action_interface_get_keybinding (AtkAction *action, gint index)
 {
 	GtkWidget *widget;
-	EaCalView *ea_cal_view;
 
 	widget = GTK_ACCESSIBLE (action)->widget;
 	if (widget == NULL)
@@ -403,29 +398,22 @@ action_interface_get_keybinding (AtkAction *action, gint index)
 	if (!GTK_WIDGET_IS_SENSITIVE (widget) || !GTK_WIDGET_VISIBLE (widget))
 		return FALSE;
 
-	 ea_cal_view = EA_CAL_VIEW (action);
-
 	 switch (index) {
 	 case 0:
 		 /* New Appointment */
 		 return "<Alt>fna;<Control>n";
-		 break;
 	 case 1:
 		 /* New Event */
 		 return "<Alt>fnd;<Shift><Control>d";
-		 break;
 	 case 2:
 		 /* New Meeting */
 		 return "<Alt>fne;<Shift><Control>e";
-		 break;
 	 case 3:
 		 /* Go to today */
 		 return "<Alt>vt;<Alt><Control>t";
-		 break;
 	 case 4:
 		 /* Go to date */
 		 return "<Alt>vd;<Alt><Control>g";
-		 break;
 	 default:
 		 break;
 	 }
diff --git a/a11y/calendar/ea-day-view-main-item.c b/a11y/calendar/ea-day-view-main-item.c
index ea8b9ce2ee..4912e6ee7a 100644
--- a/a11y/calendar/ea-day-view-main-item.c
+++ b/a11y/calendar/ea-day-view-main-item.c
@@ -1085,8 +1085,6 @@ table_interface_get_row_description (AtkTable    *table,
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EDayViewMainItem *main_item;
-	EDayView *day_view;
 	EaDayViewMainItem* ea_main_item = EA_DAY_VIEW_MAIN_ITEM (table);
 	const gchar *description;
 	EaCellTable *cell_data;
@@ -1096,9 +1094,6 @@ table_interface_get_row_description (AtkTable    *table,
 	if (!g_obj)
 		return NULL;
 
-	main_item = E_DAY_VIEW_MAIN_ITEM (g_obj);
-	day_view = main_item->day_view;
-
 	if (row < 0 || row >= 12 * 24)
 		return NULL;
 	cell_data = ea_day_view_main_item_get_cell_data (ea_main_item);
diff --git a/a11y/calendar/ea-week-view-main-item.c b/a11y/calendar/ea-week-view-main-item.c
index 81a66cfedc..490c235446 100644
--- a/a11y/calendar/ea-week-view-main-item.c
+++ b/a11y/calendar/ea-week-view-main-item.c
@@ -491,8 +491,6 @@ ea_week_view_main_item_get_row_at_index (EaWeekViewMainItem *ea_main_item,
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EWeekViewMainItem *main_item;
-	EWeekView *week_view;
 	gint n_children;
 
 	g_return_val_if_fail (ea_main_item, -1);
@@ -502,9 +500,6 @@ ea_week_view_main_item_get_row_at_index (EaWeekViewMainItem *ea_main_item,
 	if (!g_obj)
 		return -1;
 
-	main_item = E_WEEK_VIEW_MAIN_ITEM (g_obj);
-	week_view = main_item->week_view;
-
 	n_children = ea_week_view_main_item_get_n_children (ATK_OBJECT (ea_main_item));
 	if (index >= 0 && index < n_children)
 		return index / 7;
@@ -517,8 +512,6 @@ ea_week_view_main_item_get_column_at_index (EaWeekViewMainItem *ea_main_item,
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EWeekViewMainItem *main_item;
-	EWeekView *week_view;
 	gint n_children;
 
 	g_return_val_if_fail (ea_main_item, -1);
@@ -528,9 +521,6 @@ ea_week_view_main_item_get_column_at_index (EaWeekViewMainItem *ea_main_item,
 	if (!g_obj)
 		return -1;
 
-	main_item = E_WEEK_VIEW_MAIN_ITEM (g_obj);
-	week_view = main_item->week_view;
-
 	n_children = ea_week_view_main_item_get_n_children (ATK_OBJECT (ea_main_item));
 	if (index >= 0 && index < n_children)
 		return index % 7;
@@ -543,8 +533,6 @@ ea_week_view_main_item_get_row_label (EaWeekViewMainItem *ea_main_item,
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EWeekViewMainItem *main_item;
-	EWeekView *week_view;
 
 	g_return_val_if_fail (ea_main_item, 0);
 
@@ -553,9 +541,6 @@ ea_week_view_main_item_get_row_label (EaWeekViewMainItem *ea_main_item,
 	if (!g_obj)
 		return 0 ;
 
-	main_item = E_WEEK_VIEW_MAIN_ITEM (g_obj);
-	week_view = main_item->week_view;
-
 	return g_snprintf (buffer, buffer_size, "the %i week",
 			   row + 1);
 
@@ -715,8 +700,6 @@ table_interface_get_n_columns (AtkTable *table)
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EWeekViewMainItem *main_item;
-	EWeekView *week_view;
 	EaWeekViewMainItem* ea_main_item = EA_WEEK_VIEW_MAIN_ITEM (table);
 
 	atk_gobj = ATK_GOBJECT_ACCESSIBLE (ea_main_item);
@@ -724,9 +707,6 @@ table_interface_get_n_columns (AtkTable *table)
 	if (!g_obj)
 		return -1;
 
-	main_item = E_WEEK_VIEW_MAIN_ITEM (g_obj);
-	week_view = main_item->week_view;
-
 	return 7;
 }
 
@@ -1037,8 +1017,6 @@ table_interface_get_column_description (AtkTable	  *table,
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	EWeekViewMainItem *main_item;
-	EWeekView *week_view;
 	EaWeekViewMainItem* ea_main_item = EA_WEEK_VIEW_MAIN_ITEM (table);
 	const gchar *description;
 	EaCellTable *cell_data;
@@ -1048,9 +1026,6 @@ table_interface_get_column_description (AtkTable	  *table,
 	if (!g_obj)
 		return NULL;
 
-	main_item = E_WEEK_VIEW_MAIN_ITEM (g_obj);
-	week_view = main_item->week_view;
-
 	if (in_col < 0 || in_col > 6)
 		return NULL;
 	cell_data = ea_week_view_main_item_get_cell_data (ea_main_item);
diff --git a/a11y/e-table/gal-a11y-e-table-item.c b/a11y/e-table/gal-a11y-e-table-item.c
index 2f2d72d4eb..35eae0cf54 100644
--- a/a11y/e-table/gal-a11y-e-table-item.c
+++ b/a11y/e-table/gal-a11y-e-table-item.c
@@ -1002,7 +1002,6 @@ gal_a11y_e_table_item_new (ETableItem *item)
 {
 	GalA11yETableItem *a11y;
 	AtkObject *accessible;
-	int n;
 	ESelectionModel * esm;
 	AtkObject *parent;
 	const char *name;
@@ -1024,7 +1023,6 @@ gal_a11y_e_table_item_new (ETableItem *item)
 	accessible  = ATK_OBJECT(a11y);
 
 	/* Initialize cell data. */
-	n = item->cols * item->rows;
 	GET_PRIVATE (a11y)->cols = item->cols;
 	GET_PRIVATE (a11y)->rows = item->rows;
 
diff --git a/a11y/e-table/gal-a11y-e-table.c b/a11y/e-table/gal-a11y-e-table.c
index de4a58a50b..56cfdbd2f0 100644
--- a/a11y/e-table/gal-a11y-e-table.c
+++ b/a11y/e-table/gal-a11y-e-table.c
@@ -162,7 +162,6 @@ et_ref_child (AtkObject *accessible,
 			}
 		}
 	} else if (i == child_no -1) {
-        	AtkObject * accessible;
 		ETableClickToAdd * etcta;
 
 		if (et && et->use_click_to_add && et->click_to_add) {
diff --git a/a11y/e-table/gal-a11y-e-tree.c b/a11y/e-table/gal-a11y-e-tree.c
index f175f78564..6ad507a4f3 100644
--- a/a11y/e-table/gal-a11y-e-tree.c
+++ b/a11y/e-table/gal-a11y-e-tree.c
@@ -155,9 +155,6 @@ AtkObject *
 gal_a11y_e_tree_new (GObject *widget)
 {
 	GalA11yETree *a11y;
-	ETree *tree;
-
-	tree = E_TREE (widget);
 
 	a11y = g_object_new (gal_a11y_e_tree_get_type (), NULL);
 
diff --git a/a11y/e-text/gal-a11y-e-text.c b/a11y/e-text/gal-a11y-e-text.c
index d23deef07d..1fd0a63476 100644
--- a/a11y/e-text/gal-a11y-e-text.c
+++ b/a11y/e-text/gal-a11y-e-text.c
@@ -766,14 +766,12 @@ et_set_selection (AtkText *text,
 		  gint end_offset)
 {
 	GObject *obj;
-	EText *etext;
 
 	g_return_val_if_fail (ATK_IS_GOBJECT_ACCESSIBLE (text), FALSE);
 	obj = atk_gobject_accessible_get_object (ATK_GOBJECT_ACCESSIBLE (text));
 	if (obj == NULL)
 		return FALSE;
 	g_return_val_if_fail (E_IS_TEXT (obj), FALSE);
-	etext = E_TEXT (obj);
 	if (selection_num == 0)
                 return et_add_selection (text, start_offset, end_offset);
         return FALSE;
@@ -1015,7 +1013,6 @@ static void
 et_real_initialize (AtkObject *obj,
                     gpointer  data)
 {
-	GalA11yEText *a11y;
 	EText *etext;
 
 	ATK_OBJECT_CLASS (parent_class)->initialize (obj, data);
@@ -1023,7 +1020,6 @@ et_real_initialize (AtkObject *obj,
 	g_return_if_fail (GAL_A11Y_IS_E_TEXT (obj));
 	g_return_if_fail (E_IS_TEXT (data));
 
-	a11y = GAL_A11Y_E_TEXT (obj);
 	etext = E_TEXT (data);
 
 	/* Set up signal callbacks */
diff --git a/a11y/widgets/ea-calendar-item.c b/a11y/widgets/ea-calendar-item.c
index b8e283e604..0aeac944df 100644
--- a/a11y/widgets/ea-calendar-item.c
+++ b/a11y/widgets/ea-calendar-item.c
@@ -786,7 +786,6 @@ table_interface_get_column_description (AtkTable *table, gint in_col)
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	ECalendarItem *calitem;
 	EaCalendarItem* ea_calitem = EA_CALENDAR_ITEM (table);
 	const gchar *description = NULL;
 	EaCellTable *cell_data;
@@ -797,7 +796,6 @@ table_interface_get_column_description (AtkTable *table, gint in_col)
 	if (!g_obj)
 		return NULL;
 
-	calitem = E_CALENDAR_ITEM (g_obj);
 	n_columns = table_interface_get_n_columns (table);
 	if (in_col < 0 || in_col >= n_columns)
 		return NULL;
@@ -822,7 +820,6 @@ table_interface_get_row_description (AtkTable *table, gint row)
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	ECalendarItem *calitem;
 	EaCalendarItem* ea_calitem = EA_CALENDAR_ITEM (table);
 	const gchar *description = NULL;
 	EaCellTable *cell_data;
@@ -833,7 +830,6 @@ table_interface_get_row_description (AtkTable *table, gint row)
 	if (!g_obj)
 		return NULL;
 
-	calitem = E_CALENDAR_ITEM (g_obj);
 	n_rows = table_interface_get_n_rows (table);
 	if (row < 0 || row >= n_rows)
 		return NULL;
@@ -972,7 +968,6 @@ selection_interface_is_child_selected (AtkSelection *selection, gint index)
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	ECalendarItem *calitem;
 	EaCalendarItem* ea_calitem = EA_CALENDAR_ITEM (selection);
 	gint row, column, n_children;
 
@@ -981,7 +976,6 @@ selection_interface_is_child_selected (AtkSelection *selection, gint index)
 	if (!g_obj)
 		return FALSE;
 
-	calitem = E_CALENDAR_ITEM (g_obj);
 	n_children = atk_object_get_n_accessible_children (ATK_OBJECT (selection));
 	if (index < 0 || index >= n_children)
 		return FALSE;
@@ -1042,7 +1036,6 @@ ea_calendar_item_get_cell_data (EaCalendarItem *ea_calitem)
 {
 	AtkGObjectAccessible *atk_gobj;
 	GObject *g_obj;
-	ECalendarItem *calitem;
 	EaCellTable *cell_data;
 
 	g_return_val_if_fail (ea_calitem, NULL);
@@ -1052,8 +1045,6 @@ ea_calendar_item_get_cell_data (EaCalendarItem *ea_calitem)
 	if (!g_obj)
 		return NULL;
 
-	calitem = E_CALENDAR_ITEM (g_obj);
-
 	cell_data = g_object_get_data (G_OBJECT(ea_calitem),
 				       "ea-calendar-cell-table");
 
diff --git a/a11y/widgets/ea-widgets.c b/a11y/widgets/ea-widgets.c
index feef6d1ef3..372783b92d 100644
--- a/a11y/widgets/ea-widgets.c
+++ b/a11y/widgets/ea-widgets.c
@@ -28,8 +28,8 @@
 #include "widgets/ea-combo-button.h"
 #include "ea-widgets.h"
 
-EA_FACTORY_GOBJECT (EA_TYPE_CALENDAR_ITEM, ea_calendar_item, ea_calendar_item_new);
-EA_FACTORY (EA_TYPE_COMBO_BUTTON, ea_combo_button, ea_combo_button_new);
+EA_FACTORY_GOBJECT (EA_TYPE_CALENDAR_ITEM, ea_calendar_item, ea_calendar_item_new)
+EA_FACTORY (EA_TYPE_COMBO_BUTTON, ea_combo_button, ea_combo_button_new)
 
 void e_calendar_item_a11y_init (void)
 {