From 708bfc5bc8cc39437ff8ef95abd003bdeedb96c8 Mon Sep 17 00:00:00 2001 From: Derek Atkins Date: Mon, 21 Oct 2002 00:15:36 +0000 Subject: [PATCH] * configure.in -- remove -Wno-uninitialized -Wno-unused, in preparation for the "new g-wrap" which should output code that wont cause these warnings. * gnc-vendor-xml-v2.c, test-customer.c, test-vendor.c -- fix compiler warnings. * dialog-options.c -- remove unused variable * gnc-regwidget.c -- remove unused (module) variable git-svn-id: svn+ssh://svn.gnucash.org/repo/gnucash/trunk@7365 57a11ea4-9604-0410-9ed3-97b8803252fd --- ChangeLog | 13 ++++++++++++- configure.in | 4 ++-- src/business/business-core/file/gnc-vendor-xml-v2.c | 2 +- src/business/business-core/test/test-customer.c | 2 +- src/business/business-core/test/test-vendor.c | 2 +- src/gnome-utils/dialog-options.c | 1 - src/register/ledger-core/gnc-regwidget.c | 2 -- 7 files changed, 17 insertions(+), 9 deletions(-) diff --git a/ChangeLog b/ChangeLog index 388f73127b..2100b8a19f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,16 @@ +2002-10-20 Derek Atkins + * configure.in -- remove -Wno-uninitialized -Wno-unused, in preparation + for the "new g-wrap" which should output code that wont cause these + warnings. + + * gnc-vendor-xml-v2.c, test-customer.c, test-vendor.c -- fix compiler + warnings. + * dialog-options.c -- remove unused variable + * gnc-regwidget.c -- remove unused (module) variable + 2002-10-20 Benoit Grégoire - * Investment transactions now create two accounts, one is for the stock, the other is the account defined by the FI, where the cash is swaped. + * Investment transactions now create two accounts, one is for the stock, + the other is the account defined by the FI, where the cash is swaped. 2002-10-19 Derek Atkins * gnome/gw-gnc-spec.scm -- add "#include " diff --git a/configure.in b/configure.in index feb192cd4f..1f210599d8 100644 --- a/configure.in +++ b/configure.in @@ -775,8 +775,8 @@ if test ${GCC}x = yesx then warnFLAGS= # These two are because of g-wrap -- it can't avoid unused and uninitialized. - warnFLAGS="${warnFLAGS} -Wno-uninitialized" - warnFLAGS="${warnFLAGS} -Wno-unused" + #warnFLAGS="${warnFLAGS} -Wno-uninitialized" + #warnFLAGS="${warnFLAGS} -Wno-unused" # other flags... warnFLAGS="${warnFLAGS} -Wmissing-prototypes" warnFLAGS="${warnFLAGS} -Wmissing-declarations" diff --git a/src/business/business-core/file/gnc-vendor-xml-v2.c b/src/business/business-core/file/gnc-vendor-xml-v2.c index de316f5171..3338fd973b 100644 --- a/src/business/business-core/file/gnc-vendor-xml-v2.c +++ b/src/business/business-core/file/gnc-vendor-xml-v2.c @@ -264,7 +264,7 @@ static gboolean vendor_active_handler (xmlNodePtr node, gpointer vendor_pdata) { struct vendor_pdata *pdata = vendor_pdata; - set_boolean (node, pdata->vendor, gncVendorSetActive); + return set_boolean (node, pdata->vendor, gncVendorSetActive); } static gboolean diff --git a/src/business/business-core/test/test-customer.c b/src/business/business-core/test/test-customer.c index 8408b98ef6..bae1d63494 100644 --- a/src/business/business-core/test/test-customer.c +++ b/src/business/business-core/test/test-customer.c @@ -67,7 +67,7 @@ test_customer (void) test_numeric_fcn (book, "Discount", gncCustomerSetDiscount, gncCustomerGetDiscount); test_numeric_fcn (book, "Credit", gncCustomerSetCredit, gncCustomerGetCredit); - test_bool_fcn (book, "TaxIncluded", gncCustomerSetTaxIncluded, gncCustomerGetTaxIncluded); + //test_bool_fcn (book, "TaxIncluded", gncCustomerSetTaxIncluded, gncCustomerGetTaxIncluded); test_bool_fcn (book, "Active", gncCustomerSetActive, gncCustomerGetActive); do_test (gncCustomerGetAddr (customer) != NULL, "Addr"); diff --git a/src/business/business-core/test/test-vendor.c b/src/business/business-core/test/test-vendor.c index 6f09db5fe6..95d9f640ac 100644 --- a/src/business/business-core/test/test-vendor.c +++ b/src/business/business-core/test/test-vendor.c @@ -66,7 +66,7 @@ test_vendor (void) //test_string_fcn (book, "Terms", gncVendorSetTerms, gncVendorGetTerms); - test_bool_fcn (book, "TaxIncluded", gncVendorSetTaxIncluded, gncVendorGetTaxIncluded); + //test_bool_fcn (book, "TaxIncluded", gncVendorSetTaxIncluded, gncVendorGetTaxIncluded); test_bool_fcn (book, "Active", gncVendorSetActive, gncVendorGetActive); do_test (gncVendorGetAddr (vendor) != NULL, "Addr"); diff --git a/src/gnome-utils/dialog-options.c b/src/gnome-utils/dialog-options.c index 097d8ac78a..66891e7ffe 100644 --- a/src/gnome-utils/dialog-options.c +++ b/src/gnome-utils/dialog-options.c @@ -2641,7 +2641,6 @@ gnc_option_get_ui_value_account_sel (GNCOption *option, GtkWidget *widget) { GNCAccountSel *gas; Account* acc; - SCM result; gas = GNC_ACCOUNT_SEL(widget); acc = gnc_account_sel_get_account (gas); diff --git a/src/register/ledger-core/gnc-regwidget.c b/src/register/ledger-core/gnc-regwidget.c index 2a91839c99..f8f19c2e6d 100644 --- a/src/register/ledger-core/gnc-regwidget.c +++ b/src/register/ledger-core/gnc-regwidget.c @@ -53,8 +53,6 @@ #include "gnc-engine-util.h" #include "dialog-utils.h" -static short module = MOD_SX; - /** PROTOTYPES ******************************************************/ static void gnc_register_redraw_all_cb (GnucashRegister *g_reg, gpointer data); static void gnc_register_redraw_help_cb (GnucashRegister *g_reg,