summaryrefslogtreecommitdiff
path: root/core/fxcrt
diff options
context:
space:
mode:
authorDan Sinclair <dsinclair@chromium.org>2018-04-12 13:14:49 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-04-12 13:14:49 +0000
commit6d503b875e6f75f0d8b5f29fcf811a89f12ad12d (patch)
treedfe3e6f2125e327e57b6688c9020f698f3accbfa /core/fxcrt
parente1a41afbe146c9a976d96828a3a09a8a384741d9 (diff)
downloadpdfium-6d503b875e6f75f0d8b5f29fcf811a89f12ad12d.tar.xz
Cleanup some CFX_XMLParser code
This CL cleans up minor nits in the CFX_XMLParser code. Change-Id: Ie19d12d3dcce16c9ce6088160ecdec3d9855c11f Reviewed-on: https://pdfium-review.googlesource.com/30170 Reviewed-by: Ryan Harrison <rharrison@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'core/fxcrt')
-rw-r--r--core/fxcrt/xml/cfx_xmldoc.cpp10
-rw-r--r--core/fxcrt/xml/cfx_xmldoc.h3
-rw-r--r--core/fxcrt/xml/cfx_xmlparser.cpp87
-rw-r--r--core/fxcrt/xml/cfx_xmlparser.h5
4 files changed, 40 insertions, 65 deletions
diff --git a/core/fxcrt/xml/cfx_xmldoc.cpp b/core/fxcrt/xml/cfx_xmldoc.cpp
index 64ed5a98d0..57be180f9a 100644
--- a/core/fxcrt/xml/cfx_xmldoc.cpp
+++ b/core/fxcrt/xml/cfx_xmldoc.cpp
@@ -19,8 +19,7 @@
#include "third_party/base/stl_util.h"
CFX_XMLDoc::CFX_XMLDoc(const RetainPtr<CFX_SeekableStreamProxy>& pStream)
- : m_iStatus(0),
- m_pRoot(pdfium::MakeUnique<CFX_XMLNode>()),
+ : m_pRoot(pdfium::MakeUnique<CFX_XMLNode>()),
m_pXMLParser(pdfium::MakeUnique<CFX_XMLParser>(m_pRoot.get(), pStream)) {
ASSERT(pStream);
@@ -29,11 +28,8 @@ CFX_XMLDoc::CFX_XMLDoc(const RetainPtr<CFX_SeekableStreamProxy>& pStream)
CFX_XMLDoc::~CFX_XMLDoc() {}
-int32_t CFX_XMLDoc::DoLoad() {
- if (m_iStatus < 100)
- m_iStatus = m_pXMLParser->DoParser();
-
- return m_iStatus;
+int32_t CFX_XMLDoc::Load() {
+ return m_pXMLParser->Parse();
}
void CFX_XMLDoc::CloseXML() {
diff --git a/core/fxcrt/xml/cfx_xmldoc.h b/core/fxcrt/xml/cfx_xmldoc.h
index 07be2d4e84..55976b01b3 100644
--- a/core/fxcrt/xml/cfx_xmldoc.h
+++ b/core/fxcrt/xml/cfx_xmldoc.h
@@ -19,14 +19,13 @@ class CFX_XMLDoc {
explicit CFX_XMLDoc(const RetainPtr<CFX_SeekableStreamProxy>& pStream);
~CFX_XMLDoc();
- int32_t DoLoad();
+ int32_t Load();
void CloseXML();
CFX_XMLNode* GetRoot() const { return m_pRoot.get(); }
CFX_XMLParser* GetParser() const { return m_pXMLParser.get(); }
private:
- int32_t m_iStatus;
std::unique_ptr<CFX_XMLNode> m_pRoot;
std::unique_ptr<CFX_XMLParser> m_pXMLParser;
};
diff --git a/core/fxcrt/xml/cfx_xmlparser.cpp b/core/fxcrt/xml/cfx_xmlparser.cpp
index 05e52015bd..76ea32df6c 100644
--- a/core/fxcrt/xml/cfx_xmlparser.cpp
+++ b/core/fxcrt/xml/cfx_xmlparser.cpp
@@ -15,58 +15,46 @@
CFX_XMLParser::CFX_XMLParser(CFX_XMLNode* pParent,
const RetainPtr<CFX_SeekableStreamProxy>& pStream)
- : m_pStream(pStream),
- m_pParser(pdfium::MakeUnique<CFX_XMLSyntaxParser>(m_pStream)),
+ : m_pParser(pdfium::MakeUnique<CFX_XMLSyntaxParser>(pStream)),
m_pParent(pParent),
- m_pChild(nullptr),
- m_syntaxParserResult(FX_XmlSyntaxResult::None) {
- ASSERT(m_pParent && m_pStream);
+ m_pChild(nullptr) {
+ ASSERT(m_pParent && pStream);
m_NodeStack.push(m_pParent);
}
CFX_XMLParser::~CFX_XMLParser() {}
-int32_t CFX_XMLParser::DoParser() {
- if (m_syntaxParserResult == FX_XmlSyntaxResult::Error)
- return -1;
- if (m_syntaxParserResult == FX_XmlSyntaxResult::EndOfString)
- return 100;
-
+int32_t CFX_XMLParser::Parse() {
int32_t iCount = 0;
while (true) {
- m_syntaxParserResult = m_pParser->DoSyntaxParse();
- switch (m_syntaxParserResult) {
- case FX_XmlSyntaxResult::InstructionOpen:
- break;
+ FX_XmlSyntaxResult result = m_pParser->DoSyntaxParse();
+ if (result == FX_XmlSyntaxResult::Error)
+ return -1;
+ if (result == FX_XmlSyntaxResult::EndOfString)
+ break;
+
+ switch (result) {
case FX_XmlSyntaxResult::InstructionClose:
- if (m_pChild) {
- if (m_pChild->GetType() != FX_XMLNODE_Instruction) {
- m_syntaxParserResult = FX_XmlSyntaxResult::Error;
- break;
- }
- }
+ if (m_pChild && m_pChild->GetType() != FX_XMLNODE_Instruction)
+ return -1;
+
m_pChild = m_pParent;
break;
- case FX_XmlSyntaxResult::ElementOpen:
- case FX_XmlSyntaxResult::ElementBreak:
- break;
case FX_XmlSyntaxResult::ElementClose:
- if (m_pChild->GetType() != FX_XMLNODE_Element) {
- m_syntaxParserResult = FX_XmlSyntaxResult::Error;
- break;
- }
+ if (m_pChild->GetType() != FX_XMLNODE_Element)
+ return -1;
+
m_ws1 = m_pParser->GetTagName();
- m_ws2 = static_cast<CFX_XMLElement*>(m_pChild)->GetName();
- if (m_ws1.GetLength() > 0 && m_ws1 != m_ws2) {
- m_syntaxParserResult = FX_XmlSyntaxResult::Error;
- break;
+ if (m_ws1.GetLength() > 0 &&
+ m_ws1 != static_cast<CFX_XMLElement*>(m_pChild)->GetName()) {
+ return -1;
}
+
if (!m_NodeStack.empty())
m_NodeStack.pop();
- if (m_NodeStack.empty()) {
- m_syntaxParserResult = FX_XmlSyntaxResult::Error;
- break;
- }
+ if (m_NodeStack.empty())
+ return -1;
+
m_pParent = m_NodeStack.top();
m_pChild = m_pParent;
iCount++;
@@ -92,10 +80,9 @@ int32_t CFX_XMLParser::DoParser() {
m_ws1 = m_pParser->GetAttributeName();
break;
case FX_XmlSyntaxResult::AttriValue:
- if (m_pChild) {
- m_ws2 = m_pParser->GetAttributeName();
- if (m_pChild->GetType() == FX_XMLNODE_Element)
- static_cast<CFX_XMLElement*>(m_pChild)->SetString(m_ws1, m_ws2);
+ if (m_pChild && m_pChild->GetType() == FX_XMLNODE_Element) {
+ static_cast<CFX_XMLElement*>(m_pChild)->SetString(
+ m_ws1, m_pParser->GetAttributeName());
}
m_ws1.clear();
break;
@@ -113,27 +100,23 @@ int32_t CFX_XMLParser::DoParser() {
break;
case FX_XmlSyntaxResult::TargetData:
if (m_pChild) {
- if (m_pChild->GetType() != FX_XMLNODE_Instruction) {
- m_syntaxParserResult = FX_XmlSyntaxResult::Error;
- break;
- }
+ if (m_pChild->GetType() != FX_XMLNODE_Instruction)
+ return -1;
+
auto* instruction = static_cast<CFX_XMLInstruction*>(m_pChild);
if (!m_ws1.IsEmpty())
instruction->AppendData(m_ws1);
+
instruction->AppendData(m_pParser->GetTargetData());
}
m_ws1.clear();
break;
+ case FX_XmlSyntaxResult::ElementOpen:
+ case FX_XmlSyntaxResult::ElementBreak:
+ case FX_XmlSyntaxResult::InstructionOpen:
default:
break;
}
- if (m_syntaxParserResult == FX_XmlSyntaxResult::Error ||
- m_syntaxParserResult == FX_XmlSyntaxResult::EndOfString) {
- break;
- }
}
- return (m_syntaxParserResult == FX_XmlSyntaxResult::Error ||
- m_NodeStack.size() != 1)
- ? -1
- : m_pParser->GetStatus();
+ return m_NodeStack.size() != 1 ? -1 : m_pParser->GetStatus();
}
diff --git a/core/fxcrt/xml/cfx_xmlparser.h b/core/fxcrt/xml/cfx_xmlparser.h
index 2998a44ad5..8c4c354699 100644
--- a/core/fxcrt/xml/cfx_xmlparser.h
+++ b/core/fxcrt/xml/cfx_xmlparser.h
@@ -24,17 +24,14 @@ class CFX_XMLParser {
const RetainPtr<CFX_SeekableStreamProxy>& pStream);
~CFX_XMLParser();
- int32_t DoParser();
+ int32_t Parse();
private:
- RetainPtr<CFX_SeekableStreamProxy> m_pStream;
std::unique_ptr<CFX_XMLSyntaxParser> m_pParser;
CFX_XMLNode* m_pParent;
CFX_XMLNode* m_pChild;
std::stack<CFX_XMLNode*> m_NodeStack;
WideString m_ws1;
- WideString m_ws2;
- FX_XmlSyntaxResult m_syntaxParserResult;
};
#endif // CORE_FXCRT_XML_CFX_XMLPARSER_H_