sc/source/ui/condformat/condformateasydlg.cxx | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-)
New commits: commit 89fe841cdcccfd34e7e092af7d8cd4d952175a2f Author: codewithvk <vivek.jav...@collabora.com> AuthorDate: Thu Oct 17 16:16:05 2024 +0530 Commit: Pranam Lashkari <lpra...@collabora.com> CommitDate: Fri Oct 18 14:44:45 2024 +0200 sc: simplify boolean checks by removing redundant comparisons Signed-off-by: codewithvk <vivek.jav...@collabora.com> Change-Id: I2012a757ce9a70b9edb6a344973b5088b5cfa20b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175078 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Pranam Lashkari <lpra...@collabora.com> diff --git a/sc/source/ui/condformat/condformateasydlg.cxx b/sc/source/ui/condformat/condformateasydlg.cxx index 8bc37d863810..8480a47e884a 100644 --- a/sc/source/ui/condformat/condformateasydlg.cxx +++ b/sc/source/ui/condformat/condformateasydlg.cxx @@ -356,14 +356,12 @@ IMPL_LINK(ConditionalFormatEasyDialog, ButtonPressed, weld::Button&, rButton, vo std::unique_ptr<ScConditionalFormat> pFormat(new ScConditionalFormat(0, mpDocument)); - OUString sExpression1 - = (mxNumberEntry->get_visible() == true && mxAllInputs->get_visible() == true - ? mxNumberEntry->get_text() - : ""); - OUString sExpression2 - = (mxNumberEntry2->get_visible() == true && mxAllInputs->get_visible() == true - ? mxNumberEntry2->get_text() - : ""); + OUString sExpression1 = (mxNumberEntry->get_visible() && mxAllInputs->get_visible() + ? mxNumberEntry->get_text() + : ""); + OUString sExpression2 = (mxNumberEntry2->get_visible() && mxAllInputs->get_visible() + ? mxNumberEntry2->get_text() + : ""); switch (meMode) {