diff options
author | Lei Zhang <thestig@chromium.org> | 2018-04-12 19:51:56 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-12 19:51:56 +0000 |
commit | 9e625db795ca7e112d692bda7200b69a873d75f7 (patch) | |
tree | e97bbad731acb17c61cf62e67381c394886de390 /core/fpdfapi/page/cpdf_colorspace.cpp | |
parent | 80a6cbe0a427e155de8555bc867af745d10f9777 (diff) | |
download | pdfium-9e625db795ca7e112d692bda7200b69a873d75f7.tar.xz |
Slightly simplify CPDF_Color::SetColorSpace().
All the callers pass in a valid colorspace. SetColorSpace() is the only
caller to CPDF_ColorSpace::GetDefaultColor(). Also adjust that method to
make it harder to use incorrectly.
Change-Id: I5cf7b6d2abef630e1b9443043b13d1c2b5bed18a
Reviewed-on: https://pdfium-review.googlesource.com/30351
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfapi/page/cpdf_colorspace.cpp')
-rw-r--r-- | core/fpdfapi/page/cpdf_colorspace.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/core/fpdfapi/page/cpdf_colorspace.cpp b/core/fpdfapi/page/cpdf_colorspace.cpp index 7cecb19188..d9717f1d58 100644 --- a/core/fpdfapi/page/cpdf_colorspace.cpp +++ b/core/fpdfapi/page/cpdf_colorspace.cpp @@ -485,25 +485,25 @@ void CPDF_ColorSpace::Release() { delete this; } -int CPDF_ColorSpace::GetBufSize() const { +size_t CPDF_ColorSpace::GetBufSize() const { if (m_Family == PDFCS_PATTERN) return sizeof(PatternValue); return m_nComponents * sizeof(float); } -float* CPDF_ColorSpace::CreateBuf() { - int size = GetBufSize(); - return reinterpret_cast<float*>(FX_Alloc(uint8_t, size)); +float* CPDF_ColorSpace::CreateBuf() const { + return reinterpret_cast<float*>(FX_Alloc(uint8_t, GetBufSize())); } -void CPDF_ColorSpace::GetDefaultColor(float* buf) const { - if (!buf || m_Family == PDFCS_PATTERN) - return; +float* CPDF_ColorSpace::CreateBufAndSetDefaultColor() const { + ASSERT(m_Family != PDFCS_PATTERN); + float* buf = CreateBuf(); float min; float max; for (uint32_t i = 0; i < m_nComponents; i++) GetDefaultValue(i, &buf[i], &min, &max); + return buf; } uint32_t CPDF_ColorSpace::CountComponents() const { |