From 559f974dbb92e35a1bfb739032d82a2536fcf59c Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Tue, 21 Nov 2017 22:03:20 +0000 Subject: Cleanup CXFA_StrokeData This CL fixes return types and makes methods consts. Change-Id: I97da09a491d10760d6adf4efcc0557130cf8b405 Reviewed-on: https://pdfium-review.googlesource.com/19110 Reviewed-by: Henrique Nakashima Commit-Queue: dsinclair --- xfa/fxfa/parser/cxfa_strokedata.cpp | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'xfa/fxfa/parser/cxfa_strokedata.cpp') diff --git a/xfa/fxfa/parser/cxfa_strokedata.cpp b/xfa/fxfa/parser/cxfa_strokedata.cpp index edbac24237..0845719c9e 100644 --- a/xfa/fxfa/parser/cxfa_strokedata.cpp +++ b/xfa/fxfa/parser/cxfa_strokedata.cpp @@ -10,18 +10,19 @@ #include "xfa/fxfa/parser/cxfa_node.h" #include "xfa/fxfa/parser/xfa_utils.h" -int32_t CXFA_StrokeData::GetPresence() const { - return m_pNode ? m_pNode->JSNode()->GetEnum(XFA_Attribute::Presence) - : XFA_ATTRIBUTEENUM_Invisible; +bool CXFA_StrokeData::IsVisible() const { + return m_pNode ? m_pNode->JSNode()->GetEnum(XFA_Attribute::Presence) == + XFA_ATTRIBUTEENUM_Visible + : false; } -int32_t CXFA_StrokeData::GetCapType() const { +XFA_ATTRIBUTEENUM CXFA_StrokeData::GetCapType() const { if (!m_pNode) return XFA_ATTRIBUTEENUM_Square; return m_pNode->JSNode()->GetEnum(XFA_Attribute::Cap); } -int32_t CXFA_StrokeData::GetStrokeType() const { +XFA_ATTRIBUTEENUM CXFA_StrokeData::GetStrokeType() const { return m_pNode ? m_pNode->JSNode()->GetEnum(XFA_Attribute::Stroke) : XFA_ATTRIBUTEENUM_Solid; } @@ -70,7 +71,7 @@ void CXFA_StrokeData::SetColor(FX_ARGB argb) { false); } -int32_t CXFA_StrokeData::GetJoinType() const { +XFA_ATTRIBUTEENUM CXFA_StrokeData::GetJoinType() const { return m_pNode ? m_pNode->JSNode()->GetEnum(XFA_Attribute::Join) : XFA_ATTRIBUTEENUM_Square; } -- cgit v1.2.3