diff options
author | Noel Grandin <noel@peralex.com> | 2016-04-11 10:07:25 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2016-04-11 11:38:57 +0200 |
commit | bfff081e74ad06a7e353669eec345c6440db57cb (patch) | |
tree | 0c313317850eba08686344c75035e0ae4388729d /svl | |
parent | 9f877f677346968774d4366f105114fe7f36dd69 (diff) |
clang-tidy performance-unnecessary-value-param in svl
Change-Id: I8abc272d12e62e25be358dc4c73008971f1cd609
Diffstat (limited to 'svl')
-rw-r--r-- | svl/source/items/IndexedStyleSheets.cxx | 6 | ||||
-rw-r--r-- | svl/source/items/style.cxx | 2 | ||||
-rw-r--r-- | svl/source/items/stylepool.cxx | 2 | ||||
-rw-r--r-- | svl/source/misc/documentlockfile.cxx | 2 | ||||
-rw-r--r-- | svl/source/numbers/currencytable.cxx | 2 |
5 files changed, 7 insertions, 7 deletions
diff --git a/svl/source/items/IndexedStyleSheets.cxx b/svl/source/items/IndexedStyleSheets.cxx index f543d18b15d6..750a7fd18771 100644 --- a/svl/source/items/IndexedStyleSheets.cxx +++ b/svl/source/items/IndexedStyleSheets.cxx @@ -86,7 +86,7 @@ IndexedStyleSheets::GetNumberOfStyleSheets() const } void -IndexedStyleSheets::AddStyleSheet(rtl::Reference< SfxStyleSheetBase > style) +IndexedStyleSheets::AddStyleSheet(const rtl::Reference< SfxStyleSheetBase >& style) { if (!HasStyleSheet(style)) { mStyleSheets.push_back(style); @@ -96,7 +96,7 @@ IndexedStyleSheets::AddStyleSheet(rtl::Reference< SfxStyleSheetBase > style) } bool -IndexedStyleSheets::RemoveStyleSheet(rtl::Reference< SfxStyleSheetBase > style) +IndexedStyleSheets::RemoveStyleSheet(const rtl::Reference< SfxStyleSheetBase >& style) { rtl::OUString styleName = style->GetName(); std::vector<unsigned> positions = FindPositionsByName(styleName); @@ -207,7 +207,7 @@ IndexedStyleSheets::~IndexedStyleSheets() {;} bool -IndexedStyleSheets::HasStyleSheet(rtl::Reference< SfxStyleSheetBase > style) const +IndexedStyleSheets::HasStyleSheet(const rtl::Reference< SfxStyleSheetBase >& style) const { rtl::OUString styleName = style->GetName(); std::vector<unsigned> positions = FindPositionsByName(styleName); diff --git a/svl/source/items/style.cxx b/svl/source/items/style.cxx index 1ec3a1870c4b..4221020c5077 100644 --- a/svl/source/items/style.cxx +++ b/svl/source/items/style.cxx @@ -965,7 +965,7 @@ SfxUnoStyleSheet* SfxUnoStyleSheet::getUnoStyleSheet( const css::uno::Reference< } void -SfxStyleSheetBasePool::StoreStyleSheet(rtl::Reference< SfxStyleSheetBase > xStyle) +SfxStyleSheetBasePool::StoreStyleSheet(const rtl::Reference< SfxStyleSheetBase >& xStyle) { pImp->mxIndexedStyleSheets->AddStyleSheet(xStyle); } diff --git a/svl/source/items/stylepool.cxx b/svl/source/items/stylepool.cxx index f75435a6c59b..f260672d8d4e 100644 --- a/svl/source/items/stylepool.cxx +++ b/svl/source/items/stylepool.cxx @@ -332,7 +332,7 @@ namespace { * This static method creates a unique name from a shared pointer to a SfxItemSet * The name is the memory address of the SfxItemSet itself. */ -OUString StylePool::nameOf( SfxItemSet_Pointer_t pSet ) +OUString StylePool::nameOf( const SfxItemSet_Pointer_t& pSet ) { return OUString::number( reinterpret_cast<sal_IntPtr>( pSet.get() ), 16 ); } diff --git a/svl/source/misc/documentlockfile.cxx b/svl/source/misc/documentlockfile.cxx index 6cd169972739..d71e2e958e5a 100644 --- a/svl/source/misc/documentlockfile.cxx +++ b/svl/source/misc/documentlockfile.cxx @@ -63,7 +63,7 @@ DocumentLockFile::~DocumentLockFile() } -void DocumentLockFile::WriteEntryToStream( const LockFileEntry& aEntry, uno::Reference< io::XOutputStream > xOutput ) +void DocumentLockFile::WriteEntryToStream( const LockFileEntry& aEntry, const uno::Reference< io::XOutputStream >& xOutput ) { ::osl::MutexGuard aGuard( m_aMutex ); diff --git a/svl/source/numbers/currencytable.cxx b/svl/source/numbers/currencytable.cxx index 17e362d1c833..380e47bd4a2e 100644 --- a/svl/source/numbers/currencytable.cxx +++ b/svl/source/numbers/currencytable.cxx @@ -29,7 +29,7 @@ size_t NfCurrencyTable::size() const return maData.size(); } -void NfCurrencyTable::insert(iterator it, std::unique_ptr<NfCurrencyEntry> p) +void NfCurrencyTable::insert(const iterator& it, std::unique_ptr<NfCurrencyEntry> p) { maData.insert(it, std::move(p)); } |