diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2017-08-30 10:22:55 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-08-30 14:33:53 +0000 |
| commit | 980a3ea30872cef9ada360aa85e7c3573d7668b5 (patch) | |
| tree | bf4306399f11aa9e46a6274f1665a5d41c417bb6 /core/fxcrt/xml/cfx_xmlsyntaxparser.h | |
| parent | 304eefb58759e56be3fb357c78204accd4fa98fc (diff) | |
| download | pdfium-980a3ea30872cef9ada360aa85e7c3573d7668b5.tar.xz | |
Cleanup CFX_SeekableStreamProxy
Clean up being done in preperation for removal of negative length
strings. This means that FX_STRSIZE will become unsigned so the return
methods cannot return -1. Other cleanup suggested by reviewers is
included.
BUG=pdfium:828
Change-Id: I2cfb78c8ab7738516e0c9f8a99caec6f6cb12cde
Reviewed-on: https://pdfium-review.googlesource.com/12170
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'core/fxcrt/xml/cfx_xmlsyntaxparser.h')
| -rw-r--r-- | core/fxcrt/xml/cfx_xmlsyntaxparser.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fxcrt/xml/cfx_xmlsyntaxparser.h b/core/fxcrt/xml/cfx_xmlsyntaxparser.h index 32e55f1ff0..c59cabae19 100644 --- a/core/fxcrt/xml/cfx_xmlsyntaxparser.h +++ b/core/fxcrt/xml/cfx_xmlsyntaxparser.h @@ -102,13 +102,13 @@ class CFX_XMLSyntaxParser { CFX_RetainPtr<CFX_SeekableStreamProxy> m_pStream; FX_STRSIZE m_iXMLPlaneSize; - int32_t m_iCurrentPos; + FX_FILESIZE m_iCurrentPos; int32_t m_iCurrentNodeNum; int32_t m_iLastNodeNum; int32_t m_iParsedBytes; FX_FILESIZE m_ParsedChars; std::vector<wchar_t> m_Buffer; - int32_t m_iBufferChars; + FX_STRSIZE m_iBufferChars; bool m_bEOS; FX_FILESIZE m_Start; // Start position in m_Buffer FX_FILESIZE m_End; // End position in m_Buffer |