Bug 93037 - purge deletes objects with scalars in use
Summary: purge deletes objects with scalars in use
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: Compiled Sources Linux
: NOR major
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-11-10 17:02 UTC by George Staikos
Modified: 2010-08-14 14:40 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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