diff options
author | tsepez <tsepez@chromium.org> | 2016-10-25 15:16:40 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-10-25 15:16:40 -0700 |
commit | 04c1009af887eb5d3a183dd26421f644f63d8b77 (patch) | |
tree | e0400b66f317c60447f127dcc17307f3c27c513c /xfa | |
parent | 530c7acf7aa11ff8db8743af0d62b39cf36bf539 (diff) | |
download | pdfium-04c1009af887eb5d3a183dd26421f644f63d8b77.tar.xz |
Fix some return FALSE in functions that return pointerschromium/2901
This previously compiled because FALSE is #defined to 0, and 0
is a perfectly fine value for a pointer.
Review-Url: https://codereview.chromium.org/2448943003
Diffstat (limited to 'xfa')
-rw-r--r-- | xfa/fxfa/parser/cxfa_simple_parser.cpp | 2 | ||||
-rw-r--r-- | xfa/fxfa/parser/xfa_document_datamerger_imp.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/parser/cxfa_simple_parser.cpp b/xfa/fxfa/parser/cxfa_simple_parser.cpp index f40060bca2..997094351e 100644 --- a/xfa/fxfa/parser/cxfa_simple_parser.cpp +++ b/xfa/fxfa/parser/cxfa_simple_parser.cpp @@ -37,7 +37,7 @@ CFDE_XMLNode* GetDocumentNode(CFDE_XMLDoc* pXMLDoc, pNextNode; pNextNode = pNextNode->GetNodeItem(CFDE_XMLNode::NextSibling)) { if (pNextNode->GetType() == FDE_XMLNODE_Element) - return FALSE; + return nullptr; } return pXMLNode; } diff --git a/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp b/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp index 8d50a5a96d..a0ff81405d 100644 --- a/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp +++ b/xfa/fxfa/parser/xfa_document_datamerger_imp.cpp @@ -52,7 +52,7 @@ CXFA_Node* FormValueNode_CreateChild(CXFA_Node* pValueNode, XFA_Element iType) { CXFA_Node* pChildNode = pValueNode->GetNodeItem(XFA_NODEITEM_FirstChild); if (!pChildNode) { if (iType == XFA_Element::Unknown) - return FALSE; + return nullptr; pChildNode = pValueNode->GetProperty(0, iType); } return pChildNode; |