From 0545d2a5d759dbbb6ac3cce6d82bc0f2188cc4aa Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Thu, 4 Jan 2018 16:20:05 -0500 Subject: Convert RecognizeXFAVersionNumber to us a retval instead of a out param Change-Id: Iefca0b9cb00a7ba9f37fc4eae618831e4e209ac4 Reviewed-on: https://pdfium-review.googlesource.com/22261 Reviewed-by: dsinclair Commit-Queue: Ryan Harrison --- xfa/fxfa/parser/xfa_utils.cpp | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/xfa/fxfa/parser/xfa_utils.cpp b/xfa/fxfa/parser/xfa_utils.cpp index 0105752937..c41835f19c 100644 --- a/xfa/fxfa/parser/xfa_utils.cpp +++ b/xfa/fxfa/parser/xfa_utils.cpp @@ -391,23 +391,20 @@ void RegenerateFormFile_Container( } } -void RecognizeXFAVersionNumber(CXFA_Node* pTemplateRoot, - WideString& wsVersionNumber) { - wsVersionNumber.clear(); +WideString RecognizeXFAVersionNumber(CXFA_Node* pTemplateRoot) { if (!pTemplateRoot) - return; + return WideString(); Optional templateNS = pTemplateRoot->JSObject()->TryNamespace(); if (!templateNS) - return; + return WideString(); XFA_VERSION eVersion = pTemplateRoot->GetDocument()->RecognizeXFAVersionNumber(*templateNS); if (eVersion == XFA_VERSION_UNKNOWN) eVersion = XFA_VERSION_DEFAULT; - wsVersionNumber = - WideString::Format(L"%i.%i", eVersion / 100, eVersion % 100); + return WideString::Format(L"%i.%i", eVersion / 100, eVersion % 100); } } // namespace @@ -541,10 +538,8 @@ void XFA_DataExporter_RegenerateFormFile( pStream->WriteString(L" xmlns=\""); pStream->WriteString(WideStringView(kFormNS)); - WideString wsVersionNumber; - RecognizeXFAVersionNumber( - ToNode(pNode->GetDocument()->GetXFAObject(XFA_HASHCODE_Template)), - wsVersionNumber); + WideString wsVersionNumber = RecognizeXFAVersionNumber( + ToNode(pNode->GetDocument()->GetXFAObject(XFA_HASHCODE_Template))); if (wsVersionNumber.IsEmpty()) wsVersionNumber = L"2.8"; -- cgit v1.2.3