diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-20 20:28:43 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-20 20:28:43 +0000 |
| commit | 03d9180f30f2ba79ef48b4003306224428f4ad47 (patch) | |
| tree | b2d9daf2b6b026bac946589c2781ea555ce93059 /fxjs/cjx_node.cpp | |
| parent | e74ae8c02f6b5efa37ba11bcad6b714046cfc7b0 (diff) | |
| download | pdfium-03d9180f30f2ba79ef48b4003306224428f4ad47.tar.xz | |
Remove return value from SetAttributeValue
The CJX_Node::SetAttributeValue method always returns |true| and we
never check the value. This CL changes the method to return |void|
instead.
Change-Id: Ia3162627dee926d602e7ffe74f9dca2c1fd81ca8
Reviewed-on: https://pdfium-review.googlesource.com/18770
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/cjx_node.cpp')
| -rw-r--r-- | fxjs/cjx_node.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fxjs/cjx_node.cpp b/fxjs/cjx_node.cpp index ba55086261..bf8c1f726e 100644 --- a/fxjs/cjx_node.cpp +++ b/fxjs/cjx_node.cpp @@ -3165,7 +3165,7 @@ bool CJX_Node::SetCData(XFA_Attribute eAttr, return true; } -bool CJX_Node::SetAttributeValue(const WideString& wsValue, +void CJX_Node::SetAttributeValue(const WideString& wsValue, const WideString& wsXMLValue, bool bNotify, bool bScriptModify) { @@ -3176,7 +3176,7 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue, SetUserData(pKey, pClone, &deleteWideStringCallBack); OnChanged(XFA_Attribute::Value, bNotify, bScriptModify); if (!GetXFANode()->IsNeedSavingXMLNode()) - return true; + return; auto* elem = static_cast<CFX_XMLElement*>(GetXFANode()->GetXMLMappingNode()); FX_XMLNODETYPE eXMLType = elem->GetType(); @@ -3211,7 +3211,6 @@ bool CJX_Node::SetAttributeValue(const WideString& wsValue, default: ASSERT(0); } - return true; } pdfium::Optional<WideString> CJX_Node::TryCData(XFA_Attribute eAttr, |