diff options
author | Julien Nabet <serval2412@yahoo.fr> | 2013-05-12 13:45:03 +0200 |
---|---|---|
committer | Julien Nabet <serval2412@yahoo.fr> | 2013-05-12 13:45:33 +0200 |
commit | 86f4856c78cf31fcaa0f0fc61b3795dc75859972 (patch) | |
tree | 3ced0c6d1b920f56ad4f131bc9197d911951b207 /filter | |
parent | 180b1e1e18d6cc2b2ab35979ea53982ee0b58396 (diff) |
Quiet cppcheck + simplify/optimize a bit
Change-Id: Ibb89a7e5b2889ae996684bf4d7664d1fd0cea8ba
Diffstat (limited to 'filter')
-rw-r--r-- | filter/source/config/cache/filterfactory.cxx | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/filter/source/config/cache/filterfactory.cxx b/filter/source/config/cache/filterfactory.cxx index ac0d5ad827cc..2193300030e5 100644 --- a/filter/source/config/cache/filterfactory.cxx +++ b/filter/source/config/cache/filterfactory.cxx @@ -518,28 +518,28 @@ OUStringList FilterFactory::impl_getSortedFilterListForModule(const OUString& sM // merge both lists together OUStringList lMergedFilters = lSortedFilters; - OUStringList::iterator pIt2; - OUStringList::iterator pIt3; - for ( pIt2 = lOtherFilters.begin(); - pIt2 != lOtherFilters.end() ; - ++pIt2 ) + const OUStringList::const_iterator itlOtherFiltersEnd = lOtherFilters.end(); + const OUStringList::const_iterator itlSortedFiltersEnd = lSortedFilters.end(); + for (OUStringList::const_iterator pIt = lOtherFilters.begin(); + pIt != itlOtherFiltersEnd ; + ++pIt ) { - const OUString& rFilter = *pIt2; - pIt3 = ::std::find(lSortedFilters.begin(), lSortedFilters.end(), rFilter); - if (pIt3 == lSortedFilters.end()) - lMergedFilters.push_back(rFilter); + if (::std::find(lSortedFilters.begin(), lSortedFilters.end(), *pIt) == itlSortedFiltersEnd) + lMergedFilters.push_back(*pIt); } + OUStringList::iterator pItToErase; + // remove all filters from this merged list, which does not fit the flag specification if (nIFlags != -1) { - pIt2 = ::std::remove_if(lMergedFilters.begin(), lMergedFilters.end(), stlcomp_removeIfMatchFlags(pCache, nIFlags, sal_True)); - lMergedFilters.erase(pIt2, lMergedFilters.end()); + pItToErase = ::std::remove_if(lMergedFilters.begin(), lMergedFilters.end(), stlcomp_removeIfMatchFlags(pCache, nIFlags, sal_True)); + lMergedFilters.erase(pItToErase, lMergedFilters.end()); } if (nEFlags != -1) { - pIt2 = ::std::remove_if(lMergedFilters.begin(), lMergedFilters.end(), stlcomp_removeIfMatchFlags(pCache, nEFlags, sal_False)); - lMergedFilters.erase(pIt2, lMergedFilters.end()); + pItToErase = ::std::remove_if(lMergedFilters.begin(), lMergedFilters.end(), stlcomp_removeIfMatchFlags(pCache, nEFlags, sal_False)); + lMergedFilters.erase(pItToErase, lMergedFilters.end()); } // sort the default filter to the front of this list |