diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-06-29 21:33:34 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-06-30 04:45:24 +0000 |
| commit | a173900b19e158130df049dd1c31b7494baefffd (patch) | |
| tree | 68a10871f738b3dca78a9341138ca5e7680d973b /xfa/fxfa/cxfa_ffdocview.h | |
| parent | 91e39d2218c1b4e67321f845daf0fdb8d171982b (diff) | |
| download | pdfium-a173900b19e158130df049dd1c31b7494baefffd.tar.xz | |
Fix nits in a bunch of random XFA files.
- Simplify if (cond) return true; return false; pattern.
- Use size_t in CXFA_FFDocView::RunCalculateRecursive().
Change-Id: I1c426556bc927a118cb062999812ab06bbfcfec7
Reviewed-on: https://pdfium-review.googlesource.com/7130
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffdocview.h')
| -rw-r--r-- | xfa/fxfa/cxfa_ffdocview.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxfa/cxfa_ffdocview.h b/xfa/fxfa/cxfa_ffdocview.h index 3164bcfb6a..314ce29393 100644 --- a/xfa/fxfa/cxfa_ffdocview.h +++ b/xfa/fxfa/cxfa_ffdocview.h @@ -49,8 +49,8 @@ class CXFA_FFDocView { void StopLayout(); int32_t GetLayoutStatus(); void UpdateDocView(); - int32_t CountPageViews(); - CXFA_FFPageView* GetPageView(int32_t nIndex); + int32_t CountPageViews() const; + CXFA_FFPageView* GetPageView(int32_t nIndex) const; void ResetWidgetData(CXFA_WidgetAcc* pWidgetAcc); int32_t ProcessWidgetEvent(CXFA_EventParam* pParam, @@ -111,7 +111,7 @@ class CXFA_FFDocView { void RunBindItems(); bool InitCalculate(CXFA_Node* pNode); void InitLayout(CXFA_Node* pNode); - void RunCalculateRecursive(int32_t& iIndex); + size_t RunCalculateRecursive(size_t index); void ShowNullTestMsg(); bool ResetSingleWidgetAccData(CXFA_WidgetAcc* pWidgetAcc); CXFA_Node* GetRootSubform(); |