From 9a3a7709103a872037dcea1f3cf0b7785a3da191 Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Mon, 23 Apr 2018 18:14:16 +0000 Subject: Change CFX_XML Save to take a write stream This CL changes CFX_XML to use an IFX_SeekableWriteStream instead of the more generic IFX_SeekableStream. Change-Id: I6e4def380c43eca755d91ad5cb6146c2dfdaee10 Reviewed-on: https://pdfium-review.googlesource.com/30877 Commit-Queue: dsinclair Reviewed-by: Tom Sepez --- core/fpdfdoc/cpdf_metadata.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'core/fpdfdoc') diff --git a/core/fpdfdoc/cpdf_metadata.cpp b/core/fpdfdoc/cpdf_metadata.cpp index 161fc93edd..c928952dca 100644 --- a/core/fpdfdoc/cpdf_metadata.cpp +++ b/core/fpdfdoc/cpdf_metadata.cpp @@ -19,12 +19,11 @@ void CheckForSharedFormInternal(CFX_XMLElement* element, std::vector* unsupported) { WideString attr = element->GetAttribute(L"xmlns:adhocwf"); if (attr == L"http://ns.adobe.com/AcrobatAdhocWorkflow/1.0/") { - for (const auto* child = element->GetFirstChild(); child; - child = child->GetNextSibling()) { + for (const auto& child : *element) { if (child->GetType() != FX_XMLNODE_Element) continue; - const auto* child_elem = static_cast(child); + const auto* child_elem = static_cast(child.get()); if (child_elem->GetName() != L"adhocwf:workflowType") continue; @@ -46,12 +45,11 @@ void CheckForSharedFormInternal(CFX_XMLElement* element, } } - for (auto* child = element->GetFirstChild(); child; - child = child->GetNextSibling()) { + for (const auto& child : *element) { if (child->GetType() != FX_XMLNODE_Element) continue; - CheckForSharedFormInternal(static_cast(child), + CheckForSharedFormInternal(static_cast(child.get()), unsupported); } } -- cgit v1.2.3