cppCheck: Possible inefficient checking for emptiness

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

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();
 

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();
 }