mirror of
https://invent.kde.org/system/dolphin
synced 2024-09-17 15:31:20 +00:00
Remove redundant code
Summary: Remove redundant negation and the AND check. OR is enough. Reviewers: meven, elvisangelaccio Reviewed By: meven, elvisangelaccio Subscribers: elvisangelaccio, meven, kfm-devel Tags: #dolphin Differential Revision: https://phabricator.kde.org/D23129
This commit is contained in:
parent
c18f5fde4b
commit
0ab2779058
|
@ -280,7 +280,7 @@ void DolphinViewActionHandler::createActions()
|
||||||
QActionGroup* DolphinViewActionHandler::createFileItemRolesActionGroup(const QString& groupPrefix)
|
QActionGroup* DolphinViewActionHandler::createFileItemRolesActionGroup(const QString& groupPrefix)
|
||||||
{
|
{
|
||||||
const bool isSortGroup = (groupPrefix == QLatin1String("sort_by_"));
|
const bool isSortGroup = (groupPrefix == QLatin1String("sort_by_"));
|
||||||
Q_ASSERT(isSortGroup || (!isSortGroup && groupPrefix == QLatin1String("show_")));
|
Q_ASSERT(isSortGroup || groupPrefix == QLatin1String("show_"));
|
||||||
|
|
||||||
QActionGroup* rolesActionGroup = new QActionGroup(m_actionCollection);
|
QActionGroup* rolesActionGroup = new QActionGroup(m_actionCollection);
|
||||||
rolesActionGroup->setExclusive(isSortGroup);
|
rolesActionGroup->setExclusive(isSortGroup);
|
||||||
|
|
Loading…
Reference in a new issue