diff options
author | tsepez <tsepez@chromium.org> | 2016-09-22 18:46:45 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-22 18:46:45 -0700 |
commit | 917d24852841f4919f418076c0277d2742f961ca (patch) | |
tree | 144cf4f0fb5931572df34541232321abe377f432 /core/fpdfapi/fpdf_page/fpdf_page_doc.cpp | |
parent | 39ee9dfac1c6d286a4075c7e2435fe1cfe365bad (diff) | |
download | pdfium-917d24852841f4919f418076c0277d2742f961ca.tar.xz |
Make CPDF_ColorSpace::Load() return a unique_ptr
Specialize default_delete to avoid having to say
ReleaseDeleter<CPDF_ColorSpace> throughout the code.
Review-Url: https://codereview.chromium.org/2368433003
Diffstat (limited to 'core/fpdfapi/fpdf_page/fpdf_page_doc.cpp')
-rw-r--r-- | core/fpdfapi/fpdf_page/fpdf_page_doc.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/core/fpdfapi/fpdf_page/fpdf_page_doc.cpp b/core/fpdfapi/fpdf_page/fpdf_page_doc.cpp index e050060a9e..77c18df5fa 100644 --- a/core/fpdfapi/fpdf_page/fpdf_page_doc.cpp +++ b/core/fpdfapi/fpdf_page/fpdf_page_doc.cpp @@ -76,7 +76,7 @@ void CPDF_DocPageData::Clear(FX_BOOL bForceRelease) { continue; if (bForceRelease || csData->use_count() < 2) { - csData->get()->ReleaseCS(); + csData->get()->Release(); csData->reset(nullptr); } } @@ -294,15 +294,16 @@ CPDF_ColorSpace* CPDF_DocPageData::GetColorSpaceImpl( } } - CPDF_ColorSpace* pCS = CPDF_ColorSpace::Load(m_pPDFDoc, pArray); + std::unique_ptr<CPDF_ColorSpace> pCS = + CPDF_ColorSpace::Load(m_pPDFDoc, pArray); if (!pCS) return nullptr; if (!csData) { - csData = new CPDF_CountedColorSpace(pCS); + csData = new CPDF_CountedColorSpace(pCS.release()); m_ColorSpaceMap[pCSObj] = csData; } else { - csData->reset(pCS); + csData->reset(pCS.release()); } return csData->AddRef(); } @@ -335,7 +336,7 @@ void CPDF_DocPageData::ReleaseColorSpace(const CPDF_Object* pColorSpace) { return; // We have item only in m_ColorSpaceMap cache. Clean it. - pCountedColorSpace->get()->ReleaseCS(); + pCountedColorSpace->get()->Release(); pCountedColorSpace->reset(nullptr); } |