Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2828)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: xfa/src/fxfa/src/parser/xfa_object_imp.cpp

Issue 1730713002: Moar _CAPS work, part 3. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« xfa/src/fxfa/src/app/xfa_textlayout.h ('K') | « xfa/src/fxfa/src/common/xfa_script.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/src/fxfa/src/parser/xfa_object_imp.cpp
diff --git a/xfa/src/fxfa/src/parser/xfa_object_imp.cpp b/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
index a4fd017c528a42b7f6606455f583ed1660f3004e..c228575105626ec3ca5a0dbf0c196ba31b3f963e 100644
--- a/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
+++ b/xfa/src/fxfa/src/parser/xfa_object_imp.cpp
@@ -5010,14 +5010,14 @@ FX_BOOL CXFA_Node::IsNeedSavingXMLNode() {
return m_pXMLNode && (GetPacketID() == XFA_XDPPACKET_Datasets ||
GetClassID() == XFA_ELEMENT_Xfa);
}
-XFA_LPMAPMODULEDATA CXFA_Node::GetMapModuleData(FX_BOOL bCreateNew) {
+XFA_MAPMODULEDATA* CXFA_Node::GetMapModuleData(FX_BOOL bCreateNew) {
if (!m_pMapModuleData && bCreateNew) {
m_pMapModuleData = new XFA_MAPMODULEDATA;
}
return m_pMapModuleData;
}
void CXFA_Node::SetMapModuleValue(void* pKey, void* pValue) {
- XFA_LPMAPMODULEDATA pMoudle = this->GetMapModuleData(TRUE);
+ XFA_MAPMODULEDATA* pMoudle = this->GetMapModuleData(TRUE);
if (!pMoudle) {
return;
}
@@ -5026,7 +5026,7 @@ void CXFA_Node::SetMapModuleValue(void* pKey, void* pValue) {
FX_BOOL CXFA_Node::GetMapModuleValue(void* pKey, void*& pValue) {
CXFA_Node* pNode = this;
while (pNode) {
- XFA_LPMAPMODULEDATA pMoudle = pNode->GetMapModuleData(FALSE);
+ XFA_MAPMODULEDATA* pMoudle = pNode->GetMapModuleData(FALSE);
if (pMoudle && pMoudle->m_ValueMap.Lookup(pKey, pValue)) {
return TRUE;
}
@@ -5054,20 +5054,20 @@ void CXFA_Node::SetMapModuleBuffer(
void* pValue,
int32_t iBytes,
XFA_MAPDATABLOCKCALLBACKINFO* pCallbackInfo) {
- XFA_LPMAPMODULEDATA pMoudle = this->GetMapModuleData(TRUE);
+ XFA_MAPMODULEDATA* pMoudle = this->GetMapModuleData(TRUE);
if (!pMoudle) {
return;
}
- XFA_LPMAPDATABLOCK& pBuffer = pMoudle->m_BufferMap[pKey];
+ XFA_MAPDATABLOCK*& pBuffer = pMoudle->m_BufferMap[pKey];
if (pBuffer == NULL) {
- pBuffer = (XFA_LPMAPDATABLOCK)FX_Alloc(uint8_t,
- sizeof(XFA_MAPDATABLOCK) + iBytes);
+ pBuffer =
+ (XFA_MAPDATABLOCK*)FX_Alloc(uint8_t, sizeof(XFA_MAPDATABLOCK) + iBytes);
} else if (pBuffer->iBytes != iBytes) {
if (pBuffer->pCallbackInfo && pBuffer->pCallbackInfo->pFree) {
pBuffer->pCallbackInfo->pFree(*(void**)pBuffer->GetData());
}
- pBuffer = (XFA_LPMAPDATABLOCK)FX_Realloc(uint8_t, pBuffer,
- sizeof(XFA_MAPDATABLOCK) + iBytes);
+ pBuffer = (XFA_MAPDATABLOCK*)FX_Realloc(uint8_t, pBuffer,
+ sizeof(XFA_MAPDATABLOCK) + iBytes);
} else if (pBuffer->pCallbackInfo && pBuffer->pCallbackInfo->pFree) {
pBuffer->pCallbackInfo->pFree(*(void**)pBuffer->GetData());
}
@@ -5082,10 +5082,10 @@ FX_BOOL CXFA_Node::GetMapModuleBuffer(void* pKey,
void*& pValue,
int32_t& iBytes,
FX_BOOL bProtoAlso) {
- XFA_LPMAPDATABLOCK pBuffer = NULL;
+ XFA_MAPDATABLOCK* pBuffer = NULL;
CXFA_Node* pNode = this;
while (pNode) {
- XFA_LPMAPMODULEDATA pMoudle = pNode->GetMapModuleData(FALSE);
+ XFA_MAPMODULEDATA* pMoudle = pNode->GetMapModuleData(FALSE);
if (pMoudle && pMoudle->m_BufferMap.Lookup(pKey, pBuffer)) {
break;
}
@@ -5104,10 +5104,10 @@ FX_BOOL CXFA_Node::HasMapModuleKey(void* pKey, FX_BOOL bProtoAlso) {
CXFA_Node* pNode = this;
while (pNode) {
void* pVal;
- XFA_LPMAPMODULEDATA pMoudle = pNode->GetMapModuleData(FALSE);
+ XFA_MAPMODULEDATA* pMoudle = pNode->GetMapModuleData(FALSE);
if (pMoudle &&
(pMoudle->m_ValueMap.Lookup(pKey, pVal) ||
- pMoudle->m_BufferMap.Lookup(pKey, (XFA_LPMAPDATABLOCK&)pVal))) {
+ pMoudle->m_BufferMap.Lookup(pKey, (XFA_MAPDATABLOCK*&)pVal))) {
return TRUE;
}
pNode = (bProtoAlso && pNode->GetPacketID() != XFA_XDPPACKET_Datasets)
@@ -5117,12 +5117,12 @@ FX_BOOL CXFA_Node::HasMapModuleKey(void* pKey, FX_BOOL bProtoAlso) {
return FALSE;
}
void CXFA_Node::RemoveMapModuleKey(void* pKey) {
- XFA_LPMAPMODULEDATA pMoudle = this->GetMapModuleData(FALSE);
+ XFA_MAPMODULEDATA* pMoudle = this->GetMapModuleData(FALSE);
if (!pMoudle) {
return;
}
if (pKey) {
- XFA_LPMAPDATABLOCK pBuffer = NULL;
+ XFA_MAPDATABLOCK* pBuffer = NULL;
pMoudle->m_BufferMap.Lookup(pKey, pBuffer);
if (pBuffer) {
if (pBuffer->pCallbackInfo && pBuffer->pCallbackInfo->pFree) {
@@ -5133,7 +5133,7 @@ void CXFA_Node::RemoveMapModuleKey(void* pKey) {
pMoudle->m_BufferMap.RemoveKey(pKey);
pMoudle->m_ValueMap.RemoveKey(pKey);
} else {
- XFA_LPMAPDATABLOCK pBuffer;
+ XFA_MAPDATABLOCK* pBuffer;
FX_POSITION posBuffer = pMoudle->m_BufferMap.GetStartPosition();
while (posBuffer) {
pMoudle->m_BufferMap.GetNextAssoc(posBuffer, pKey, pBuffer);
@@ -5153,12 +5153,12 @@ void CXFA_Node::RemoveMapModuleKey(void* pKey) {
}
}
void CXFA_Node::MergeAllData(void* pDstModule, FX_BOOL bUseSrcAttr) {
- XFA_LPMAPMODULEDATA pDstModuleData =
+ XFA_MAPMODULEDATA* pDstModuleData =
static_cast<CXFA_Node*>(pDstModule)->GetMapModuleData(TRUE);
if (!pDstModuleData) {
return;
}
- XFA_LPMAPMODULEDATA pSrcModuleData = this->GetMapModuleData(FALSE);
+ XFA_MAPMODULEDATA* pSrcModuleData = this->GetMapModuleData(FALSE);
if (!pSrcModuleData) {
return;
}
@@ -5174,9 +5174,9 @@ void CXFA_Node::MergeAllData(void* pDstModule, FX_BOOL bUseSrcAttr) {
FX_POSITION psBuffer = pSrcModuleData->m_BufferMap.GetStartPosition();
while (psBuffer) {
void* pKey;
- XFA_LPMAPDATABLOCK pSrcBuffer;
+ XFA_MAPDATABLOCK* pSrcBuffer;
pSrcModuleData->m_BufferMap.GetNextAssoc(psBuffer, pKey, pSrcBuffer);
- XFA_LPMAPDATABLOCK& pBuffer = pDstModuleData->m_BufferMap[pKey];
+ XFA_MAPDATABLOCK*& pBuffer = pDstModuleData->m_BufferMap[pKey];
if (pBuffer && !bUseSrcAttr) {
continue;
}
@@ -5189,13 +5189,13 @@ void CXFA_Node::MergeAllData(void* pDstModule, FX_BOOL bUseSrcAttr) {
continue;
}
if (pBuffer == NULL) {
- pBuffer = (XFA_LPMAPDATABLOCK)FX_Alloc(
+ pBuffer = (XFA_MAPDATABLOCK*)FX_Alloc(
uint8_t, sizeof(XFA_MAPDATABLOCK) + pSrcBuffer->iBytes);
} else if (pBuffer->iBytes != pSrcBuffer->iBytes) {
if (pBuffer->pCallbackInfo && pBuffer->pCallbackInfo->pFree) {
pBuffer->pCallbackInfo->pFree(*(void**)pBuffer->GetData());
}
- pBuffer = (XFA_LPMAPDATABLOCK)FX_Realloc(
+ pBuffer = (XFA_MAPDATABLOCK*)FX_Realloc(
uint8_t, pBuffer, sizeof(XFA_MAPDATABLOCK) + pSrcBuffer->iBytes);
} else if (pBuffer->pCallbackInfo && pBuffer->pCallbackInfo->pFree) {
pBuffer->pCallbackInfo->pFree(*(void**)pBuffer->GetData());
@@ -5222,8 +5222,8 @@ void CXFA_Node::MoveBufferMapData(CXFA_Node* pDstModule, void* pKey) {
if (pDstModule->GetClassID() != this->GetClassID()) {
bNeedMove = FALSE;
}
- XFA_LPMAPMODULEDATA pSrcModuleData = NULL;
- XFA_LPMAPMODULEDATA pDstModuleData = NULL;
+ XFA_MAPMODULEDATA* pSrcModuleData = NULL;
+ XFA_MAPMODULEDATA* pDstModuleData = NULL;
if (bNeedMove) {
pSrcModuleData = this->GetMapModuleData(FALSE);
if (!pSrcModuleData) {
@@ -5240,7 +5240,7 @@ void CXFA_Node::MoveBufferMapData(CXFA_Node* pDstModule, void* pKey) {
pSrcModuleData->m_BufferMap.RemoveKey(pKey);
pDstModuleData->m_BufferMap.RemoveKey(pKey);
pDstModuleData->m_BufferMap.SetAt(pKey,
- (XFA_LPMAPDATABLOCK)pBufferBlockData);
+ (XFA_MAPDATABLOCK*)pBufferBlockData);
}
}
if (pDstModule->GetObjectType() == XFA_OBJECTTYPE_NodeV) {
« xfa/src/fxfa/src/app/xfa_textlayout.h ('K') | « xfa/src/fxfa/src/common/xfa_script.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698