diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-02-20 15:52:31 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-20 15:52:31 +0000 |
| commit | dd019e9e8144636c75111565af5b120cf9c0ef9b (patch) | |
| tree | 6e6153acdf50585cb17b5958b616eef08cdfe209 /xfa/fxfa/fm2js/cxfa_fmexpression.h | |
| parent | e53460ff2697299d42518f587c42409b0bd673df (diff) | |
| download | pdfium-dd019e9e8144636c75111565af5b120cf9c0ef9b.tar.xz | |
Tighten up assignment instructions in formcalc.
The assignment operator can not be chained. This Cl removes the while
loop for assignments and changes it to an if(). We also can not have an
assignment inside ()'s so remove that option.
Bug: chromium:779349
Change-Id: I6934e18815f843ae8241023df6c03d8bbcd8168d
Reviewed-on: https://pdfium-review.googlesource.com/27350
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'xfa/fxfa/fm2js/cxfa_fmexpression.h')
| -rw-r--r-- | xfa/fxfa/fm2js/cxfa_fmexpression.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/fm2js/cxfa_fmexpression.h b/xfa/fxfa/fm2js/cxfa_fmexpression.h index 28f82aaedd..af8175a700 100644 --- a/xfa/fxfa/fm2js/cxfa_fmexpression.h +++ b/xfa/fxfa/fm2js/cxfa_fmexpression.h @@ -65,14 +65,14 @@ class CXFA_FMVarExpression : public CXFA_FMExpression { public: CXFA_FMVarExpression(uint32_t line, const WideStringView& wsName, - std::unique_ptr<CXFA_FMExpression> pInit); + std::unique_ptr<CXFA_FMSimpleExpression> pInit); ~CXFA_FMVarExpression() override; bool ToJavaScript(CFX_WideTextBuf& javascript, ReturnType type) override; private: WideStringView m_wsName; - std::unique_ptr<CXFA_FMExpression> m_pInit; + std::unique_ptr<CXFA_FMSimpleExpression> m_pInit; }; class CXFA_FMExpExpression : public CXFA_FMExpression { |