diff options
author | dsinclair <dsinclair@chromium.org> | 2016-05-18 13:16:12 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-05-18 13:16:12 -0700 |
commit | cbfef5772c52fbd3378905a61fc9432da1515433 (patch) | |
tree | be4c25e41d2467609612232a92e598ccd9baa7eb /xfa/fxfa/parser/xfa_script_imp.cpp | |
parent | 1763f62972a3ebf080f645899a8de79b00dbfb23 (diff) | |
download | pdfium-cbfef5772c52fbd3378905a61fc9432da1515433.tar.xz |
Cleanup XFA_HASHCODE usage.
This CL cleans up several uses of XFA_HASHCODE. The defines have been converted
into an enum.
For the SavePackage call the type is now used as the param instead of a string.
The callers pass in the correct type instead of doing an internal conversion.
The GetXFAObject accepting a string was removed as it was unused. The other
variant was changed to accept the XFA_HashCode type instead of uint32_t.
GetPackageData was removed as it is unused.
Review-Url: https://codereview.chromium.org/1989313002
Diffstat (limited to 'xfa/fxfa/parser/xfa_script_imp.cpp')
-rw-r--r-- | xfa/fxfa/parser/xfa_script_imp.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xfa/fxfa/parser/xfa_script_imp.cpp b/xfa/fxfa/parser/xfa_script_imp.cpp index 893c5b79ea..b218121165 100644 --- a/xfa/fxfa/parser/xfa_script_imp.cpp +++ b/xfa/fxfa/parser/xfa_script_imp.cpp @@ -174,7 +174,8 @@ void CXFA_ScriptContext::GlobalPropertyGetter(FXJSE_HOBJECT hObject, XFA_FM2JS_GlobalPropertyGetter(lpScriptContext->m_hFM2JSContext, hValue); return; } - uint32_t uHashCode = FX_HashCode_GetW(wsPropName.AsStringC(), false); + XFA_HashCode uHashCode = static_cast<XFA_HashCode>( + FX_HashCode_GetW(wsPropName.AsStringC(), false)); if (uHashCode != XFA_HASHCODE_Layout) { CXFA_Object* pObject = lpScriptContext->GetDocument()->GetXFAObject(uHashCode); |