diff options
| author | tsepez <tsepez@chromium.org> | 2016-11-09 13:28:26 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-11-09 13:28:26 -0800 |
| commit | 335cf093231c984a23cb9ea113148ea1f19621ba (patch) | |
| tree | e9c7803b0ce71269beb3d423549a2d6a0ac7784a /core/fpdfdoc/cpdf_formfield.cpp | |
| parent | 3ff4deea307c38462393e4f83dabe32949338168 (diff) | |
| download | pdfium-335cf093231c984a23cb9ea113148ea1f19621ba.tar.xz | |
Return unique_ptr from CPDF_Object::Clone().
Because that's what clone does. There are numerous
release() calls that will go away as more code is
converted.
Review-Url: https://codereview.chromium.org/2484033002
Diffstat (limited to 'core/fpdfdoc/cpdf_formfield.cpp')
| -rw-r--r-- | core/fpdfdoc/cpdf_formfield.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/core/fpdfdoc/cpdf_formfield.cpp b/core/fpdfdoc/cpdf_formfield.cpp index 8170b7fcbf..4999f2d6b8 100644 --- a/core/fpdfdoc/cpdf_formfield.cpp +++ b/core/fpdfdoc/cpdf_formfield.cpp @@ -219,14 +219,14 @@ bool CPDF_FormField::ResetField(bool bNotify) { return false; if (pDV) { - CPDF_Object* pClone = pDV->Clone(); + std::unique_ptr<CPDF_Object> pClone = pDV->Clone(); if (!pClone) return false; - m_pDict->SetFor("V", pClone); + m_pDict->SetFor("V", pClone.release()); if (pRV) { - CPDF_Object* pCloneR = pDV->Clone(); - m_pDict->SetFor("RV", pCloneR); + std::unique_ptr<CPDF_Object> pCloneR = pDV->Clone(); + m_pDict->SetFor("RV", pCloneR.release()); } } else { m_pDict->RemoveFor("V"); |