D11518: Use model reset rather than beginRemoveRows() to clear font groups
Daniel Vrátil
noreply at phabricator.kde.org
Tue Mar 20 17:37:27 UTC 2018
dvratil created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
dvratil requested review of this revision.
REVISION SUMMARY
Calling beginRemoveRows() when the model is empty hits an assert in
QAbstractItemModel which requires that the "last" argument is less
than model row count, which is not true when the model is empty.
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D11518
AFFECTED FILES
kcms/kfontinst/kcmfontinst/GroupList.cpp
To: dvratil
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180320/5adccf3e/attachment.html>
More information about the Plasma-devel
mailing list