Changeset 8c076a for src/UIElements/Views/Qt4/QtFragmentList.cpp

Timestamp:

May 5, 2016, 5:14:19 PM (10 years ago)

Author:

Frederik Heber <heber@…>

Children:

4a33f6

Parents:

fa6af2

git-author:

Frederik Heber <heber@…> (05/05/16 11:46:01)

git-committer:

Frederik Heber <heber@…> (05/05/16 17:14:19)

Message:

FIX: Replaced all uses of getInstance() inside dstors to signOff() by stored Observable ptrs.

• this is cleaner anyway and how it should be done. We require the pointer eventually to signOff. Also, it serves as an indicator to whether we already received subjectKilled() on it.

File:

• src/UIElements/Views/Qt4/QtFragmentList.cpp (modified) (4 diffs)

src/UIElements/Views/Qt4/QtFragmentList.cpp

@@ -59 +59 @@
 QtFragmentList::QtFragmentList(QWidget * _parent) :
     QTreeWidget (_parent),
-    Observer("QtFragmentList")
+    Observer("QtFragmentList"),
+    observed_fragmentationresultcontainer(FragmentationResultContainer::getPointer())
 {
   setColumnCount(COLUMNCOUNT);
@@ -74 +75 @@
   refill();
 
-  FragmentationResultContainer &fragments =
-      FragmentationResultContainer::getInstance();
-        fragments.signOn(this);
+  observed_fragmentationresultcontainer->signOn(this);
 
   connect(this,SIGNAL(itemSelectionChanged()),this,SLOT(rowSelected()));
@@ -84 +83 @@
 QtFragmentList::~QtFragmentList()
 {
-  FragmentationResultContainer &fragments =
-      FragmentationResultContainer::getInstance();
-  fragments.signOff(this);
+  if (observed_fragmentationresultcontainer)
+    observed_fragmentationresultcontainer->signOff(this);
 }
 
@@ -186 +184 @@
 void QtFragmentList::subjectKilled(Observable *publisher)
 {
-  // as a new instance should always already be present ... just sign on
-  FragmentationResultContainer &fragments =
-      FragmentationResultContainer::getInstance();
-  fragments.signOn(this);
+  if (publisher == observed_fragmentationresultcontainer)
+    observed_fragmentationresultcontainer = NULL;
+  else
+    ASSERT(0,"QtFragmentList::subjectKilled() - unknown publisher "
+        +toString(publisher));
 }