Index: xfa/fwl/core/ifwl_checkbox.cpp |
diff --git a/xfa/fwl/core/ifwl_checkbox.cpp b/xfa/fwl/core/ifwl_checkbox.cpp |
index 3aac3598c25d97b76806bff63b50bff60507f72e..04c1c95ea08ce9a260a2f78a3e5fbc68219bb9a1 100644 |
--- a/xfa/fwl/core/ifwl_checkbox.cpp |
+++ b/xfa/fwl/core/ifwl_checkbox.cpp |
@@ -143,20 +143,7 @@ void IFWL_CheckBox::DrawWidget(CFX_Graphics* pGraphics, |
} |
} |
-int32_t IFWL_CheckBox::GetCheckState() { |
- if ((m_pProperties->m_dwStyleExes & FWL_STYLEEXT_CKB_3State) && |
- ((m_pProperties->m_dwStates & FWL_STATE_CKB_CheckMask) == |
- FWL_STATE_CKB_Neutral)) { |
- return 2; |
- } |
- if ((m_pProperties->m_dwStates & FWL_STATE_CKB_CheckMask) == |
- FWL_STATE_CKB_Checked) { |
- return 1; |
- } |
- return 0; |
-} |
- |
-FWL_Error IFWL_CheckBox::SetCheckState(int32_t iCheck) { |
+void IFWL_CheckBox::SetCheckState(int32_t iCheck) { |
m_pProperties->m_dwStates &= ~FWL_STATE_CKB_CheckMask; |
switch (iCheck) { |
case 0: { |
npm
2016/11/14 15:19:30
nit: I think we don't use braces in switch cases?
dsinclair
2016/11/14 16:40:37
Done.
|
@@ -172,10 +159,10 @@ FWL_Error IFWL_CheckBox::SetCheckState(int32_t iCheck) { |
} |
break; |
} |
- default: {} |
+ default: |
+ break; |
} |
Repaint(&m_rtClient); |
- return FWL_Error::Succeeded; |
} |
void IFWL_CheckBox::Layout() { |