diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-27 14:49:41 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-28 09:05:11 +0200 |
| commit | 5f19287a9cd4167e3be937e3df820446f2e30518 (patch) | |
| tree | 64f33b2e66cec0aea2972dc7324534eab3113c6f /accessibility | |
| parent | e871c9cb7ce4f3df2ba6780be62ed46b5ee7a410 (diff) | |
loplugin:simplifybool in accessibility..basic
Change-Id: Ibf6cef4baa2d3d400d953ac8bc97a66b5901def9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94972
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'accessibility')
4 files changed, 10 insertions, 10 deletions
diff --git a/accessibility/source/extended/accessibletabbarpagelist.cxx b/accessibility/source/extended/accessibletabbarpagelist.cxx index 680e0e70ba97..7fd901037c07 100644 --- a/accessibility/source/extended/accessibletabbarpagelist.cxx +++ b/accessibility/source/extended/accessibletabbarpagelist.cxx @@ -88,7 +88,7 @@ namespace accessibility void AccessibleTabBarPageList::UpdatePageText( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; Reference< XAccessible > xChild( m_aAccessibleChildren[i] ); @@ -106,7 +106,7 @@ namespace accessibility void AccessibleTabBarPageList::InsertChild( sal_Int32 i ) { - if ( !(i >= 0 && i <= static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i > static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // insert entry in child list @@ -125,7 +125,7 @@ namespace accessibility void AccessibleTabBarPageList::RemoveChild( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // get the accessible of the removed page diff --git a/accessibility/source/standard/accessiblemenubasecomponent.cxx b/accessibility/source/standard/accessiblemenubasecomponent.cxx index d3070d1bacb6..3c5cdd53c71f 100644 --- a/accessibility/source/standard/accessiblemenubasecomponent.cxx +++ b/accessibility/source/standard/accessiblemenubasecomponent.cxx @@ -409,7 +409,7 @@ void OAccessibleMenuBaseComponent::InsertChild( sal_Int32 i ) void OAccessibleMenuBaseComponent::RemoveChild( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // keep the accessible of the removed item diff --git a/accessibility/source/standard/vclxaccessiblestatusbar.cxx b/accessibility/source/standard/vclxaccessiblestatusbar.cxx index ff6351c6bd7c..e84825ea0b3f 100644 --- a/accessibility/source/standard/vclxaccessiblestatusbar.cxx +++ b/accessibility/source/standard/vclxaccessiblestatusbar.cxx @@ -62,7 +62,7 @@ void VCLXAccessibleStatusBar::UpdateShowing( sal_Int32 i, bool bShowing ) void VCLXAccessibleStatusBar::UpdateItemName( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; Reference< XAccessible > xChild( m_aAccessibleChildren[i] ); @@ -80,7 +80,7 @@ void VCLXAccessibleStatusBar::UpdateItemName( sal_Int32 i ) void VCLXAccessibleStatusBar::UpdateItemText( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; Reference< XAccessible > xChild( m_aAccessibleChildren[i] ); @@ -98,7 +98,7 @@ void VCLXAccessibleStatusBar::UpdateItemText( sal_Int32 i ) void VCLXAccessibleStatusBar::InsertChild( sal_Int32 i ) { - if ( !(i >= 0 && i <= static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i > static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // insert entry in child list @@ -117,7 +117,7 @@ void VCLXAccessibleStatusBar::InsertChild( sal_Int32 i ) void VCLXAccessibleStatusBar::RemoveChild( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // get the accessible of the removed page diff --git a/accessibility/source/standard/vclxaccessibletabcontrol.cxx b/accessibility/source/standard/vclxaccessibletabcontrol.cxx index 4842e43b2c6c..29fea97781a6 100644 --- a/accessibility/source/standard/vclxaccessibletabcontrol.cxx +++ b/accessibility/source/standard/vclxaccessibletabcontrol.cxx @@ -116,7 +116,7 @@ void VCLXAccessibleTabControl::UpdateTabPage( sal_Int32 i, bool bNew ) void VCLXAccessibleTabControl::InsertChild( sal_Int32 i ) { - if ( !(i >= 0 && i <= static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i > static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // insert entry in child list @@ -135,7 +135,7 @@ void VCLXAccessibleTabControl::InsertChild( sal_Int32 i ) void VCLXAccessibleTabControl::RemoveChild( sal_Int32 i ) { - if ( !(i >= 0 && i < static_cast<sal_Int32>(m_aAccessibleChildren.size())) ) + if ( i < 0 || i >= static_cast<sal_Int32>(m_aAccessibleChildren.size()) ) return; // get the accessible of the removed page |