diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-23 09:17:30 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-12-23 12:36:24 +0100 |
| commit | a8c1c0ae1617c68d49deb33e9236a2c1b7cac14d (patch) | |
| tree | dec7ceaca24da8340a911502fd42dd19efb882ff /svl | |
| parent | 38bf5f69663f64434a3a0a74e02c1a23f876b677 (diff) | |
loplugin:passstuffbyref improved returns in vcl
Change-Id: I0b103df2e7ce59093869f547225c95865d33da27
Reviewed-on: https://gerrit.libreoffice.org/46916
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/items/stylepool.cxx | 6 | ||||
| -rw-r--r-- | svl/source/misc/urihelper.cxx | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/svl/source/items/stylepool.cxx b/svl/source/items/stylepool.cxx index 15be4eb971ed..04d801c3ac5b 100644 --- a/svl/source/items/stylepool.cxx +++ b/svl/source/items/stylepool.cxx @@ -61,11 +61,11 @@ namespace { // #i86923# bool hasItemSet( const bool bCheckUsage ) const; // #i87808# - const std::shared_ptr<SfxItemSet> getItemSet() const + std::shared_ptr<SfxItemSet> const & getItemSet() const { return maItemSet.back(); } - const std::shared_ptr<SfxItemSet> getUsedOrLastAddedItemSet() const; + std::shared_ptr<SfxItemSet> const & getUsedOrLastAddedItemSet() const; void setItemSet( const SfxItemSet& rSet ){ maItemSet.push_back( std::shared_ptr<SfxItemSet>( rSet.Clone() ) ); } // #i86923# Node* findChildNode( const SfxPoolItem& rItem, @@ -80,7 +80,7 @@ namespace { }; // #i87808# - const std::shared_ptr<SfxItemSet> Node::getUsedOrLastAddedItemSet() const + std::shared_ptr<SfxItemSet> const & Node::getUsedOrLastAddedItemSet() const { std::vector< std::shared_ptr<SfxItemSet> >::const_reverse_iterator aIter; diff --git a/svl/source/misc/urihelper.cxx b/svl/source/misc/urihelper.cxx index fc5e6322ec02..8c059fc15a80 100644 --- a/svl/source/misc/urihelper.cxx +++ b/svl/source/misc/urihelper.cxx @@ -112,7 +112,7 @@ void URIHelper::SetMaybeFileHdl(Link<OUString *, bool> const & rTheMaybeFileHdl) MaybeFileHdl::get() = rTheMaybeFileHdl; } -Link<OUString *, bool> URIHelper::GetMaybeFileHdl() +Link<OUString *, bool> const & URIHelper::GetMaybeFileHdl() { return MaybeFileHdl::get(); } |