mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

loplugin:inlinefields in ucb_impl::RegexpMap

Browse Source 
Change-Id: Id8cf4a9e7f85048dac37fc1256427214abe3ac8c
Reviewed-on: https://gerrit.libreoffice.org/36265
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Noel Grandin
2017-04-07 15:05:26 +02:00
parent 340d58f223
commit c6dc4ba419
1 changed files with 27 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

56
ucb/source/inc/regexpmap.hxx
View File

@@ -386,29 +386,27 @@ public:
Val const * map(OUString const & rString) const; Val const * map(OUString const & rString) const;
private: private:
RegexpMapImpl< Val > * m_pImpl; RegexpMapImpl< Val > m_aImpl;
}; };
template< typename Val > template< typename Val >
RegexpMap< Val >::RegexpMap(): RegexpMap< Val >::RegexpMap()
m_pImpl(new RegexpMapImpl< Val >)
{} {}
template< typename Val > template< typename Val >
RegexpMap< Val >::RegexpMap(RegexpMap const & rOther): RegexpMap< Val >::RegexpMap(RegexpMap const & rOther):
m_pImpl(new RegexpMapImpl< Val >(*rOther.m_pImpl)) m_aImpl(rOther.m_pImpl)
{} {}
template< typename Val > template< typename Val >
RegexpMap< Val >::~RegexpMap() RegexpMap< Val >::~RegexpMap()
{ {
delete m_pImpl;
} }
template< typename Val > template< typename Val >
RegexpMap< Val > & RegexpMap< Val >::operator =(RegexpMap const & rOther) RegexpMap< Val > & RegexpMap< Val >::operator =(RegexpMap const & rOther)
{ {
*m_pImpl = *rOther.m_pImpl; m_aImpl = rOther.m_aImpl;
return *this; return *this;
} }
@@ -419,15 +417,15 @@ void RegexpMap< Val >::add(rtl::OUString const & rKey, Val const & rValue)
if (aRegexp.isDefault()) if (aRegexp.isDefault())
{ {
if (m_pImpl->m_pDefault) if (m_aImpl.m_pDefault)
{ {
return; return;
} }
m_pImpl->m_pDefault.reset( new Entry< Val >(aRegexp, rValue) ); m_aImpl.m_pDefault.reset( new Entry< Val >(aRegexp, rValue) );
} }
else else
{ {
List< Val > & rTheList = m_pImpl->m_aList[aRegexp.getKind()]; List< Val > & rTheList = m_aImpl.m_aList[aRegexp.getKind()];
typename List< Val >::iterator aEnd(rTheList.end()); typename List< Val >::iterator aEnd(rTheList.end());
for (typename List< Val >::iterator aIt(rTheList.begin()); aIt != aEnd; ++aIt) for (typename List< Val >::iterator aIt(rTheList.begin()); aIt != aEnd; ++aIt)
@@ -449,36 +447,36 @@ typename RegexpMap< Val >::iterator RegexpMap< Val >::find(rtl::OUString const &
if (aRegexp.isDefault()) if (aRegexp.isDefault())
{ {
if (m_pImpl->m_pDefault) if (m_aImpl.m_pDefault)
return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl,
true)); true));
} }
else else
{ {
List< Val > & rTheList = m_pImpl->m_aList[aRegexp.getKind()]; List< Val > & rTheList = m_aImpl.m_aList[aRegexp.getKind()];
typename List< Val > ::iterator aEnd(rTheList.end()); typename List< Val > ::iterator aEnd(rTheList.end());
for (typename List< Val >::iterator aIt(rTheList.begin()); aIt != aEnd; ++aIt) for (typename List< Val >::iterator aIt(rTheList.begin()); aIt != aEnd; ++aIt)
if (aIt->m_aRegexp == aRegexp) if (aIt->m_aRegexp == aRegexp)
return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >( return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(
m_pImpl, &m_aImpl,
aRegexp.getKind(), aIt)); aRegexp.getKind(), aIt));
} }
return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, false)); return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl, false));
} }
template< typename Val > template< typename Val >
void RegexpMap< Val >::erase(iterator const & rPos) void RegexpMap< Val >::erase(iterator const & rPos)
{ {
if (rPos.m_pImpl->getMap() == m_pImpl) if (rPos.m_pImpl->getMap() == &m_aImpl)
{ {
if (rPos.m_pImpl->getList() == -1) if (rPos.m_pImpl->getList() == -1)
{ {
m_pImpl->m_pDefault.reset(); m_aImpl.m_pDefault.reset();
} }
else else
m_pImpl->m_aList[rPos.m_pImpl->getList()]. m_aImpl.m_aList[rPos.m_pImpl->getList()].
erase(rPos.m_pImpl->getIndex()); erase(rPos.m_pImpl->getIndex());
} }
} }
@@ -486,36 +484,36 @@ void RegexpMap< Val >::erase(iterator const & rPos)
template< typename Val > template< typename Val >
typename RegexpMap< Val >::iterator RegexpMap< Val >::begin() typename RegexpMap< Val >::iterator RegexpMap< Val >::begin()
{ {
return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, true)); return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl, true));
} }
template< typename Val > template< typename Val >
typename RegexpMap< Val >::const_iterator RegexpMap< Val >::begin() const typename RegexpMap< Val >::const_iterator RegexpMap< Val >::begin() const
{ {
return RegexpMapConstIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, return RegexpMapConstIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl,
true)); true));
} }
template< typename Val > template< typename Val >
typename RegexpMap< Val >::iterator RegexpMap< Val >::end() typename RegexpMap< Val >::iterator RegexpMap< Val >::end()
{ {
return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, false)); return RegexpMapIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl, false));
} }
template< typename Val > template< typename Val >
typename RegexpMap< Val >::const_iterator RegexpMap< Val >::end() const typename RegexpMap< Val >::const_iterator RegexpMap< Val >::end() const
{ {
return RegexpMapConstIter< Val >(new RegexpMapIterImpl< Val >(m_pImpl, return RegexpMapConstIter< Val >(new RegexpMapIterImpl< Val >(&m_aImpl,
false)); false));
} }
template< typename Val > template< typename Val >
typename RegexpMap< Val >::size_type RegexpMap< Val >::size() const typename RegexpMap< Val >::size_type RegexpMap< Val >::size() const
{ {
return (m_pImpl->m_pDefault ? 1 : 0) return (m_aImpl.m_pDefault ? 1 : 0)
+ m_pImpl->m_aList[Regexp::KIND_PREFIX].size() + m_aImpl.m_aList[Regexp::KIND_PREFIX].size()
+ m_pImpl->m_aList[Regexp::KIND_AUTHORITY].size() + m_aImpl.m_aList[Regexp::KIND_AUTHORITY].size()
+ m_pImpl->m_aList[Regexp::KIND_DOMAIN].size(); + m_aImpl.m_aList[Regexp::KIND_DOMAIN].size();
} }
template< typename Val > template< typename Val >
@@ -523,7 +521,7 @@ Val const * RegexpMap< Val >::map(rtl::OUString const & rString) const
{ {
for (int n = Regexp::KIND_DOMAIN; n >= Regexp::KIND_PREFIX; --n) for (int n = Regexp::KIND_DOMAIN; n >= Regexp::KIND_PREFIX; --n)
{ {
List< Val > const & rTheList = m_pImpl->m_aList[n]; List< Val > const & rTheList = m_aImpl.m_aList[n];
typename List< Val >::const_iterator aEnd(rTheList.end()); typename List< Val >::const_iterator aEnd(rTheList.end());
for (typename List< Val >::const_iterator aIt(rTheList.begin()); aIt != aEnd; for (typename List< Val >::const_iterator aIt(rTheList.begin()); aIt != aEnd;
@@ -531,9 +529,9 @@ Val const * RegexpMap< Val >::map(rtl::OUString const & rString) const
if (aIt->m_aRegexp.matches(rString)) if (aIt->m_aRegexp.matches(rString))
return &aIt->m_aValue; return &aIt->m_aValue;
} }
if (m_pImpl->m_pDefault if (m_aImpl.m_pDefault
&& m_pImpl->m_pDefault->m_aRegexp.matches(rString)) && m_aImpl.m_pDefault->m_aRegexp.matches(rString))
return &m_pImpl->m_pDefault->m_aValue; return &m_aImpl.m_pDefault->m_aValue;
return 0; return 0;
} }