summaryrefslogtreecommitdiff
path: root/sc/source/core/data/dpdimsave.cxx
diff options
context:
space:
mode:
authorTakeshi Abe <tabe@fixedpoint.jp>2018-12-13 10:46:28 +0900
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-12-13 07:32:21 +0100
commit320a77a3145b4eb9092c4379d4b5b999bd14901f (patch)
treea245ceb0639d7476a7e4189be201b41781e19512 /sc/source/core/data/dpdimsave.cxx
parente98bcfcc3cdad46620e3d59119b0ac262db88054 (diff)
sc: Use std::find() to simplify code
Change-Id: I903a59591cd204556e873429280aac9cf8d5325a Reviewed-on: https://gerrit.libreoffice.org/65067 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/core/data/dpdimsave.cxx')
-rw-r--r--sc/source/core/data/dpdimsave.cxx13
1 files changed, 6 insertions, 7 deletions
diff --git a/sc/source/core/data/dpdimsave.cxx b/sc/source/core/data/dpdimsave.cxx
index 2319294b49c5..f2f87aa668e1 100644
--- a/sc/source/core/data/dpdimsave.cxx
+++ b/sc/source/core/data/dpdimsave.cxx
@@ -57,13 +57,12 @@ void ScDPSaveGroupItem::AddElementsFromGroup( const ScDPSaveGroupItem& rGroup )
bool ScDPSaveGroupItem::RemoveElement( const OUString& rName )
{
- for (std::vector<OUString>::iterator aIter = aElements.begin(); aIter != aElements.end(); ++aIter)
- if (*aIter == rName) //TODO: ignore case
- {
- aElements.erase(aIter); // found -> remove
- return true; // don't have to look further
- }
-
+ auto it = std::find(aElements.begin(), aElements.end(), rName); //TODO: ignore case
+ if (it != aElements.end())
+ {
+ aElements.erase(it);
+ return true;
+ }
return false; // not found
}