diff options
| author | Noel Grandin <noel@peralex.com> | 2016-04-12 12:08:00 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-04-14 11:37:19 +0000 |
| commit | 62633dfe026bc3badf7066e5fb454036bae9cfdc (patch) | |
| tree | b1a30ddd709ba80b8629d916d54c03d733651d12 /forms/source/runtime | |
| parent | aa458c31a5dfa2be8b6bd8f1b4e402ce8c27edde (diff) | |
clang-tidy performance-unnecessary-value-param in various
Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907
Reviewed-on: https://gerrit.libreoffice.org/24019
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'forms/source/runtime')
| -rw-r--r-- | forms/source/runtime/formoperations.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/forms/source/runtime/formoperations.cxx b/forms/source/runtime/formoperations.cxx index 4f9884f78baf..e0f936726ea6 100644 --- a/forms/source/runtime/formoperations.cxx +++ b/forms/source/runtime/formoperations.cxx @@ -460,7 +460,7 @@ namespace frm return true; } - bool commit1Form(Reference< XFormController > xCntrl, bool &needConfirmation, bool &shouldCommit) + bool commit1Form(const Reference< XFormController >& xCntrl, bool &needConfirmation, bool &shouldCommit) { Reference< XFormOperations > xFrmOps(xCntrl->getFormOperations()); if (!xFrmOps->commitCurrentControl()) @@ -477,7 +477,7 @@ namespace frm return true; } - bool commitFormAndSubforms(Reference< XFormController > xCntrl, bool needConfirmation) + bool commitFormAndSubforms(const Reference< XFormController >& xCntrl, bool needConfirmation) { bool shouldCommit(true); assert(xCntrl.is()); @@ -504,7 +504,7 @@ namespace frm return true; } - bool commit1Form(Reference< XForm > xFrm, bool &needConfirmation, bool &shouldCommit) + bool commit1Form(const Reference< XForm >& xFrm, bool &needConfirmation, bool &shouldCommit) { Reference< XPropertySet > xProps(xFrm, UNO_QUERY_THROW); // nothing to do if the record is not modified @@ -525,7 +525,7 @@ namespace frm return true; } - bool commitFormAndSubforms(Reference< XForm > xFrm, bool needConfirmation) + bool commitFormAndSubforms(const Reference< XForm >& xFrm, bool needConfirmation) { // No control... do what we can with the models bool shouldCommit(true); |