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/cfx_seekablestreamproxy.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/cfx_seekablestreamproxy.h')
-rw-r--r-- | core/fxcrt/cfx_seekablestreamproxy.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fxcrt/cfx_seekablestreamproxy.h b/core/fxcrt/cfx_seekablestreamproxy.h index d059fb8956..1a8e6f2f3b 100644 --- a/core/fxcrt/cfx_seekablestreamproxy.h +++ b/core/fxcrt/cfx_seekablestreamproxy.h @@ -15,7 +15,7 @@ class CFX_SeekableStreamProxy : public CFX_Retainable { public: - enum class Pos { + enum class From { Begin = 0, Current, }; @@ -28,7 +28,7 @@ class CFX_SeekableStreamProxy : public CFX_Retainable { FX_STRSIZE GetBOMLength() const { return std::max(0, m_wBOMLength); } bool IsEOF() const { return m_iPosition >= GetLength(); } - void Seek(CFX_SeekableStreamProxy::Pos eSeek, FX_FILESIZE iOffset); + void Seek(From eSeek, FX_FILESIZE iOffset); FX_STRSIZE ReadString(wchar_t* pStr, FX_STRSIZE iMaxLength, bool* bEOS); void WriteString(const CFX_WideStringC& str); |