diff options
author | Dan Sinclair <dsinclair@chromium.org> | 2017-02-23 13:30:34 -0500 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-02-23 18:53:44 +0000 |
commit | b9c69ab7f1338684ffddbc1076052c546f1a95ad (patch) | |
tree | 59e5ecfcfaf1ae28956b725168854c1fb8adf684 /xfa/fxgraphics/cfx_color.cpp | |
parent | efcf362b6658346ae0f8f3cdb73d871fdd82c8af (diff) | |
download | pdfium-b9c69ab7f1338684ffddbc1076052c546f1a95ad.tar.xz |
Cleanup CFX_Graphics.
This Cl removes unused methods, removes unused return values, removes unused
hatch data.
Change-Id: Id76240170a3bd1c3ef156fbda25a6ac96462cbda
Reviewed-on: https://pdfium-review.googlesource.com/2827
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxgraphics/cfx_color.cpp')
-rw-r--r-- | xfa/fxgraphics/cfx_color.cpp | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/xfa/fxgraphics/cfx_color.cpp b/xfa/fxgraphics/cfx_color.cpp index 2044b76054..36025a82c8 100644 --- a/xfa/fxgraphics/cfx_color.cpp +++ b/xfa/fxgraphics/cfx_color.cpp @@ -24,26 +24,23 @@ CFX_Color::~CFX_Color() { m_type = FX_COLOR_None; } -FWL_Error CFX_Color::Set(const FX_ARGB argb) { +void CFX_Color::Set(const FX_ARGB argb) { m_type = FX_COLOR_Solid; m_info.argb = argb; m_info.pattern = nullptr; - return FWL_Error::Succeeded; } -FWL_Error CFX_Color::Set(CFX_Pattern* pattern, const FX_ARGB argb) { +void CFX_Color::Set(CFX_Pattern* pattern, const FX_ARGB argb) { if (!pattern) - return FWL_Error::ParameterInvalid; + return; m_type = FX_COLOR_Pattern; m_info.argb = argb; m_info.pattern = pattern; - return FWL_Error::Succeeded; } -FWL_Error CFX_Color::Set(CFX_Shading* shading) { +void CFX_Color::Set(CFX_Shading* shading) { if (!shading) - return FWL_Error::ParameterInvalid; + return; m_type = FX_COLOR_Shading; m_shading = shading; - return FWL_Error::Succeeded; } |