AUDIT: r15853 - gnucash/trunk - Phil Longstaff's patch to fix a kvp_frame bug.

[Derek Atkins]

Author: warlord
Date: 2007-04-09 19:02:33 -0400 (Mon, 09 Apr 2007)
New Revision: 15853
Trac: http://svn.gnucash.org/trac/changeset/15853

Modified:
   gnucash/trunk/
   gnucash/trunk/lib/libqof/qof/kvp_frame.c
Log:
Phil Longstaff's patch to fix a kvp_frame bug.
Save the current frame so we know where to build the new one.
BP



Property changes on: gnucash/trunk
___________________________________________________________________
Name: svk:merge
   - 3889ce50-311e-0410-a464-f059747ec5d1:/local/gnucash/branches/swig-redo:802
3889ce50-311e-0410-a464-f059747ec5d1:/local/gnucash/trunk:1037
57a11ea4-9604-0410-9ed3-97b8803252fd:/gnucash/branches/gobject-engine-dev-warlord:15827
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/branches/gobject-engine-dev-warlord:14369
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/branches/gobject-engine-dev-warlord1:14446
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/trunk:14471
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/trunk2:13366
   + 3889ce50-311e-0410-a464-f059747ec5d1:/local/gnucash/branches/swig-redo:802
3889ce50-311e-0410-a464-f059747ec5d1:/local/gnucash/trunk:1037
57a11ea4-9604-0410-9ed3-97b8803252fd:/gnucash/branches/gobject-engine-dev-warlord:15827
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/branches/gobject-engine-dev-warlord:14369
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/branches/gobject-engine-dev-warlord1:14446
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/trunk:14481
d2ab10a8-8a95-4986-baff-8d511d9f15b2:/local/gnucash/trunk2:13366

Modified: gnucash/trunk/lib/libqof/qof/kvp_frame.c
===================================================================
--- gnucash/trunk/lib/libqof/qof/kvp_frame.c	2007-04-09 20:00:46 UTC (rev 15852)
+++ gnucash/trunk/lib/libqof/qof/kvp_frame.c	2007-04-09 23:02:33 UTC (rev 15853)
@@ -512,6 +512,7 @@
 {
   char *key = NULL;
   KvpValue *oldvalue;
+  KvpFrame* orig_frame = frame;
 
   frame = (KvpFrame *) get_trailer_or_null (frame, path, &key);
   oldvalue = kvp_frame_get_slot (frame, key);
@@ -544,6 +545,7 @@
 
   /* Hmm, if we are here, the path doesn't exist. We need to 
    * create the path, add the value to it. */
+  frame = orig_frame;
   frame = kvp_frame_set_value_nc (frame, path, value);
   LEAVE ("new frame=%s", kvp_frame_to_string(frame));
   return frame;