r17545 - gnucash/branches/2.2/src/gnome-utils - [r17496] Bug #539962: Remove return value from some void functions.

Andreas Köhler andi5 at cvs.gnucash.org
Wed Sep 17 13:35:31 EDT 2008


Author: andi5
Date: 2008-09-17 13:35:31 -0400 (Wed, 17 Sep 2008)
New Revision: 17545
Trac: http://svn.gnucash.org/trac/changeset/17545

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

Patch from Halton Huo.

Modified: gnucash/branches/2.2/src/gnome-utils/dialog-transfer.c
===================================================================
--- gnucash/branches/2.2/src/gnome-utils/dialog-transfer.c	2008-09-17 17:35:23 UTC (rev 17544)
+++ gnucash/branches/2.2/src/gnome-utils/dialog-transfer.c	2008-09-17 17:35:31 UTC (rev 17545)
@@ -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/branches/2.2/src/gnome-utils/gnc-dense-cal-model.c
===================================================================
--- gnucash/branches/2.2/src/gnome-utils/gnc-dense-cal-model.c	2008-09-17 17:35:23 UTC (rev 17544)
+++ gnucash/branches/2.2/src/gnome-utils/gnc-dense-cal-model.c	2008-09-17 17:35:31 UTC (rev 17545)
@@ -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/branches/2.2/src/gnome-utils/gnc-dense-cal.c
===================================================================
--- gnucash/branches/2.2/src/gnome-utils/gnc-dense-cal.c	2008-09-17 17:35:23 UTC (rev 17544)
+++ gnucash/branches/2.2/src/gnome-utils/gnc-dense-cal.c	2008-09-17 17:35:31 UTC (rev 17545)
@@ -536,7 +536,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/branches/2.2/src/gnome-utils/gnc-plugin-page.c
===================================================================
--- gnucash/branches/2.2/src/gnome-utils/gnc-plugin-page.c	2008-09-17 17:35:23 UTC (rev 17544)
+++ gnucash/branches/2.2/src/gnome-utils/gnc-plugin-page.c	2008-09-17 17:35:31 UTC (rev 17545)
@@ -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