diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-19 08:12:17 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-19 10:02:39 +0200 |
commit | 2c1b7e8d6a7fa22cb91919238418816671c3a497 (patch) | |
tree | 6f356017d24dffcd25261295ab25a21b738dc835 /framework | |
parent | ad18bb24d51e4f735085d50c496d28bd637dbb0b (diff) |
clang-tidy readability-container-size-empty
Change-Id: I1df70b7dff5ebb6048f7fc618789faa15ca5d422
Reviewed-on: https://gerrit.libreoffice.org/61967
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'framework')
-rw-r--r-- | framework/source/dispatch/dispatchdisabler.cxx | 2 | ||||
-rw-r--r-- | framework/source/dispatch/interceptionhelper.cxx | 2 | ||||
-rw-r--r-- | framework/source/jobs/jobresult.cxx | 2 | ||||
-rw-r--r-- | framework/source/layoutmanager/toolbarlayoutmanager.cxx | 2 | ||||
-rw-r--r-- | framework/source/services/autorecovery.cxx | 2 | ||||
-rw-r--r-- | framework/source/uielement/thesaurusmenucontroller.cxx | 2 |
6 files changed, 6 insertions, 6 deletions
diff --git a/framework/source/dispatch/dispatchdisabler.cxx b/framework/source/dispatch/dispatchdisabler.cxx index b847031bf742..0ad5932c0251 100644 --- a/framework/source/dispatch/dispatchdisabler.cxx +++ b/framework/source/dispatch/dispatchdisabler.cxx @@ -101,7 +101,7 @@ uno::Type SAL_CALL DispatchDisabler::getElementType() ::sal_Bool SAL_CALL DispatchDisabler::hasElements() { - return maDisabledURLs.size() > 0; + return !maDisabledURLs.empty(); } // XNameAccess diff --git a/framework/source/dispatch/interceptionhelper.cxx b/framework/source/dispatch/interceptionhelper.cxx index 819e96475393..27fb2a14b538 100644 --- a/framework/source/dispatch/interceptionhelper.cxx +++ b/framework/source/dispatch/interceptionhelper.cxx @@ -52,7 +52,7 @@ css::uno::Reference< css::frame::XDispatch > SAL_CALL InterceptionHelper::queryD // b) No match by registration - but a valid interceptor list. // Find first interceptor w/o pattern, so we need to query it - if (!xInterceptor.is() && m_lInterceptionRegs.size()>0) + if (!xInterceptor.is() && !m_lInterceptionRegs.empty()) { for (auto const& lInterceptionReg : m_lInterceptionRegs) { diff --git a/framework/source/jobs/jobresult.cxx b/framework/source/jobs/jobresult.cxx index fe79bf68cdb8..86ae9b5f0dea 100644 --- a/framework/source/jobs/jobresult.cxx +++ b/framework/source/jobs/jobresult.cxx @@ -117,7 +117,7 @@ JobResult::JobResult( /*IN*/ const css::uno::Any& aResult ) css::uno::Sequence<css::beans::NamedValue> aTmp; pIt->second >>= aTmp; comphelper::sequenceToContainer(m_lArguments, aTmp); - if (!m_lArguments.size()) + if (m_lArguments.empty()) m_eParts |= E_ARGUMENTS; } diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.cxx b/framework/source/layoutmanager/toolbarlayoutmanager.cxx index 96e96ba143e6..e374da107921 100644 --- a/framework/source/layoutmanager/toolbarlayoutmanager.cxx +++ b/framework/source/layoutmanager/toolbarlayoutmanager.cxx @@ -3917,7 +3917,7 @@ uno::Sequence< uno::Reference< ui::XUIElement > > ToolbarLayoutManager::getToolb uno::Sequence< uno::Reference< ui::XUIElement > > aSeq; SolarMutexGuard g; - if ( m_aUIElements.size() > 0 ) + if ( !m_aUIElements.empty() ) { sal_uInt32 nCount(0); for (auto const& elem : m_aUIElements) diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx index 14c9148fd92c..844591d56a20 100644 --- a/framework/source/services/autorecovery.cxx +++ b/framework/source/services/autorecovery.cxx @@ -3896,7 +3896,7 @@ void SAL_CALL AutoRecovery::getFastPropertyValue(css::uno::Any& aValue , case AUTORECOVERY_PROPHANDLE_EXISTS_RECOVERYDATA : { bool bSessionData = officecfg::Office::Recovery::RecoveryInfo::SessionData::get(m_xContext); - bool bRecoveryData = m_lDocCache.size() > 0; + bool bRecoveryData = !m_lDocCache.empty(); // exists session data ... => then we can't say, that these // data are valid for recovery. So we have to return sal_False then! diff --git a/framework/source/uielement/thesaurusmenucontroller.cxx b/framework/source/uielement/thesaurusmenucontroller.cxx index de0845440825..a7237ad9c3ec 100644 --- a/framework/source/uielement/thesaurusmenucontroller.cxx +++ b/framework/source/uielement/thesaurusmenucontroller.cxx @@ -77,7 +77,7 @@ void ThesaurusMenuController::fillPopupMenu() VCLXMenu* pAwtMenu = VCLXMenu::GetImplementation( m_xPopupMenu ); Menu* pVCLMenu = pAwtMenu->GetMenu(); pVCLMenu->SetMenuFlags( MenuFlags::NoAutoMnemonics ); - if ( aSynonyms.size() > 0 ) + if ( !aSynonyms.empty() ) { SvtLinguConfig aCfg; Image aImage; |