diff options
author | dan sinclair <dsinclair@chromium.org> | 2018-02-19 18:23:13 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-19 18:23:13 +0000 |
commit | 0f9e15cf365faf025d00e7df2565c8482a245953 (patch) | |
tree | 9a4db2247c0fb8f4764f9d1630ab493ec31bb52d /xfa/fxfa/fm2js/cxfa_fmparser.h | |
parent | 43e0be7b93c24d6a37aa3a0dca1f8c89dc78b165 (diff) | |
download | pdfium-0f9e15cf365faf025d00e7df2565c8482a245953.tar.xz |
FormCalc Assignment is not a SimpleExpression
Currently the parser builds assignment statements as part of the
SimpleExpression declaration. This isn't correct according to the
grammar where AssignmentExpression and SimpleExpression are siblings.
This CL moves the assignment calculation into the ExpExpression
declaration to make it a sibling of the SimpleExpression.
Change-Id: I6afac2379ab6783b84ee619863c8308ca0db454d
Reviewed-on: https://pdfium-review.googlesource.com/27310
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fxfa/fm2js/cxfa_fmparser.h')
-rw-r--r-- | xfa/fxfa/fm2js/cxfa_fmparser.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/fm2js/cxfa_fmparser.h b/xfa/fxfa/fm2js/cxfa_fmparser.h index 3958930224..c2da48496f 100644 --- a/xfa/fxfa/fm2js/cxfa_fmparser.h +++ b/xfa/fxfa/fm2js/cxfa_fmparser.h @@ -30,10 +30,10 @@ class CXFA_FMParser { bool CheckThenNext(XFA_FM_TOKEN op); bool IncrementParseDepthAndCheck(); - std::vector<std::unique_ptr<CXFA_FMExpression>> ParseTopExpression(); + std::vector<std::unique_ptr<CXFA_FMExpression>> ParseExpressionList(); std::unique_ptr<CXFA_FMExpression> ParseFunction(); std::unique_ptr<CXFA_FMExpression> ParseExpression(); - std::unique_ptr<CXFA_FMExpression> ParseVarExpression(); + std::unique_ptr<CXFA_FMExpression> ParseDeclarationExpression(); std::unique_ptr<CXFA_FMExpression> ParseExpExpression(); std::unique_ptr<CXFA_FMExpression> ParseBlockExpression(); std::unique_ptr<CXFA_FMExpression> ParseIfExpression(); |