diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-08-05 11:21:44 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-08-07 13:32:30 +0200 |
| commit | 5b112c8f8843613873181fce50990a8feef837ca (patch) | |
| tree | 9ebd9d01d49a0c6a058de954dc167fd4ccb4ff5c /sc/source/ui/dataprovider/dataprovider.cxx | |
| parent | 799753cb80f4c81a6e87bf016924e0f2a29f03f0 (diff) | |
clang-tidy modernize-pass-by-value in sc
Change-Id: Ia7ff651d1cbc119b36a9f8052594d03650988f59
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137848
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/ui/dataprovider/dataprovider.cxx')
| -rw-r--r-- | sc/source/ui/dataprovider/dataprovider.cxx | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/sc/source/ui/dataprovider/dataprovider.cxx b/sc/source/ui/dataprovider/dataprovider.cxx index 72f674a7b99d..cc83294366cb 100644 --- a/sc/source/ui/dataprovider/dataprovider.cxx +++ b/sc/source/ui/dataprovider/dataprovider.cxx @@ -24,6 +24,7 @@ #include <datamapper.hxx> #include <dbdata.hxx> #include <docsh.hxx> +#include <utility> using namespace com::sun::star; @@ -63,10 +64,10 @@ std::unique_ptr<SvStream> DataProvider::FetchStreamFromURL(const OUString& rURL, } } -ExternalDataSource::ExternalDataSource(const OUString& rURL, - const OUString& rProvider, ScDocument* pDoc) - : maURL(rURL) - , maProvider(rProvider) +ExternalDataSource::ExternalDataSource(OUString aURL, + OUString aProvider, ScDocument* pDoc) + : maURL(std::move(aURL)) + , maProvider(std::move(aProvider)) , mpDoc(pDoc) { } @@ -246,8 +247,8 @@ void ScDBDataManager::WriteToDoc(ScDocument& rDoc) pDocShell->PostPaint(aDestRange, PaintPartFlags::All); } -ScDBDataManager::ScDBDataManager(const OUString& rDBName, ScDocument* pDoc): - maDBName(rDBName), +ScDBDataManager::ScDBDataManager(OUString aDBName, ScDocument* pDoc): + maDBName(std::move(aDBName)), mpDoc(pDoc) { } |