From e385244f8cd6ae376f6b3cf1265a0795d5d30eff Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Mon, 14 Dec 2015 18:29:28 -0800 Subject: Get rid of most instance of 'foo == NULL' R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1520063002 . --- core/src/fxcrt/fx_xml_parser.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'core/src/fxcrt/fx_xml_parser.cpp') diff --git a/core/src/fxcrt/fx_xml_parser.cpp b/core/src/fxcrt/fx_xml_parser.cpp index 56fbada3ad..0b71836ef9 100644 --- a/core/src/fxcrt/fx_xml_parser.cpp +++ b/core/src/fxcrt/fx_xml_parser.cpp @@ -463,7 +463,7 @@ CXML_Element* CXML_Parser::ParseElement(CXML_Element* pParent, iState = 0; m_dwIndex--; CXML_Element* pSubElement = ParseElement(pElement, TRUE); - if (pSubElement == NULL) { + if (!pSubElement) { break; } pSubElement->m_pParent = pElement; @@ -767,7 +767,7 @@ FX_DWORD CXML_Element::FindElement(CXML_Element* pChild) const { } const CFX_WideString* CXML_AttrMap::Lookup(const CFX_ByteStringC& space, const CFX_ByteStringC& name) const { - if (m_pMap == NULL) { + if (!m_pMap) { return NULL; } for (int i = 0; i < m_pMap->GetSize(); i++) { @@ -803,7 +803,7 @@ void CXML_AttrMap::SetAt(const CFX_ByteStringC& space, } void CXML_AttrMap::RemoveAt(const CFX_ByteStringC& space, const CFX_ByteStringC& name) { - if (m_pMap == NULL) { + if (!m_pMap) { return; } for (int i = 0; i < m_pMap->GetSize(); i++) { @@ -816,7 +816,7 @@ void CXML_AttrMap::RemoveAt(const CFX_ByteStringC& space, } } int CXML_AttrMap::GetSize() const { - return m_pMap == NULL ? 0 : m_pMap->GetSize(); + return m_pMap ? m_pMap->GetSize() : 0; } CXML_AttrItem& CXML_AttrMap::GetAt(int index) const { return (*m_pMap)[index]; -- cgit v1.2.3