diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-03-06 11:36:09 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-03-06 18:48:03 +0000 |
| commit | f04b42a9beedb1287977794211d05d92903559db (patch) | |
| tree | 837f851e4b1204d5fd9d19963324df1ae1af9fe1 /xfa/fgas/layout/fgas_rtfbreak.h | |
| parent | 19fad57424146f0987265c701a58450c43f09008 (diff) | |
| download | pdfium-f04b42a9beedb1287977794211d05d92903559db.tar.xz | |
Simplify RTFBreak AppendChar.
This Cl simplifies the AppendChar code by removing the array of callback
methods and using a switch instead.
Change-Id: I07475d3950395dea2189569d95bfba1be1f9ceb5
Reviewed-on: https://pdfium-review.googlesource.com/2918
Reviewed-by: Nicolás Peña <npm@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa/fgas/layout/fgas_rtfbreak.h')
| -rw-r--r-- | xfa/fgas/layout/fgas_rtfbreak.h | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/xfa/fgas/layout/fgas_rtfbreak.h b/xfa/fgas/layout/fgas_rtfbreak.h index 2dfc0248a1..63ad67f6c0 100644 --- a/xfa/fgas/layout/fgas_rtfbreak.h +++ b/xfa/fgas/layout/fgas_rtfbreak.h @@ -172,13 +172,12 @@ class CFX_RTFBreak { CFX_RTFLine* GetCurrentLineForTesting() const { return m_pCurLine; } - CFX_RTFBreakType AppendChar_Combination(CFX_RTFChar* pCurChar); - CFX_RTFBreakType AppendChar_Tab(CFX_RTFChar* pCurChar); + private: + void AppendChar_Combination(CFX_RTFChar* pCurChar); + void AppendChar_Tab(CFX_RTFChar* pCurChar); CFX_RTFBreakType AppendChar_Control(CFX_RTFChar* pCurChar); CFX_RTFBreakType AppendChar_Arabic(CFX_RTFChar* pCurChar); CFX_RTFBreakType AppendChar_Others(CFX_RTFChar* pCurChar); - - private: void FontChanged(); void SetBreakStatus(); CFX_RTFChar* GetLastChar(int32_t index) const; |