Bug 93037

Summary: purge deletes objects with scalars in use
Product: [Applications] kst Reporter: George Staikos <staikos>
Component: generalAssignee: kst
Status: RESOLVED FIXED    
Severity: major    
Priority: NOR    
Version: 1.x   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description George Staikos 2004-11-10 17:02:37 UTC 
How to reproduce: 
1) Create a vector, don't use it 
2) Create a label and use a scalar from the vector to monitor, for instance, 
last sample 
3) Purge 
 
The vector is destroyed so the label never updates.
Comment 1 Andrew Walker 2004-11-11 23:23:06 UTC 
Options appear to be:

* Replace value with something appropriate (e.g. --- or ???)
* Reference count the vector in the label so it isn't purged

Which is prefered?
Comment 2 George Staikos 2004-11-11 23:54:45 UTC 
On Thursday 11 November 2004 17:23, Andrew Walker wrote:
> * Replace value with something appropriate (e.g. --- or ???)
> * Reference count the vector in the label so it isn't purged
>
> Which is prefered?

   or c) reference count scalars.

   I am very hesitant to put this into 1.0 at this point, but it is a rather 
big bug too.  I noticed that somehow we manage to reference count with 
scalars properly in some places and not others, so maybe we're just missing 
something in one small place.  Let's approach this cautiously.  I don't like 
solution a), but b) and c) are both reasonable to me at this point.
Comment 3 George Staikos 2005-03-21 03:01:32 UTC 
CVS commit by staikos: 

Object dependency fixes - scalars in particular, plus other random fixes
CCBUG: 93037


  M +13 -11    enodes.cpp   1.46
  M +6 -5      enodes.h   1.31
  M +6 -3      kstequation.cpp   1.32
  M +3 -2      ksteventmonitorentry.cpp   1.37
  M +1 -1      kstplugin.cpp   1.88
  M +11 -0     kstvector.cpp   1.101
  M +1 -1      kstvector.h   1.66
Comment 4 George Staikos 2005-03-21 03:12:53 UTC 
Generally fixed, but there might be more individual cases where it fails.
Comment 5 Peter Kümmel 2010-08-14 14:40:30 UTC 
Change version to 1.x