diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-21 21:52:50 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-21 21:52:50 +0000 |
| commit | 64e16079bb87085fdff5d6268267fd01eacbc5d9 (patch) | |
| tree | 0c13b2e9fb686dc18cdf49f4091e22f089eea65c /xfa/fxfa/parser/cxfa_valuedata.cpp | |
| parent | f9958ed00c5f288d95ab4c6c1d90fc639d5c0f8d (diff) | |
| download | pdfium-64e16079bb87085fdff5d6268267fd01eacbc5d9.tar.xz | |
Cleanup CXFA_ImageData
This CL removes the second constructor parameter which is always true
and removes all related conditionals. The methods have been changed to
const and return instead of using out parameters.
Change-Id: Id6d2aa533a212e961e3b8169eef57b7e868d503e
Reviewed-on: https://pdfium-review.googlesource.com/19010
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_valuedata.cpp')
| -rw-r--r-- | xfa/fxfa/parser/cxfa_valuedata.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/xfa/fxfa/parser/cxfa_valuedata.cpp b/xfa/fxfa/parser/cxfa_valuedata.cpp index e44a76dbcc..1ca051e637 100644 --- a/xfa/fxfa/parser/cxfa_valuedata.cpp +++ b/xfa/fxfa/parser/cxfa_valuedata.cpp @@ -59,7 +59,6 @@ CXFA_ExDataData CXFA_ValueData::GetExData() { } CXFA_ImageData CXFA_ValueData::GetImageData() { - return CXFA_ImageData( - m_pNode ? (m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild)) : nullptr, - true); + return CXFA_ImageData(m_pNode ? m_pNode->GetNodeItem(XFA_NODEITEM_FirstChild) + : nullptr); } |