From: Gerrit V. <vo...@vo...> - 2009-06-27 02:23:20
|
Hi, On Fri, 2009-06-26 at 21:05 -0500, Carsten Neumann wrote: > Hello Gerrit, > > Gerrit Voß wrote: > > the problem is the explicit recorded refcounts in createScenegraph > > make the nodes be deleted before they are linked into their places > > during the app -> anim sync. > > ouch, that's not nice. but not the first time ;) > > Simple solution: > > - Don't use RecPtr for temporary stuff, they don't make any > > sense for that. > > true, but I think we both agree that crashing is not really user > friendly ;) ;) at least we don't format their harddrive ;) > > Real solution: > > - There should infrastructure to avoid this I'll see why these > > seem to make a way around it. > > one thing I noticed (might be unrelated though) is that in > ChangeList::doApply() add/subRecorded() is used when playing back ref > count changes. Shouldn't that be unrecorded? I thought the rule is that > changelist apply is not transitive, has that changed? I'm not really sure anymore. So I would guess currently they might be caught in the twilight zone in between. I just stumbled across the rec part too. I leave it for now and just fix the time when it is executed. kind regards, gerrit |