clang-tidy readability-simplify-boolean-expr in dbaccess..framework

Change-Id: I96e1bd4000f4ade6ccfac53c57653772b249df99
Reviewed-on: https://gerrit.libreoffice.org/36678
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

2 files changed, 3 insertions, 11 deletions

diff --git a/forms/source/component/CheckBox.cxx b/forms/source/component/CheckBox.cxx
index 65178f68ddc4..9910ac26c9bf 100644
--- a/forms/source/component/CheckBox.cxx
+++ b/forms/source/component/CheckBox.cxx
@@ -181,9 +181,7 @@ void SAL_CALL OCheckBoxModel::read(const Reference<css::io::XObjectInputStream>&
 
 bool OCheckBoxModel::DbUseBool()
 {
-    if ( ! (getReferenceValue().isEmpty() && getNoCheckReferenceValue().isEmpty()) )
-        return false;
-    return true;
+    return getReferenceValue().isEmpty() && getNoCheckReferenceValue().isEmpty();
 }
 
 
diff --git a/forms/source/runtime/formoperations.cxx b/forms/source/runtime/formoperations.cxx
index 4365661b9c60..665c61a8631f 100644
--- a/forms/source/runtime/formoperations.cxx
+++ b/forms/source/runtime/formoperations.cxx
@@ -486,10 +486,7 @@ namespace frm
                 }
             }
 
-            if(!commit1Form(xCntrl, needConfirmation, shouldCommit))
-                return false;
-
-            return true;
+            return commit1Form(xCntrl, needConfirmation, shouldCommit);
         }
 
         bool commit1Form(const Reference< XForm >& xFrm, bool &needConfirmation, bool &shouldCommit)
@@ -531,10 +528,7 @@ namespace frm
                 }
             }
 
-            if(!commit1Form(xFrm, needConfirmation, shouldCommit))
-                return false;
-
-            return true;
+            return commit1Form(xFrm, needConfirmation, shouldCommit);
         }
     }