diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-02 19:40:58 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-02 19:40:58 +0000 |
| commit | eda1761e6a90058a97cee1bcd763a2447eb6a67b (patch) | |
| tree | e2ad7c55d46caed4c6338de4794ba5c00b64c21f /fxjs | |
| parent | a6adac83c6aadbacf2a338d2e55148f7d3417762 (diff) | |
| download | pdfium-eda1761e6a90058a97cee1bcd763a2447eb6a67b.tar.xz | |
Remove default values from Set{UserData|Object} and GetUserData.
This CL removes the default values and inlines at the call sites.
Change-Id: I47ff04a01c30af89949e9c8dd5d51f7ffaef1f98
Reviewed-on: https://pdfium-review.googlesource.com/17610
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs')
| -rw-r--r-- | fxjs/cjx_node.cpp | 2 | ||||
| -rw-r--r-- | fxjs/cjx_node.h | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp index b4f480e537..b4d99dd38e 100644 --- a/fxjs/cjx_node.cpp +++ b/fxjs/cjx_node.cpp @@ -3236,7 +3236,7 @@ void* CJX_Node::GetObject(XFA_ATTRIBUTE eAttr) { bool CJX_Node::TryObject(XFA_ATTRIBUTE eAttr, void*& pData) { void* pKey = GetMapKey_Element(GetXFANode()->GetElementType(), eAttr); - pData = GetUserData(pKey); + pData = GetUserData(pKey, false); return !!pData; } diff --git a/fxjs/cjx_node.h b/fxjs/cjx_node.h index a492a32f39..1434029def 100644 --- a/fxjs/cjx_node.h +++ b/fxjs/cjx_node.h @@ -112,13 +112,13 @@ class CJX_Node : public CJX_Object { bool SetUserData(void* pKey, void* pData, - XFA_MAPDATABLOCKCALLBACKINFO* pCallbackInfo = nullptr); - void* GetUserData(void* pKey, bool bProtoAlso = false); + XFA_MAPDATABLOCKCALLBACKINFO* pCallbackInfo); + void* GetUserData(void* pKey, bool bProtoAlso); bool TryObject(XFA_ATTRIBUTE eAttr, void*& pData); bool SetObject(XFA_ATTRIBUTE eAttr, void* pData, - XFA_MAPDATABLOCKCALLBACKINFO* pCallbackInfo = nullptr); + XFA_MAPDATABLOCKCALLBACKINFO* pCallbackInfo); void* GetObject(XFA_ATTRIBUTE eAttr); bool TryNamespace(WideString& wsNamespace); |