don't hold vector as pointer here ...

and make impl types private

Change-Id: I7ead64bac38da3d813e0b47fa124970937c36617
Reviewed-on: https://gerrit.libreoffice.org/31438
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

1 files changed, 8 insertions, 10 deletions

diff --git a/include/svl/poolcach.hxx b/include/svl/poolcach.hxx
index 6c55fae31b2c..cfb148d30bd3 100644
--- a/include/svl/poolcach.hxx
+++ b/include/svl/poolcach.hxx
@@ -27,19 +27,17 @@ class SfxItemSet;
 class SfxPoolItem;
 class SfxSetItem;
 
-struct SfxItemModifyImpl
-{
-    const SfxSetItem  *pOrigItem;
-    SfxSetItem        *pPoolItem;
-};
-
-typedef std::vector<SfxItemModifyImpl> SfxItemModifyArr_Impl;
-
-
 class SVL_DLLPUBLIC SfxItemPoolCache
 {
+    struct SfxItemModifyImpl
+    {
+        const SfxSetItem  *pOrigItem;
+        SfxSetItem        *pPoolItem;
+    };
+    typedef std::vector<SfxItemModifyImpl> SfxItemModifyArr_Impl;
+
     SfxItemPool             *pPool;
-    SfxItemModifyArr_Impl   *pCache;
+    SfxItemModifyArr_Impl   m_aCache;
     const SfxItemSet        *pSetToPut;
     const SfxPoolItem       *pItemToPut;