diff options
| author | jailletc36 <christophe.jaillet@wanadoo.fr> | 2012-11-04 18:26:53 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2012-11-06 14:12:52 +0000 |
| commit | 282d0a2109b6909df778bafa0ef37a8baa972a80 (patch) | |
| tree | 9ca9f9ba26faa47c7a203f4c01f6346d4505d735 /ucb | |
| parent | 739f99e2eaa0f3271a9471b6fe776c0dc370eacb (diff) | |
cppCheck: Possible inefficient checking for emptiness
Change-Id: Ief04fa81abbba9e66353c57eb586f864c37deed0
Signed-off-by: jailletc36 <christophe.jaillet@wanadoo.fr>
Reviewed-on: https://gerrit.libreoffice.org/986
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'ucb')
| -rw-r--r-- | ucb/source/ucp/webdav/DAVResourceAccess.cxx | 2 | ||||
| -rw-r--r-- | ucb/source/ucp/webdav/SerfLockStore.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/ucb/source/ucp/webdav/DAVResourceAccess.cxx b/ucb/source/ucp/webdav/DAVResourceAccess.cxx index 2df72486525d..bbdac2f644fb 100644 --- a/ucb/source/ucp/webdav/DAVResourceAccess.cxx +++ b/ucb/source/ucp/webdav/DAVResourceAccess.cxx @@ -1121,7 +1121,7 @@ sal_Bool DAVResourceAccess::detectRedirectCycle( void DAVResourceAccess::resetUri() { osl::Guard< osl::Mutex > aGuard( m_aMutex ); - if ( m_aRedirectURIs.size() > 0 ) + if ( ! m_aRedirectURIs.empty() ) { std::vector< SerfUri >::const_iterator it = m_aRedirectURIs.begin(); diff --git a/ucb/source/ucp/webdav/SerfLockStore.cxx b/ucb/source/ucp/webdav/SerfLockStore.cxx index 62e1698b2950..ec53fc57f6b2 100644 --- a/ucb/source/ucp/webdav/SerfLockStore.cxx +++ b/ucb/source/ucp/webdav/SerfLockStore.cxx @@ -191,7 +191,7 @@ void SerfLockStore::removeLock( SerfLock * pLock ) m_aLockInfoMap.erase( pLock ); ne_lockstore_remove( m_pSerfLockStore, pLock ); - if ( m_aLockInfoMap.size() == 0 ) + if ( m_aLockInfoMap.empty() ) stopTicker(); } |