cid#1555415 COPY_INSTEAD_OF_MOVE

and

cid#1555430 COPY_INSTEAD_OF_MOVE
cid#1555497 COPY_INSTEAD_OF_MOVE
cid#1555542 COPY_INSTEAD_OF_MOVE
cid#1555543 COPY_INSTEAD_OF_MOVE
cid#1555564 COPY_INSTEAD_OF_MOVE
cid#1555593 COPY_INSTEAD_OF_MOVE
cid#1555602 COPY_INSTEAD_OF_MOVE
cid#1555610 COPY_INSTEAD_OF_MOVE
cid#1555667 COPY_INSTEAD_OF_MOVE
cid#1555693 COPY_INSTEAD_OF_MOVE
cid#1555719 COPY_INSTEAD_OF_MOVE
cid#1555757 COPY_INSTEAD_OF_MOVE
cid#1555795 COPY_INSTEAD_OF_MOVE
cid#1555826 COPY_INSTEAD_OF_MOVE
cid#1555847 COPY_INSTEAD_OF_MOVE
cid#1555851 COPY_INSTEAD_OF_MOVE
cid#1555863 COPY_INSTEAD_OF_MOVE
cid#1555865 COPY_INSTEAD_OF_MOVE
cid#1555867 COPY_INSTEAD_OF_MOVE
cid#1555869 COPY_INSTEAD_OF_MOVE
cid#1555879 COPY_INSTEAD_OF_MOVE
cid#1555890 COPY_INSTEAD_OF_MOVE
cid#1555893 COPY_INSTEAD_OF_MOVE
cid#1555900 COPY_INSTEAD_OF_MOVE
cid#1555922 COPY_INSTEAD_OF_MOVE
cid#1555925 COPY_INSTEAD_OF_MOVE

Change-Id: I5d362a09e80eecd282e63c745bfe26eac2631a0d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171492
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Tested-by: Jenkins

6 files changed, 8 insertions, 8 deletions

diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx
index 028975850f96..e1455e735e88 100644
--- a/sc/source/core/data/documen2.cxx
+++ b/sc/source/core/data/documen2.cxx
@@ -1039,14 +1039,14 @@ bool ScDocument::TransferTab( ScDocument& rSrcDoc, SCTAB nSrcPos,
         {
             aRepeatColRange->aStart.SetTab(nDestPos);
             aRepeatColRange->aEnd.SetTab(nDestPos);
-            maTabs[nDestPos]->SetRepeatColRange(aRepeatColRange);
+            maTabs[nDestPos]->SetRepeatColRange(std::move(aRepeatColRange));
         }
 
         if (auto aRepeatRowRange = rSrcDoc.maTabs[nSrcPos]->GetRepeatRowRange())
         {
             aRepeatRowRange->aStart.SetTab(nDestPos);
             aRepeatRowRange->aEnd.SetTab(nDestPos);
-            maTabs[nDestPos]->SetRepeatRowRange(aRepeatRowRange);
+            maTabs[nDestPos]->SetRepeatRowRange(std::move(aRepeatRowRange));
         }
 
         if (rSrcDoc.IsPrintEntireSheet(nSrcPos))
diff --git a/sc/source/core/data/queryiter.cxx b/sc/source/core/data/queryiter.cxx
index 5c93502db908..e1558fcc5304 100644
--- a/sc/source/core/data/queryiter.cxx
+++ b/sc/source/core/data/queryiter.cxx
@@ -1022,7 +1022,7 @@ bool ScQueryCellIterator< accessType >::FindEqualOrSortedLastInRange( SCCOL& nFo
         // Step back conditions are the same as above
         nCol = nFoundCol;
         nRow = nFoundRow;
-        maCurPos = aPosSave;
+        maCurPos = std::move(aPosSave);
         return true;
     }
     if ( (maParam.eSearchType != utl::SearchParam::SearchType::Normal) &&
diff --git a/sc/source/core/tool/interpr5.cxx b/sc/source/core/tool/interpr5.cxx
index 04cc6d34eac4..3db43c8677cd 100644
--- a/sc/source/core/tool/interpr5.cxx
+++ b/sc/source/core/tool/interpr5.cxx
@@ -1448,11 +1448,11 @@ void ScInterpreter::ScAmpersand()
     {
         OUString sStr;
         bool bFlag;
-        ScMatrixRef pMat = pMat1;
+        ScMatrixRef pMat = std::move(pMat1);
         if (!pMat)
         {
             sStr = sStr1;
-            pMat = pMat2;
+            pMat = std::move(pMat2);
             bFlag = true;           // double - Matrix
         }
         else
diff --git a/sc/source/filter/xml/xmlwrap.cxx b/sc/source/filter/xml/xmlwrap.cxx
index 9052cb2106c1..2bab3ae9a43e 100644
--- a/sc/source/filter/xml/xmlwrap.cxx
+++ b/sc/source/filter/xml/xmlwrap.cxx
@@ -199,7 +199,7 @@ ErrCodeMsg ScXMLImportWrapper::ImportFromComponent(const uno::Reference<uno::XCo
         {
             xml::sax::SAXException aTmp;
             if ( aSaxEx.WrappedException >>= aTmp )
-                aSaxEx = aTmp;
+                aSaxEx = std::move(aTmp);
             else
                 bTryChild = false;
         }
diff --git a/sc/source/ui/drawfunc/drawsh.cxx b/sc/source/ui/drawfunc/drawsh.cxx
index c4006146827e..c818820a7cda 100644
--- a/sc/source/ui/drawfunc/drawsh.cxx
+++ b/sc/source/ui/drawfunc/drawsh.cxx
@@ -374,7 +374,7 @@ void ScDrawShell::ExecDrawAttr( SfxRequest& rReq )
                                 SvxAbstractDialogFactory* pFact = SvxAbstractDialogFactory::Create();
                                 VclPtr<SfxAbstractTabDialog> pDlg(pFact->CreateSvxTransformTabDialog(pWin ? pWin->GetFrameWeld() : nullptr, &aNewAttr, pView));
 
-                                pDlg->StartExecuteAsync([pDlg, pRequest, pView, this](
+                                pDlg->StartExecuteAsync([pDlg, pRequest=std::move(pRequest), pView, this](
                                                             sal_Int32 nResult){
                                     if (nResult == RET_OK)
                                     {
diff --git a/sc/source/ui/miscdlgs/optsolver.cxx b/sc/source/ui/miscdlgs/optsolver.cxx
index 3a9d319bf015..7ef0eba43b15 100644
--- a/sc/source/ui/miscdlgs/optsolver.cxx
+++ b/sc/source/ui/miscdlgs/optsolver.cxx
@@ -353,7 +353,7 @@ void ScOptSolverDlg::ReadConditions()
             m_aConditions.resize( nVecPos + 1 );
 
         if ( nVecPos < static_cast<tools::Long>(m_aConditions.size()) )
-            m_aConditions[nVecPos] = aRowEntry;
+            m_aConditions[nVecPos] = std::move(aRowEntry);
 
         // remove default entries at the end
         size_t nSize = m_aConditions.size();