Changeset 7516f6 for src/UIElements/Views/Qt4/QtGeometryList.cpp

Timestamp:

Jul 12, 2017, 7:11:51 PM (8 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Action_Thermostats, Adding_MD_integration_tests, Adding_StructOpt_integration_tests, AutomationFragmentation_failures, Candidate_v1.6.1, Candidate_v1.7.0, Candidate_v1.7.1, ChemicalSpaceEvaluator, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Exclude_Hydrogens_annealWithBondGraph, Fix_Verbose_Codepatterns, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults, ForceAnnealing_with_BondGraph_contraction-expansion, Gui_displays_atomic_force_velocity, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, PythonUI_with_named_parameters, Recreated_GuiChecks, StoppableMakroAction, TremoloParser_IncreasedPrecision, stable

Children:

4c6f0d

Parents:

b10593

git-author:

Frederik Heber <frederik.heber@…> (07/12/17 12:20:23)

git-committer:

Frederik Heber <frederik.heber@…> (07/12/17 19:11:51)

Message:

FIX: several Qt...Lists required mutex to control refill triggered by observable update.

• QtGeometryList, QtHomologyList, and QtShapeList
• also QtStatusBar, and QtTimeLine.

File:

• src/UIElements/Views/Qt4/QtGeometryList.cpp (modified) (3 diffs)

src/UIElements/Views/Qt4/QtGeometryList.cpp

@@ -109 +109 @@
 void QtGeometryList::refill(::GeometryObject *ignore)
 {
+  boost::recursive_mutex::scoped_lock lock(refill_mutex);
+
   clear();
 
@@ -125 +127 @@
 }
 
-#if 0
 void QtGeometryList::paintEvent(QPaintEvent * event)
 {
+  boost::recursive_mutex::scoped_lock lock(refill_mutex);
   /*if (dirty)
     refill(NULL);*/
-  QtGeometryList::paintEvent(event);
+  QTreeWidget::paintEvent(event);
 }
-#endif
 
 void QtGeometryList::subjectKilled(Observable *publisher) {
@@ -151 +152 @@
 void QtGeometryList::mouseMoveFunction(QMouseEvent * event)
 {
+  boost::recursive_mutex::scoped_lock lock(refill_mutex);
+
   if (event->type() == QEvent::MouseMove) {
     QTreeWidgetItem* current = itemAt(event->pos());