diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-19 09:13:03 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-19 13:26:27 +0100 |
| commit | be8c414567f49242164b1fdfb12764b16be355c1 (patch) | |
| tree | c1f505272ec27e5e069f5d6964dc4f7b20150a5a /ucb | |
| parent | be94207ecb88a9005ee6624e354d70c9613d7653 (diff) | |
loplugin:unusedmethods also check for functions returning bool
we were previously excluding them
Change-Id: I48a68799b0de60b4995fae541eb363e043d4dd11
Reviewed-on: https://gerrit.libreoffice.org/48167
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'ucb')
| -rw-r--r-- | ucb/source/ucp/ftp/ftpcontentprovider.cxx | 6 | ||||
| -rw-r--r-- | ucb/source/ucp/ftp/ftpcontentprovider.hxx | 2 |
2 files changed, 3 insertions, 5 deletions
diff --git a/ucb/source/ucp/ftp/ftpcontentprovider.cxx b/ucb/source/ucp/ftp/ftpcontentprovider.cxx index 359259ac94ad..c74b0ae7cf10 100644 --- a/ucb/source/ucp/ftp/ftpcontentprovider.cxx +++ b/ucb/source/ucp/ftp/ftpcontentprovider.cxx @@ -212,7 +212,7 @@ CURL* FTPContentProvider::handle() } -bool FTPContentProvider::forHost( const OUString& host, +void FTPContentProvider::forHost( const OUString& host, const OUString& port, const OUString& username, OUString& password, @@ -226,10 +226,8 @@ bool FTPContentProvider::forHost( const OUString& host, { password = i.password; account = i.account; - return true; + return; } - - return false; } bool FTPContentProvider::setHost( const OUString& host, diff --git a/ucb/source/ucp/ftp/ftpcontentprovider.hxx b/ucb/source/ucp/ftp/ftpcontentprovider.hxx index 8aa6e7dfcaea..f7ac598c69fc 100644 --- a/ucb/source/ucp/ftp/ftpcontentprovider.hxx +++ b/ucb/source/ucp/ftp/ftpcontentprovider.hxx @@ -80,7 +80,7 @@ namespace ftp /** host is in the form host:port. */ - bool forHost(const OUString& host, + void forHost(const OUString& host, const OUString& port, const OUString& username, OUString& password, |