Replace void* pointers with CFX_BaseMassArrayImp* pointers.
authorTom Sepez <tsepez@chromium.org>
Thu, 4 Dec 2014 17:54:14 +0000 (09:54 -0800)
committerTom Sepez <tsepez@chromium.org>
Thu, 4 Dec 2014 17:54:14 +0000 (09:54 -0800)
Using an incomplete type here avoids casting later on, and allows the
debugger to more easily dig through these data structures.

R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/766183009

xfa/src/fgas/include/fx_utl.h
xfa/src/fgas/src/crt/fx_utils.cpp

index 3bb7b61..354d4bf 100644 (file)
@@ -11,6 +11,7 @@ class CFX_BaseArray;
 template<class baseType> class CFX_BaseArrayTemplate;\r
 template<class baseType> class CFX_ObjectBaseArrayTemplate;\r
 class CFX_BaseMassArray;\r
+class CFX_BaseMassArrayImp;\r
 template<class baseType> class CFX_MassArrayTemplate;\r
 template<class baseType> class CFX_ObjectMassArrayTemplate;\r
 class CFX_BaseDiscreteArray;\r
@@ -233,7 +234,7 @@ protected:
     FX_INT32   Copy(const CFX_BaseMassArray &src, FX_INT32 iStart = 0, FX_INT32 iCount = -1);\r
     FX_INT32   RemoveLast(FX_INT32 iCount = -1);\r
     void               RemoveAll(FX_BOOL bLeaveMemory = FALSE);\r
-    FX_LPVOID  m_pData;\r
+    CFX_BaseMassArrayImp* m_pData;\r
 };\r
 template<class baseType>\r
 class CFX_MassArrayTemplate : public CFX_BaseMassArray\r
@@ -431,7 +432,7 @@ protected:
     FX_INT32   GetSize() const;\r
     FX_LPBYTE  GetAt(FX_INT32 index) const;\r
     void               RemoveAll(FX_BOOL bLeaveMemory = FALSE);\r
-    FX_LPVOID  m_pData;\r
+    CFX_BaseMassArrayImp* m_pData;\r
 };\r
 template<class baseType>\r
 class CFX_StackTemplate : public CFX_BaseStack\r
index f13af23..0ee8152 100644 (file)
@@ -310,35 +310,35 @@ CFX_BaseMassArray::CFX_BaseMassArray(FX_INT32 iChunkSize, FX_INT32 iBlockSize)
 }\r
 CFX_BaseMassArray::~CFX_BaseMassArray()\r
 {\r
-    FXTARGET_Delete (CFX_BaseMassArrayImp*)m_pData;\r
+    FXTARGET_Delete m_pData;\r
 }\r
 FX_INT32 CFX_BaseMassArray::GetSize() const\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->m_iBlockCount;\r
+    return m_pData->m_iBlockCount;\r
 }\r
 FX_LPBYTE CFX_BaseMassArray::AddSpaceTo(FX_INT32 index)\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->AddSpaceTo(index);\r
+    return m_pData->AddSpaceTo(index);\r
 }\r
 FX_LPBYTE CFX_BaseMassArray::GetAt(FX_INT32 index) const\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->GetAt(index);\r
+    return m_pData->GetAt(index);\r
 }\r
 FX_INT32 CFX_BaseMassArray::Append(const CFX_BaseMassArray &src, FX_INT32 iStart, FX_INT32 iCount)\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->Append(*(CFX_BaseMassArrayImp*)src.m_pData, iStart, iCount);\r
+    return m_pData->Append(*(CFX_BaseMassArrayImp*)src.m_pData, iStart, iCount);\r
 }\r
 FX_INT32 CFX_BaseMassArray::Copy(const CFX_BaseMassArray &src, FX_INT32 iStart, FX_INT32 iCount)\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->Copy(*(CFX_BaseMassArrayImp*)src.m_pData, iStart, iCount);\r
+    return m_pData->Copy(*(CFX_BaseMassArrayImp*)src.m_pData, iStart, iCount);\r
 }\r
 FX_INT32 CFX_BaseMassArray::RemoveLast(FX_INT32 iCount)\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->RemoveLast(iCount);\r
+    return m_pData->RemoveLast(iCount);\r
 }\r
 void CFX_BaseMassArray::RemoveAll(FX_BOOL bLeaveMemory)\r
 {\r
-    ((CFX_BaseMassArrayImp*)m_pData)->RemoveAll(bLeaveMemory);\r
+    m_pData->RemoveAll(bLeaveMemory);\r
 }\r
 typedef struct _FX_BASEDISCRETEARRAYDATA : public CFX_Object {\r
     FX_INT32           iBlockSize;\r
@@ -424,11 +424,11 @@ CFX_BaseStack::~CFX_BaseStack()
 }\r
 FX_LPBYTE CFX_BaseStack::Push()\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->AddSpace();\r
+    return m_pData->AddSpace();\r
 }\r
 void CFX_BaseStack::Pop()\r
 {\r
-    FX_INT32 &iBlockCount = ((CFX_BaseMassArrayImp*)m_pData)->m_iBlockCount;\r
+    FX_INT32 &iBlockCount = m_pData->m_iBlockCount;\r
     if (iBlockCount < 1) {\r
         return;\r
     }\r
@@ -436,21 +436,21 @@ void CFX_BaseStack::Pop()
 }\r
 FX_LPBYTE CFX_BaseStack::GetTopElement() const\r
 {\r
-    FX_INT32 iSize = ((CFX_BaseMassArrayImp*)m_pData)->m_iBlockCount;\r
+    FX_INT32 iSize = m_pData->m_iBlockCount;\r
     if (iSize < 1) {\r
         return NULL;\r
     }\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->GetAt(iSize - 1);\r
+    return m_pData->GetAt(iSize - 1);\r
 }\r
 FX_INT32 CFX_BaseStack::GetSize() const\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->m_iBlockCount;\r
+    return m_pData->m_iBlockCount;\r
 }\r
 FX_LPBYTE CFX_BaseStack::GetAt(FX_INT32 index) const\r
 {\r
-    return ((CFX_BaseMassArrayImp*)m_pData)->GetAt(index);\r
+    return m_pData->GetAt(index);\r
 }\r
 void CFX_BaseStack::RemoveAll(FX_BOOL bLeaveMemory )\r
 {\r
-    ((CFX_BaseMassArrayImp*)m_pData)->RemoveAll(bLeaveMemory);\r
+    m_pData->RemoveAll(bLeaveMemory);\r
 }\r