From 0ab27790588548d5c8cf114613f768f0514f92d6 Mon Sep 17 00:00:00 2001
From: Antonio Prcela <antonio.prcela@gmail.com>
Date: Tue, 13 Aug 2019 23:22:05 +0200
Subject: [PATCH] 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
---
 src/views/dolphinviewactionhandler.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/views/dolphinviewactionhandler.cpp b/src/views/dolphinviewactionhandler.cpp
index 18b1c2d970..27321f63a6 100644
--- a/src/views/dolphinviewactionhandler.cpp
+++ b/src/views/dolphinviewactionhandler.cpp
@@ -280,7 +280,7 @@ void DolphinViewActionHandler::createActions()
 QActionGroup* DolphinViewActionHandler::createFileItemRolesActionGroup(const QString& groupPrefix)
 {
     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);
     rolesActionGroup->setExclusive(isSortGroup);