diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-22 22:09:42 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-22 22:09:42 +0000 |
| commit | fd16e44aa9b0e63e6d25254a3b812fb27152b6f4 (patch) | |
| tree | 92092b4cc86a05f90518b63c4f884566380bfd92 /xfa/fxfa/cxfa_widgetacc.cpp | |
| parent | 9d0d36b7f3e2106bbeb332edf821fc2c637e6ea0 (diff) | |
| download | pdfium-fd16e44aa9b0e63e6d25254a3b812fb27152b6f4.tar.xz | |
Cleanup CXFA_ValueData
This CL makes the methods const and uses return values instead of
out-params.
Change-Id: If760cd8c10d8f3a94dd3e13c46200cb6738a4d5a
Reviewed-on: https://pdfium-review.googlesource.com/19150
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_widgetacc.cpp')
| -rw-r--r-- | xfa/fxfa/cxfa_widgetacc.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xfa/fxfa/cxfa_widgetacc.cpp b/xfa/fxfa/cxfa_widgetacc.cpp index 644b1688c0..683c8a2619 100644 --- a/xfa/fxfa/cxfa_widgetacc.cpp +++ b/xfa/fxfa/cxfa_widgetacc.cpp @@ -197,7 +197,7 @@ void CXFA_WidgetAcc::ResetData() { if (wsValue.IsEmpty()) { CXFA_ValueData defValueData = pAcc->GetDefaultValueData(); if (defValueData.HasValidNode()) { - defValueData.GetChildValueContent(wsValue); + wsValue = defValueData.GetChildValueContent(); SetValue(wsValue, XFA_VALUEPICTURE_Raw); pAcc->SetValue(wsValue, XFA_VALUEPICTURE_Raw); done = true; @@ -227,7 +227,7 @@ void CXFA_WidgetAcc::ResetData() { default: { CXFA_ValueData defValueData = GetDefaultValueData(); if (defValueData.HasValidNode()) - defValueData.GetChildValueContent(wsValue); + wsValue = defValueData.GetChildValueContent(); SetValue(wsValue, XFA_VALUEPICTURE_Raw); break; |