Speed up controllers #682
Speed up controllers #682
Conversation
|
@macumber could you add a quick walkthrough of your code via review + some quick text explaining why removing the mutex is the right choice please? |
jmarrec
added
Enhancement Request
| @@ -105,7 +103,6 @@ class OSItemList : public OSItemSelector | |||
| bool m_itemsDraggable; | |||
| bool m_itemsRemoveable; | |||
| OSItemType m_type; | |||
| bool m_dirty; | |||
There was a problem hiding this comment.
m_dirty wasn't actually controlling anything here, all that refresh was doing was to clear m_dirty
| delete m_reportItemsMutex; | ||
| } | ||
|
|
||
| void ConstructionObjectVectorController::reportItemsLater() { |
There was a problem hiding this comment.
Only a few OSVectorController derived classes were specialized to have the reportItemsLater slot which schedules a call to OSVectorController::reportItems. This PR pulls that functionality into the OSVectorController. Now reportItems will schedule a call to a reportItemsImpl method that will run once so we don't report (and redraw) the same objects multipe times per execute loop.
| @@ -60,15 +62,20 @@ class ModelObjectListController : public OSVectorController | |||
| private: | |||
| openstudio::IddObjectType m_iddObjectType; | |||
| model::Model m_model; | |||
| bool m_showLocalBCL; | |||
| bool m_isLibrary; | |||
There was a problem hiding this comment.
m_showLocalBCL meant to show LocalBCL objects, basically just meant that this was a library controller, I thought m_isLibrary was more clear.
Fixes #681 by moving optimizations in certain controllers to all controllers