diff options
author | Christian Kandeler <[email protected]> | 2012-09-20 10:31:34 +0200 |
---|---|---|
committer | hjk <[email protected]> | 2012-09-20 11:11:45 +0200 |
commit | 938612c35e82bec69868e1bf6490c1b331085d82 (patch) | |
tree | 695b915720cc20a6524e0ea2f28a54a9fb24af7f /src/plugins/debugger/moduleshandler.cpp | |
parent | 1b3d637c1ba12cb9b8880000d05d1c864f81ce86 (diff) |
Do not use QAbstractItemModel::reset().
Change-Id: Ie49bdf576a6d3543aef6df133b27c8827775c935
Rationale:
a) It is conceptually cleaner to do the changes to the model
in between calls to beginResetModel() and endResetModel,
respectively.
b) The function is deprecated in Qt 5, for exactly that reason.
Reviewed-by: hjk <[email protected]>
Diffstat (limited to 'src/plugins/debugger/moduleshandler.cpp')
-rw-r--r-- | src/plugins/debugger/moduleshandler.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/plugins/debugger/moduleshandler.cpp b/src/plugins/debugger/moduleshandler.cpp index 645e944f2db..10786d29dff 100644 --- a/src/plugins/debugger/moduleshandler.cpp +++ b/src/plugins/debugger/moduleshandler.cpp @@ -195,15 +195,17 @@ QVariant ModulesModel::data(const QModelIndex &index, int role) const void ModulesModel::setModules(const Modules &m) { + beginResetModel(); m_modules = m; - reset(); + endResetModel(); } void ModulesModel::clearModel() { if (!m_modules.isEmpty()) { + beginResetModel(); m_modules.clear(); - reset(); + endResetModel(); } } |