[Libreoffice-commits] .: ucb/source

[Libreoffice Gerrit user]

 ucb/source/ucp/webdav/DAVResourceAccess.cxx |    2 +-
 ucb/source/ucp/webdav/SerfLockStore.cxx     |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 282d0a2109b6909df778bafa0ef37a8baa972a80
Author: jailletc36 <christophe.jaillet at wanadoo.fr>
Date:   Sun Nov 4 18:26:53 2012 +0100

    cppCheck: Possible inefficient checking for emptiness
    
    Change-Id: Ief04fa81abbba9e66353c57eb586f864c37deed0
    Signed-off-by: jailletc36 <christophe.jaillet at wanadoo.fr>
    Reviewed-on: https://gerrit.libreoffice.org/986
    Tested-by: Caolán McNamara <caolanm at redhat.com>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/ucb/source/ucp/webdav/DAVResourceAccess.cxx b/ucb/source/ucp/webdav/DAVResourceAccess.cxx
index 2df7248..bbdac2f 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 62e1698..ec53fc5 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();
 }