diff options
| author | dan sinclair <dsinclair@chromium.org> | 2017-05-17 22:17:34 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-05-18 15:27:37 +0000 |
| commit | 2034b941dfa07d30c286bfcceffd6904740ac446 (patch) | |
| tree | 1bc3c9be4d3108b53ced4db9837187dddf6df88d /xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp | |
| parent | cfd56852b6375d2b4eea473399231caf7dcdbf36 (diff) | |
| download | pdfium-2034b941dfa07d30c286bfcceffd6904740ac446.tar.xz | |
Remove CXFA_FMErrorInfo
This Cl removes the CXFA_FMErrorInfo class. The message was never output, just
used as a flag to determine if there was an error. The class has been replaced
with a boolean.
Change-Id: I1cde99ce6957f5f8c6be0755a198d80ec8378b3a
Reviewed-on: https://pdfium-review.googlesource.com/5653
Reviewed-by: Nicolás Peña <npm@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp')
| -rw-r--r-- | xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp b/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp index f191707ca3..a9ce54d6a6 100644 --- a/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp +++ b/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp @@ -6119,11 +6119,9 @@ bool CXFA_FM2JSContext::Translate(const CFX_WideStringC& wsFormcalc, return true; } - CXFA_FMErrorInfo errorInfo; - CXFA_FMParse parser(wsFormcalc, &errorInfo); - + CXFA_FMParse parser(wsFormcalc); std::unique_ptr<CXFA_FMFunctionDefinition> func = parser.Parse(); - if (!errorInfo.message.IsEmpty()) + if (parser.HasError()) return false; if (!func->ToJavaScript(*wsJavascript)) |