From a17a0e219d5fcd4b8714cf97e8daf119d4388bee Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Wed, 28 Mar 2018 21:10:35 +0000 Subject: Remove CPDF_DefaultAppearance reference params This CL converts the CPDF_DefaultAppearance::GetColor methods to return instead of using reference parameters. Change-Id: I9ba6bf060a536290a83d3c9ce6e115511f28664c Reviewed-on: https://pdfium-review.googlesource.com/29410 Commit-Queue: dsinclair Reviewed-by: Henrique Nakashima --- core/fpdfdoc/cpdf_annotlist.cpp | 1 + core/fpdfdoc/cpdf_apsettings.cpp | 1 + core/fpdfdoc/cpdf_defaultappearance.cpp | 43 +++++++++++++++------------------ core/fpdfdoc/cpdf_defaultappearance.h | 8 +++--- core/fpdfdoc/cpvt_generateap.h | 1 + 5 files changed, 28 insertions(+), 26 deletions(-) (limited to 'core/fpdfdoc') diff --git a/core/fpdfdoc/cpdf_annotlist.cpp b/core/fpdfdoc/cpdf_annotlist.cpp index 192649d4a4..6b359f1eb5 100644 --- a/core/fpdfdoc/cpdf_annotlist.cpp +++ b/core/fpdfdoc/cpdf_annotlist.cpp @@ -19,6 +19,7 @@ #include "core/fpdfapi/parser/cpdf_string.h" #include "core/fpdfapi/render/cpdf_renderoptions.h" #include "core/fpdfdoc/cpdf_annot.h" +#include "core/fpdfdoc/cpdf_formfield.h" #include "core/fpdfdoc/cpdf_interform.h" #include "core/fpdfdoc/cpdf_occontext.h" #include "core/fpdfdoc/cpvt_generateap.h" diff --git a/core/fpdfdoc/cpdf_apsettings.cpp b/core/fpdfdoc/cpdf_apsettings.cpp index 076cdd9b9e..0f0989252e 100644 --- a/core/fpdfdoc/cpdf_apsettings.cpp +++ b/core/fpdfdoc/cpdf_apsettings.cpp @@ -10,6 +10,7 @@ #include "core/fpdfapi/parser/cpdf_array.h" #include "core/fpdfapi/parser/cpdf_dictionary.h" +#include "core/fpdfdoc/cpdf_formcontrol.h" #include "core/fxge/cfx_color.h" CPDF_ApSettings::CPDF_ApSettings(CPDF_Dictionary* pDict) : m_pDict(pDict) {} diff --git a/core/fpdfdoc/cpdf_defaultappearance.cpp b/core/fpdfdoc/cpdf_defaultappearance.cpp index 8878dc0093..add1c26498 100644 --- a/core/fpdfdoc/cpdf_defaultappearance.cpp +++ b/core/fpdfdoc/cpdf_defaultappearance.cpp @@ -88,64 +88,61 @@ bool CPDF_DefaultAppearance::HasColor() { return FindTagParamFromStart(&syntax, "k", 4); } -void CPDF_DefaultAppearance::GetColor(int& iColorType, float fc[4]) { - iColorType = CFX_Color::kTransparent; +CFX_Color::Type CPDF_DefaultAppearance::GetColor(float fc[4]) { for (int c = 0; c < 4; c++) fc[c] = 0; if (m_csDA.IsEmpty()) - return; + return CFX_Color::kTransparent; CPDF_SimpleParser syntax(m_csDA.AsStringView()); if (FindTagParamFromStart(&syntax, "g", 1)) { - iColorType = CFX_Color::kGray; fc[0] = FX_atof(syntax.GetWord()); - return; + return CFX_Color::kGray; } if (FindTagParamFromStart(&syntax, "rg", 3)) { - iColorType = CFX_Color::kRGB; fc[0] = FX_atof(syntax.GetWord()); fc[1] = FX_atof(syntax.GetWord()); fc[2] = FX_atof(syntax.GetWord()); - return; + return CFX_Color::kRGB; } if (FindTagParamFromStart(&syntax, "k", 4)) { - iColorType = CFX_Color::kCMYK; fc[0] = FX_atof(syntax.GetWord()); fc[1] = FX_atof(syntax.GetWord()); fc[2] = FX_atof(syntax.GetWord()); fc[3] = FX_atof(syntax.GetWord()); + return CFX_Color::kCMYK; } + + return CFX_Color::kTransparent; } -void CPDF_DefaultAppearance::GetColor(FX_ARGB& color, int& iColorType) { - color = 0; +std::pair CPDF_DefaultAppearance::GetColor() { float values[4]; - GetColor(iColorType, values); - if (iColorType == CFX_Color::kTransparent) - return; + CFX_Color::Type type = GetColor(values); + if (type == CFX_Color::kTransparent) + return {type, 0}; - if (iColorType == CFX_Color::kGray) { + if (type == CFX_Color::kGray) { int g = static_cast(values[0] * 255 + 0.5f); - color = ArgbEncode(255, g, g, g); - return; + return {type, ArgbEncode(255, g, g, g)}; } - if (iColorType == CFX_Color::kRGB) { + if (type == CFX_Color::kRGB) { int r = static_cast(values[0] * 255 + 0.5f); int g = static_cast(values[1] * 255 + 0.5f); int b = static_cast(values[2] * 255 + 0.5f); - color = ArgbEncode(255, r, g, b); - return; + return {type, ArgbEncode(255, r, g, b)}; } - if (iColorType == CFX_Color::kCMYK) { + if (type == CFX_Color::kCMYK) { float r = 1.0f - std::min(1.0f, values[0] + values[3]); float g = 1.0f - std::min(1.0f, values[1] + values[3]); float b = 1.0f - std::min(1.0f, values[2] + values[3]); - color = ArgbEncode(255, static_cast(r * 255 + 0.5f), - static_cast(g * 255 + 0.5f), - static_cast(b * 255 + 0.5f)); + return {type, ArgbEncode(255, static_cast(r * 255 + 0.5f), + static_cast(g * 255 + 0.5f), + static_cast(b * 255 + 0.5f))}; } NOTREACHED(); + return {CFX_Color::kTransparent, 0}; } bool CPDF_DefaultAppearance::FindTagParamFromStartForTesting( diff --git a/core/fpdfdoc/cpdf_defaultappearance.h b/core/fpdfdoc/cpdf_defaultappearance.h index 79ad0bb526..79578930e1 100644 --- a/core/fpdfdoc/cpdf_defaultappearance.h +++ b/core/fpdfdoc/cpdf_defaultappearance.h @@ -7,11 +7,13 @@ #ifndef CORE_FPDFDOC_CPDF_DEFAULTAPPEARANCE_H_ #define CORE_FPDFDOC_CPDF_DEFAULTAPPEARANCE_H_ +#include + #include "core/fpdfapi/parser/cpdf_simple_parser.h" -#include "core/fpdfdoc/cpdf_defaultappearance.h" #include "core/fxcrt/fx_coordinates.h" #include "core/fxcrt/fx_string.h" #include "core/fxcrt/fx_system.h" +#include "core/fxge/cfx_color.h" #include "core/fxge/fx_dib.h" enum class BorderStyle { SOLID, DASH, BEVELED, INSET, UNDERLINE }; @@ -27,8 +29,8 @@ class CPDF_DefaultAppearance { ByteString GetFont(float* fFontSize); bool HasColor(); - void GetColor(int& iColorType, float fc[4]); - void GetColor(FX_ARGB& color, int& iColorType); + CFX_Color::Type GetColor(float fc[4]); + std::pair GetColor(); bool FindTagParamFromStartForTesting(CPDF_SimpleParser* parser, const ByteStringView& token, diff --git a/core/fpdfdoc/cpvt_generateap.h b/core/fpdfdoc/cpvt_generateap.h index 6f56993575..52c26af020 100644 --- a/core/fpdfdoc/cpvt_generateap.h +++ b/core/fpdfdoc/cpvt_generateap.h @@ -9,6 +9,7 @@ #include +#include "core/fpdfdoc/cpdf_annot.h" #include "core/fpdfdoc/cpdf_defaultappearance.h" #include "core/fpdfdoc/cpdf_variabletext.h" #include "core/fxcrt/fx_coordinates.h" -- cgit v1.2.3