summaryrefslogtreecommitdiff
path: root/forms/source/runtime
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2017-04-19 13:13:16 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2017-04-19 14:52:50 +0200
commit447a013299d148df12ff17306fff77bb7f85eba1 (patch)
treed577aabfb9b650bc46fdcf2289aae2b1561f1e4e /forms/source/runtime
parent78098b8494be7123bc4a8b50faa13445e5afd8ce (diff)
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>
Diffstat (limited to 'forms/source/runtime')
-rw-r--r--forms/source/runtime/formoperations.cxx10
1 files changed, 2 insertions, 8 deletions
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);
}
}