summaryrefslogtreecommitdiff
path: root/xfa
diff options
context:
space:
mode:
authorRyan Harrison <rharrison@chromium.org>2017-06-29 16:01:00 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-06-29 21:08:18 +0000
commitcd984752d05a58c0bd6f8ddd740d5cd4bfca20c6 (patch)
treecd5c1a6c66e1c370b3e250bb54dbf2a56a67f7b4 /xfa
parentb21f174ad87e46c823c2ef8c11682167e2d12864 (diff)
downloadpdfium-cd984752d05a58c0bd6f8ddd740d5cd4bfca20c6.tar.xz
Cleaning up SomMethod handling in fm2js
Removing hand rolled hash table and binrary search. BUG=pdfium:786 Change-Id: Iac18f02c9f560f484b974762474dbb51597d2353 Reviewed-on: https://pdfium-review.googlesource.com/7092 Reviewed-by: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org> Reviewed-by: Nicolás Peña <npm@chromium.org> Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa')
-rw-r--r--xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp79
-rw-r--r--xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h2
2 files changed, 36 insertions, 45 deletions
diff --git a/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp b/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp
index 7ab2e13bf9..8078ae9841 100644
--- a/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp
+++ b/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.cpp
@@ -7,7 +7,6 @@
#include "xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h"
#include <algorithm>
-#include <iostream>
#include <utility>
#include "core/fxcrt/fx_extension.h"
@@ -15,6 +14,7 @@
namespace {
+// Indexed by XFA_FM_SimpleExpressionType
const wchar_t* const gs_lpStrExpFuncName[] = {
L"pfm_rt.asgn_val_op", L"pfm_rt.log_or_op", L"pfm_rt.log_and_op",
L"pfm_rt.eq_op", L"pfm_rt.neq_op", L"pfm_rt.lt_op",
@@ -50,32 +50,32 @@ const wchar_t* const g_BuiltInFuncs[] = {
const FX_STRSIZE g_BuiltInFuncsMaxLen = 12;
struct XFA_FMSOMMethod {
- uint32_t m_uHash;
const wchar_t* m_wsSomMethodName;
uint32_t m_dParameters;
};
+
const XFA_FMSOMMethod gs_FMSomMethods[] = {
- {0x00000068, L"h", 0x01},
- {0x00000077, L"w", 0x01},
- {0x00000078, L"x", 0x01},
- {0x00000079, L"y", 0x01},
- {0x05eb5b0f, L"pageSpan", 0x01},
- {0x10f1b1bd, L"page", 0x01},
- {0x3bf1c2a5, L"absPageSpan", 0x01},
- {0x3c752495, L"verify", 0x0d},
- {0x44c352ad, L"formNodes", 0x01},
- {0x5775c2cc, L"absPageInBatch", 0x01},
- {0x5ee00996, L"setElement", 0x01},
- {0x7033bfd5, L"insert", 0x03},
- {0x8c5feb32, L"sheetInBatch", 0x01},
- {0x8f3a8379, L"sheet", 0x01},
- {0x92dada4f, L"saveFilteredXML", 0x01},
- {0x9cab7cae, L"remove", 0x01},
- {0xa68635f1, L"sign", 0x61},
- {0xaac241c8, L"isRecordGroup", 0x01},
- {0xd8ed1467, L"clear", 0x01},
- {0xda12e518, L"append", 0x01},
- {0xe74f0653, L"absPage", 0x01},
+ {L"absPage", 0x01},
+ {L"absPageInBatch", 0x01},
+ {L"absPageSpan", 0x01},
+ {L"append", 0x01},
+ {L"clear", 0x01},
+ {L"formNodes", 0x01},
+ {L"h", 0x01},
+ {L"insert", 0x03},
+ {L"isRecordGroup", 0x01},
+ {L"page", 0x01},
+ {L"pageSpan", 0x01},
+ {L"remove", 0x01},
+ {L"saveFilteredXML", 0x01},
+ {L"setElement", 0x01},
+ {L"sheet", 0x01},
+ {L"sheetInBatch", 0x01},
+ {L"sign", 0x61},
+ {L"verify", 0x0d},
+ {L"w", 0x01},
+ {L"x", 0x01},
+ {L"y", 0x01},
};
} // namespace
@@ -536,26 +536,17 @@ bool CXFA_FMCallExpression::IsBuiltInFunc(CFX_WideTextBuf* funcName) {
}
uint32_t CXFA_FMCallExpression::IsMethodWithObjParam(
- const CFX_WideStringC& methodName) {
- uint32_t uHash = FX_HashCode_GetW(methodName, false);
- XFA_FMSOMMethod somMethodWithObjPara;
- uint32_t parameters = 0x00;
- int32_t iStart = 0,
- iEnd = (sizeof(gs_FMSomMethods) / sizeof(gs_FMSomMethods[0])) - 1;
- int32_t iMid = (iStart + iEnd) / 2;
- do {
- iMid = (iStart + iEnd) / 2;
- somMethodWithObjPara = gs_FMSomMethods[iMid];
- if (uHash == somMethodWithObjPara.m_uHash) {
- parameters = somMethodWithObjPara.m_dParameters;
- break;
- } else if (uHash < somMethodWithObjPara.m_uHash) {
- iEnd = iMid - 1;
- } else {
- iStart = iMid + 1;
- }
- } while (iStart <= iEnd);
- return parameters;
+ const CFX_WideString& methodName) {
+ const XFA_FMSOMMethod* result = std::lower_bound(
+ std::begin(gs_FMSomMethods), std::end(gs_FMSomMethods), methodName,
+ [](const XFA_FMSOMMethod iter, const CFX_WideString& val) {
+ return val.Compare(iter.m_wsSomMethodName) > 0;
+ });
+ if (result != std::end(gs_FMSomMethods) &&
+ !methodName.Compare(result->m_wsSomMethodName)) {
+ return result->m_dParameters;
+ }
+ return 0;
}
bool CXFA_FMCallExpression::ToJavaScript(CFX_WideTextBuf& javascript) {
@@ -565,7 +556,7 @@ bool CXFA_FMCallExpression::ToJavaScript(CFX_WideTextBuf& javascript) {
if (m_bIsSomMethod) {
javascript << funcName;
javascript << L"(";
- uint32_t methodPara = IsMethodWithObjParam(funcName.AsStringC());
+ uint32_t methodPara = IsMethodWithObjParam(funcName.MakeString());
if (methodPara > 0) {
for (size_t i = 0; i < m_Arguments.size(); ++i) {
// Currently none of our expressions use objects for a parameter over
diff --git a/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h b/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h
index 6f014297a7..e47334926d 100644
--- a/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h
+++ b/xfa/fxfa/fm2js/cxfa_fmsimpleexpression.h
@@ -239,7 +239,7 @@ class CXFA_FMCallExpression : public CXFA_FMUnaryExpression {
~CXFA_FMCallExpression() override;
bool IsBuiltInFunc(CFX_WideTextBuf* funcName);
- uint32_t IsMethodWithObjParam(const CFX_WideStringC& methodName);
+ uint32_t IsMethodWithObjParam(const CFX_WideString& methodName);
bool ToJavaScript(CFX_WideTextBuf& javascript) override;
private: