diff options
| author | Noel Grandin <noel@peralex.com> | 2016-07-18 09:22:27 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2016-07-27 06:48:25 +0000 |
| commit | 508c95f1b655d9cfa6be37a5a9de9aff6fd383bf (patch) | |
| tree | d1c8626818cbf26a699875ae2d82f751a1657e92 /l10ntools | |
| parent | 9f4af777a832d8a0b9a21d793d421fa6228131e0 (diff) | |
improve passstuffbyref return analysis
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c
Reviewed-on: https://gerrit.libreoffice.org/27317
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'l10ntools')
| -rw-r--r-- | l10ntools/inc/export.hxx | 2 | ||||
| -rw-r--r-- | l10ntools/inc/po.hxx | 6 | ||||
| -rw-r--r-- | l10ntools/source/merge.cxx | 2 | ||||
| -rw-r--r-- | l10ntools/source/po.cxx | 6 |
4 files changed, 8 insertions, 8 deletions
diff --git a/l10ntools/inc/export.hxx b/l10ntools/inc/export.hxx index de210396b075..4984800b8a50 100644 --- a/l10ntools/inc/export.hxx +++ b/l10ntools/inc/export.hxx @@ -257,7 +257,7 @@ class MergeDataHashMap typedef HashMap_t::const_iterator const_iterator; std::pair<iterator,bool> insert(const OString& rKey, MergeData* pMergeData); - iterator find(const OString& rKey); + iterator const & find(const OString& rKey); iterator begin() {return m_aHashMap.begin();} iterator end() {return m_aHashMap.end();} diff --git a/l10ntools/inc/po.hxx b/l10ntools/inc/po.hxx index 3004a1817cf3..ff8567e8db19 100644 --- a/l10ntools/inc/po.hxx +++ b/l10ntools/inc/po.hxx @@ -52,13 +52,13 @@ public: PoEntry( const PoEntry& rPo ); PoEntry& operator=( const PoEntry& rPo ); - OString getSourceFile() const; ///< Get name of file from which entry is extracted + OString const & getSourceFile() const; ///< Get name of file from which entry is extracted OString getGroupId() const; OString getLocalId() const; OString getResourceType() const; ///< Get the type of component from which entry is extracted TYPE getType() const; ///< Get the type of entry - OString getMsgId() const; - OString getMsgStr() const; + OString const & getMsgId() const; + OString const & getMsgStr() const; bool isFuzzy() const; /// Check whether po-s belong to the same localization component diff --git a/l10ntools/source/merge.cxx b/l10ntools/source/merge.cxx index 133f8f3f8c57..f927420fb30d 100644 --- a/l10ntools/source/merge.cxx +++ b/l10ntools/source/merge.cxx @@ -160,7 +160,7 @@ std::pair<MergeDataHashMap::iterator,bool> MergeDataHashMap::insert(const OStrin return aTemp; } -MergeDataHashMap::iterator MergeDataHashMap::find(const OString& rKey) +MergeDataHashMap::iterator const & MergeDataHashMap::find(const OString& rKey) { iterator aHint = m_aHashMap.end(); diff --git a/l10ntools/source/po.cxx b/l10ntools/source/po.cxx index 4662d58e0567..f446a2f31672 100644 --- a/l10ntools/source/po.cxx +++ b/l10ntools/source/po.cxx @@ -289,7 +289,7 @@ PoEntry& PoEntry::operator=(const PoEntry& rPo) return *this; } -OString PoEntry::getSourceFile() const +OString const & PoEntry::getSourceFile() const { assert( m_bIsInitialized ); return m_pGenPo->getReference(); @@ -343,14 +343,14 @@ bool PoEntry::isFuzzy() const } // Get translation string in merge format -OString PoEntry::getMsgId() const +OString const & PoEntry::getMsgId() const { assert( m_bIsInitialized ); return m_pGenPo->getMsgId(); } // Get translated string in merge format -OString PoEntry::getMsgStr() const +const OString& PoEntry::getMsgStr() const { assert( m_bIsInitialized ); return m_pGenPo->getMsgStr(); |