On Wednesday 07 May 2014 16:22:27 Mike Alexander wrote: > --On May 7, 2014 6:41:03 PM +0200 Geert Janssens > > <janssens-ge...@telenet.be> wrote: > > On Tuesday 06 May 2014 18:16:51 Mike Alexander wrote: > >> Just to pick another random example, the merge also removed a call > >> to > >> qof_instance_set_dirty in gnc_template_register_save_xfrm_cell > >> which > >> is in register/ledger-core/split-register-model-save.c. This call > >> was added in 613ba0d on December 7. This is only one of a number > >> of > >> changes I noticed. > > > > I think that is actually correct. From how I understand John's work > > qof_instance_set takes care of properly dirtying the kvp. If I'm > > mistaken here then there are many places in the new code that no > > longer mark kvp's as dirty. > > Perhaps it is a desired change, but then it should have been on the > private-kvp branch. Instead it was introduced as a side effect of the > merge back to master. Even if the call to qof_instance_set_dirty is > not needed after the private-kvp changes, it won't hurt anything. If > it is to be removed it should be removed explicitly, not as a side > effect of the merge. > It was actually hurting: it caused the build to fail. With the function still in there I got an undefined function error. Perhaps this could have been resolved by adding the proper header include but I didn't check that part.
Only John can tell why he did remove it during the merge and not beforehand. _______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel