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 /ucb | |
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 'ucb')
-rw-r--r-- | ucb/source/ucp/ftp/ftpurl.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ucb/source/ucp/ftp/ftpurl.cxx b/ucb/source/ucp/ftp/ftpurl.cxx index 3c6036115e06..38e4afe3345a 100644 --- a/ucb/source/ucp/ftp/ftpurl.cxx +++ b/ucb/source/ucp/ftp/ftpurl.cxx @@ -226,7 +226,7 @@ void FTPURL::parse(const OUString& url) ""/*aAccount*/); // now check for something like ";type=i" at end of url - if(m_aPathSegmentVec.size() && + if(!m_aPathSegmentVec.empty() && (l = m_aPathSegmentVec.back().indexOf(';')) != -1) { m_aType = m_aPathSegmentVec.back().copy(l); m_aPathSegmentVec.back() = m_aPathSegmentVec.back().copy(0,l); @@ -344,7 +344,7 @@ void FTPURL::child(const OUString& title) OUString FTPURL::child() const { return - m_aPathSegmentVec.size() ? + !m_aPathSegmentVec.empty() ? decodePathSegment(m_aPathSegmentVec.back()) : OUString(); } @@ -742,7 +742,7 @@ OUString FTPURL::ren(const OUString& NewTitle) curl_slist_free_all(slist); if(err != CURLE_OK) throw curl_exception(err); - else if( m_aPathSegmentVec.size() && m_aPathSegmentVec.back() != ".." ) + else if( !m_aPathSegmentVec.empty() && m_aPathSegmentVec.back() != ".." ) m_aPathSegmentVec.back() = encodePathSegment(NewTitle); return OldTitle; } |