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/fpdfapi/parser/cpdf_number.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/fpdfapi/parser/cpdf_number.cpp')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_number.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/core/fpdfapi/parser/cpdf_number.cpp b/core/fpdfapi/parser/cpdf_number.cpp index 3ae629ede9..24feb2a2e0 100644 --- a/core/fpdfapi/parser/cpdf_number.cpp +++ b/core/fpdfapi/parser/cpdf_number.cpp @@ -5,6 +5,7 @@ // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com #include "core/fpdfapi/parser/cpdf_number.h" +#include "third_party/base/ptr_util.h" CPDF_Number::CPDF_Number() : m_bInteger(true), m_Integer(0) {} @@ -21,8 +22,9 @@ CPDF_Object::Type CPDF_Number::GetType() const { return NUMBER; } -CPDF_Object* CPDF_Number::Clone() const { - return m_bInteger ? new CPDF_Number(m_Integer) : new CPDF_Number(m_Float); +std::unique_ptr<CPDF_Object> CPDF_Number::Clone() const { + return m_bInteger ? pdfium::MakeUnique<CPDF_Number>(m_Integer) + : pdfium::MakeUnique<CPDF_Number>(m_Float); } FX_FLOAT CPDF_Number::GetNumber() const { |