r17861 - gnucash/branches/2.2/src/register/ledger-core - [17840] 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.

[Christian Stimming]

Author: cstim
Date: 2009-01-31 05:45:56 -0500 (Sat, 31 Jan 2009)
New Revision: 17861
Trac: http://svn.gnucash.org/trac/changeset/17861

Modified:
   gnucash/branches/2.2/src/register/ledger-core/split-register.c
Log:
[17840] 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.

Original commit by cedayiv.

Modified: gnucash/branches/2.2/src/register/ledger-core/split-register.c
===================================================================
--- gnucash/branches/2.2/src/register/ledger-core/split-register.c	2009-01-31 10:45:49 UTC (rev 17860)
+++ gnucash/branches/2.2/src/register/ledger-core/split-register.c	2009-01-31 10:45:56 UTC (rev 17861)
@@ -2372,7 +2372,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);
@@ -2390,16 +2390,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;
@@ -2408,8 +2411,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