GnuCash Daily Source Diff

Linas Vepstas linas@linas.org
Fri, 19 Oct 2001 14:16:13 -0500


Dave,

I just reviewed another change I saw you make a few days ago,
and this one doesn't make sense to me.  It *almost* makes sense, 
but not quite.

In kvp-sql.c, you explicitly set the object type in pgendKVPDelete().
But this can't be right ... the only allowed obect type for a kvp
record is 'k'.  

I think that you were trying to fix a bug related to changes to kvp
values in multi-user mode.  Is this so?

If so, then I thnk the correct fix would be to fix events.c.   
If I look at events.c, I notice that objtype 'k' seems not to be 
handled at all, e.g. circa line 175, although it should have been.
That's a bug. 

Let me know if this was the bug. If so, either you can fix it, or you
can poke at me to fix it.  I'm kind of busy, but with the right
encouragement, I can fix this .... Let me know, I don't want to dive in
and wreck what your doing.


--linas

-- 
pub  1024D/01045933 2001-02-01 Linas Vepstas (Labas!) <linas@linas.org>
PGP Key fingerprint = 8305 2521 6000 0B5E 8984  3F54 64A9 9A82 0104 5933