From 6d8b0b3ab69714d4131262735c4c4b6c900f31f0 Mon Sep 17 00:00:00 2001 From: Jonas Kvinge Date: Tue, 22 Jun 2021 13:58:14 +0200 Subject: [PATCH] Simplify return --- src/core/mergedproxymodel.cpp | 4 +--- src/dialogs/edittagdialog.cpp | 6 +----- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/src/core/mergedproxymodel.cpp b/src/core/mergedproxymodel.cpp index 7369ab5e7..23f03873b 100644 --- a/src/core/mergedproxymodel.cpp +++ b/src/core/mergedproxymodel.cpp @@ -534,9 +534,7 @@ void MergedProxyModel::LayoutChanged() { bool MergedProxyModel::IsKnownModel(const QAbstractItemModel *model) const { - if (model == this || model == sourceModel() || merge_points_.contains(const_cast(model))) - return true; - return false; + return (model == this || model == sourceModel() || merge_points_.contains(const_cast(model))); } diff --git a/src/dialogs/edittagdialog.cpp b/src/dialogs/edittagdialog.cpp index 5bcc92bda..ba1ce53c6 100644 --- a/src/dialogs/edittagdialog.cpp +++ b/src/dialogs/edittagdialog.cpp @@ -515,11 +515,7 @@ bool EditTagDialog::DoesValueVary(const QModelIndexList &sel, const QString &id) bool EditTagDialog::IsValueModified(const QModelIndexList &sel, const QString &id) const { - if (std::any_of(sel.begin(), sel.end(), [this, id](const QModelIndex &i){ return data_[i.row()].original_value(id) != data_[i.row()].current_value(id); })) { - return true; - } - - return false; + return std::any_of(sel.begin(), sel.end(), [this, id](const QModelIndex &i){ return data_[i.row()].original_value(id) != data_[i.row()].current_value(id); }); }