diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2018-04-23 21:14:39 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-23 21:14:39 +0000 |
| commit | 2dca1cbf595618e06547a2c464d82c47b6acace6 (patch) | |
| tree | 2c58686780911e78526c988bb85f0b4789c852ee /core/fxcrt/xml/cfx_xmlparser.h | |
| parent | 1e95aeadd33fc4a13f5579501f81b6d5edc7cc05 (diff) | |
| download | pdfium-2dca1cbf595618e06547a2c464d82c47b6acace6.tar.xz | |
Cleanup CFX_XMLParser initialization
This CL moves most initialization of CFX_XMLParser into the header file
and re-organizes the members to group members together better.
Change-Id: I49031ea65dc0d68c9b47cffe942c65f488bbaf20
Reviewed-on: https://pdfium-review.googlesource.com/31250
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core/fxcrt/xml/cfx_xmlparser.h')
| -rw-r--r-- | core/fxcrt/xml/cfx_xmlparser.h | 33 |
1 files changed, 16 insertions, 17 deletions
diff --git a/core/fxcrt/xml/cfx_xmlparser.h b/core/fxcrt/xml/cfx_xmlparser.h index 6405b61a64..03f649c933 100644 --- a/core/fxcrt/xml/cfx_xmlparser.h +++ b/core/fxcrt/xml/cfx_xmlparser.h @@ -95,27 +95,26 @@ class CFX_XMLParser { bool GetStatus() const; CFX_XMLNode* m_pParent; - CFX_XMLNode* m_pChild; - std::stack<CFX_XMLNode*> m_NodeStack; + CFX_XMLNode* m_pChild = nullptr; WideString m_ws1; - RetainPtr<IFX_SeekableReadStream> m_pStream; - size_t m_iXMLPlaneSize; - std::vector<wchar_t> m_Buffer; - FX_FILESIZE m_Start; // Start position in m_Buffer - FX_FILESIZE m_End; // End position in m_Buffer - FX_XMLNODETYPE m_CurNodeType; + FX_FILESIZE m_Start = 0; // Start position in m_Buffer + FX_FILESIZE m_End = 0; // End position in m_Buffer + FX_XMLNODETYPE m_CurNodeType = FX_XMLNODE_Unknown; + FX_XmlSyntaxResult m_syntaxParserResult = FX_XmlSyntaxResult::None; + FDE_XmlSyntaxState m_syntaxParserState = FDE_XmlSyntaxState::Text; + std::stack<CFX_XMLNode*> m_NodeStack; std::stack<FX_XMLNODETYPE> m_XMLNodeTypeStack; - CFX_BlockBuffer m_BlockBuffer; - wchar_t* m_pCurrentBlock; // Pointer into CFX_BlockBuffer - size_t m_iIndexInBlock; - int32_t m_iTextDataLength; - FX_XmlSyntaxResult m_syntaxParserResult; - FDE_XmlSyntaxState m_syntaxParserState; - wchar_t m_wQuotationMark; - int32_t m_iEntityStart; std::stack<wchar_t> m_SkipStack; - wchar_t m_SkipChar; + std::vector<wchar_t> m_Buffer; + CFX_BlockBuffer m_BlockBuffer; + wchar_t* m_pCurrentBlock = nullptr; // Pointer into CFX_BlockBuffer + size_t m_iIndexInBlock = 0; + size_t m_iXMLPlaneSize = 1024; + int32_t m_iTextDataLength = 0; + int32_t m_iEntityStart = -1; + wchar_t m_wQuotationMark = 0; + wchar_t m_SkipChar = 0; }; #endif // CORE_FXCRT_XML_CFX_XMLPARSER_H_ |