tdf#96339 fix bug in sort list

this bug was caused by commit
2aacf6c2cd82322b953988ff30d3bc997ae76d7b
"sc: boost::ptr_vector->std::vector"

Since the code in question likes passing around a pointer
to the element of the vector, convert to using
std::vector<std::unique_ptr>

Change-Id: I9e9676fe7c2dc32e23ba6708aaea1f16c1bf2ff8

1 files changed, 4 insertions, 3 deletions

diff --git a/sc/inc/userlist.hxx b/sc/inc/userlist.hxx
index 1933b8455222..b103aeec97d0 100644
--- a/sc/inc/userlist.hxx
+++ b/sc/inc/userlist.hxx
@@ -22,6 +22,7 @@
 
 #include "scdllapi.h"
 
+#include <memory>
 #include <vector>
 
 /**
@@ -46,6 +47,7 @@ private:
 public:
     ScUserListData(const OUString& rStr);
     ScUserListData(const ScUserListData& rData);
+    ~ScUserListData();
 
     const OUString& GetString() const { return aStr; }
     void SetString(const OUString& rStr);
@@ -61,7 +63,7 @@ public:
  */
 class SC_DLLPUBLIC ScUserList
 {
-    typedef std::vector<ScUserListData> DataType;
+    typedef std::vector< std::unique_ptr<ScUserListData> > DataType;
     DataType maData;
 public:
     typedef DataType::iterator iterator;
@@ -83,9 +85,8 @@ public:
     iterator begin();
     const_iterator begin() const;
     void clear();
-    void reserve(size_t nSize) { maData.reserve(nSize); }
     size_t size() const;
-    void push_back(const ScUserListData& r) { maData.push_back(r); }
+    void push_back(ScUserListData* p);
     void erase(iterator itr);
 };