AUDIT: r17840 - gnucash/trunk/src/register/ledger-core - Register: Add additional debugging output during register cleanup. Also rename a variable for clarity and to match typical usage in the rest of the ledger code.
Charles Day
cedayiv at cvs.gnucash.org
Wed Jan 21 13:58:21 EST 2009
Author: cedayiv
Date: 2009-01-21 13:58:21 -0500 (Wed, 21 Jan 2009)
New Revision: 17840
Trac: http://svn.gnucash.org/trac/changeset/17840
Modified:
gnucash/trunk/src/register/ledger-core/split-register.c
Log:
Register: Add additional debugging output during register cleanup. Also rename a variable for clarity and to match typical usage in the rest of the ledger code.
BP
Modified: gnucash/trunk/src/register/ledger-core/split-register.c
===================================================================
--- gnucash/trunk/src/register/ledger-core/split-register.c 2009-01-21 14:44:45 UTC (rev 17839)
+++ gnucash/trunk/src/register/ledger-core/split-register.c 2009-01-21 18:58:21 UTC (rev 17840)
@@ -2485,7 +2485,7 @@
{
SRInfo *info = gnc_split_register_get_info (reg);
Transaction *pending_trans;
- Transaction *trans = NULL;
+ Transaction *blank_trans = NULL;
Split *blank_split;
ENTER("reg=%p", reg);
@@ -2503,16 +2503,19 @@
{
/* split destroy will automatically remove it
* from its parent account */
- trans = xaccSplitGetParent (blank_split);
+ blank_trans = xaccSplitGetParent (blank_split);
+ DEBUG("blank_split=%p, blank_trans=%p, pending_trans=%p",
+ blank_split, blank_trans, pending_trans);
+
/* Make sure we don't commit this below */
- if (trans == pending_trans)
+ if (blank_trans == pending_trans)
{
info->pending_trans_guid = *guid_null ();
pending_trans = NULL;
}
- xaccTransDestroy (trans);
+ xaccTransDestroy (blank_trans);
info->blank_split_guid = *guid_null ();
blank_split = NULL;
@@ -2521,8 +2524,8 @@
/* be sure to take care of any open transactions */
if (pending_trans != NULL)
{
- g_critical("BUG DETECTED: pending_trans=%p, blank_split=%p, trans=%p",
- pending_trans, blank_split, trans);
+ g_critical("BUG DETECTED: pending_trans=%p, blank_split=%p, blank_trans=%p",
+ pending_trans, blank_split, blank_trans);
g_assert_not_reached();
info->pending_trans_guid = *guid_null ();
/* CAS: It's not clear to me that we'd really want to commit
More information about the gnucash-changes
mailing list