diff options
author | Takeshi Abe <tabe@fixedpoint.jp> | 2018-12-13 10:46:28 +0900 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-12-13 07:32:21 +0100 |
commit | 320a77a3145b4eb9092c4379d4b5b999bd14901f (patch) | |
tree | a245ceb0639d7476a7e4189be201b41781e19512 /sc/source/core/data/dpdimsave.cxx | |
parent | e98bcfcc3cdad46620e3d59119b0ac262db88054 (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.cxx | 13 |
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 } |