diff options
| author | Tobias Hunger <tobias.hunger@qt.io> | 2017-05-31 15:48:45 +0200 |
|---|---|---|
| committer | Tim Jenssen <tim.jenssen@qt.io> | 2017-08-01 11:24:53 +0000 |
| commit | 45046f7071f6f4886d697547b508cbd087daf342 (patch) | |
| tree | 5524bb8e5bb0b559323b84b4381a967b8f901eee /src/plugins/modeleditor/modelsmanager.cpp | |
| parent | bbb54cdebae2d2a2a275c69e4d4309a83087a02a (diff) | |
ProjectNodes: Do not derive Project Nodes from QObject
That should save some memory per node, and since creator has a lot of nodes
(e.g. opening the LLVM project adds about 1 000 000 nodes) this should be
noticeable:-)
Calling update inside ProjectTree::currentNode() and rename it to
findCurrentNode() to make sure it is an still existing pointer.
Also, try to reduce the somehow more expensive currentNode() calls
and sprinkle some const around that usage.
Change-Id: I6a7c5db01a71d53d39544d3013cad557d5b96cdc
Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>
Diffstat (limited to 'src/plugins/modeleditor/modelsmanager.cpp')
| -rw-r--r-- | src/plugins/modeleditor/modelsmanager.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/modeleditor/modelsmanager.cpp b/src/plugins/modeleditor/modelsmanager.cpp index 5088f97a83..b49c9ff230 100644 --- a/src/plugins/modeleditor/modelsmanager.cpp +++ b/src/plugins/modeleditor/modelsmanager.cpp @@ -188,7 +188,7 @@ void ModelsManager::onAboutToShowContextMenu(ProjectExplorer::Project *project, void ModelsManager::onOpenDiagramFromProjectExplorer() { - if (ProjectExplorer::ProjectTree::instance()->currentNode() == d->contextMenuOwnerNode) { + if (ProjectExplorer::ProjectTree::findCurrentNode() == d->contextMenuOwnerNode) { qmt::MDiagram *diagram = 0; foreach (const ManagedModel &managedModel, d->managedModels) { if ((diagram = managedModel.m_documentController->pxNodeController()->findDiagramForExplorerNode(d->contextMenuOwnerNode))) { |