AUDIT: r17496 - gnucash/trunk/src/gnome-utils - Bug #539962: Remove return value from some void functions.

Andreas Köhler andi5 at cvs.gnucash.org
Sat Sep 6 13:22:36 EDT 2008


Author: andi5
Date: 2008-09-06 13:22:36 -0400 (Sat, 06 Sep 2008)
New Revision: 17496
Trac: http://svn.gnucash.org/trac/changeset/17496

Modified:
   gnucash/trunk/src/gnome-utils/dialog-transfer.c
   gnucash/trunk/src/gnome-utils/gnc-dense-cal-model.c
   gnucash/trunk/src/gnome-utils/gnc-dense-cal.c
   gnucash/trunk/src/gnome-utils/gnc-plugin-page.c
Log:
Bug #539962: Remove return value from some void functions.

Patch from Halton Huo.
BP

Modified: gnucash/trunk/src/gnome-utils/dialog-transfer.c
===================================================================
--- gnucash/trunk/src/gnome-utils/dialog-transfer.c	2008-09-06 17:21:00 UTC (rev 17495)
+++ gnucash/trunk/src/gnome-utils/dialog-transfer.c	2008-09-06 17:22:36 UTC (rev 17496)
@@ -257,7 +257,10 @@
 
   if (!gnc_is_euro_currency (from_currency) ||
       !gnc_is_euro_currency (to_currency))
-    return gnc_xfer_dialog_update_price (xferData);
+  {
+    gnc_xfer_dialog_update_price (xferData);
+    return;
+  }
 
   from_rate = gnc_euro_currency_get_rate (from_currency);
   to_rate = gnc_euro_currency_get_rate (to_currency);

Modified: gnucash/trunk/src/gnome-utils/gnc-dense-cal-model.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-dense-cal-model.c	2008-09-06 17:21:00 UTC (rev 17495)
+++ gnucash/trunk/src/gnome-utils/gnc-dense-cal-model.c	2008-09-06 17:22:36 UTC (rev 17496)
@@ -138,5 +138,5 @@
 void
 gnc_dense_cal_model_get_instance(GncDenseCalModel *model, guint tag, gint instance_index, GDate *date)
 {
-    return (*GNC_DENSE_CAL_MODEL_GET_INTERFACE(model)->get_instance)(model, tag, instance_index, date);
+    (*GNC_DENSE_CAL_MODEL_GET_INTERFACE(model)->get_instance)(model, tag, instance_index, date);
 }

Modified: gnucash/trunk/src/gnome-utils/gnc-dense-cal.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-dense-cal.c	2008-09-06 17:21:00 UTC (rev 17495)
+++ gnucash/trunk/src/gnome-utils/gnc-dense-cal.c	2008-09-06 17:22:36 UTC (rev 17496)
@@ -539,7 +539,7 @@
 void
 gnc_dense_cal_set_year(GncDenseCal *dcal, guint year)
 {
-    return _gnc_dense_cal_set_year(dcal, year, TRUE);
+    _gnc_dense_cal_set_year(dcal, year, TRUE);
 }
 
 static void

Modified: gnucash/trunk/src/gnome-utils/gnc-plugin-page.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-plugin-page.c	2008-09-06 17:21:00 UTC (rev 17495)
+++ gnucash/trunk/src/gnome-utils/gnc-plugin-page.c	2008-09-06 17:22:36 UTC (rev 17496)
@@ -177,7 +177,7 @@
 	g_return_if_fail (klass != NULL);
 	g_return_if_fail (klass->destroy_widget != NULL);
 
-	return klass->destroy_widget (plugin_page);
+	klass->destroy_widget (plugin_page);
 }
 
 



More information about the gnucash-changes mailing list