sc/source/core/data/dpsave.cxx | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
New commits:
commit a9504d55782fb6ce1e64d154b4a3197062277425
Author: Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk>
AuthorDate: Thu Apr 11 16:27:21 2024 +0900
Commit: Tomaž Vajngerl <qui...@gmail.com>
CommitDate: Fri Apr 12 02:28:12 2024 +0200
pivot: remove duplication in ScDPSaveDimension::GetMemberByName
Change-Id: I036e83307615a8c1f875ffc66a54027e1ec31a92
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165991
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <qui...@gmail.com>
diff --git a/sc/source/core/data/dpsave.cxx b/sc/source/core/data/dpsave.cxx
index ed888eebd737..6e1d5c56f085 100644
--- a/sc/source/core/data/dpsave.cxx
+++ b/sc/source/core/data/dpsave.cxx
@@ -458,14 +458,14 @@ ScDPSaveMember*
ScDPSaveDimension::GetExistingMemberByName(const OUString& rName
ScDPSaveMember* ScDPSaveDimension::GetMemberByName(const OUString& rName)
{
- auto res = maMemberHash.find (rName);
- if (res != maMemberHash.end())
- return res->second.get();
-
- ScDPSaveMember* pNew = new ScDPSaveMember( rName );
- maMemberHash[rName] = std::unique_ptr<ScDPSaveMember>(pNew);
- maMemberList.push_back( pNew );
- return pNew;
+ ScDPSaveMember* pResult = GetExistingMemberByName(rName);
+ if (pResult)
+ return pResult;
+
+ pResult = new ScDPSaveMember(rName);
+ maMemberHash[rName] = std::unique_ptr<ScDPSaveMember>(pResult);
+ maMemberList.push_back(pResult);
+ return pResult;
}
void ScDPSaveDimension::SetMemberPosition( const OUString& rName, sal_Int32
nNewPos )
