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 /codemaker | |
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 'codemaker')
-rw-r--r-- | codemaker/source/cppumaker/cpputype.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/codemaker/source/cppumaker/cpputype.cxx b/codemaker/source/cppumaker/cpputype.cxx index 22711d444126..15e1f116d638 100644 --- a/codemaker/source/cppumaker/cpputype.cxx +++ b/codemaker/source/cppumaker/cpputype.cxx @@ -162,7 +162,7 @@ public: void dump(CppuOptions const & options); - bool dumpFile( + void dumpFile( OUString const & uri, OUString const & name, bool hpp, CppuOptions const & options); @@ -410,7 +410,7 @@ void CppuType::dump(CppuOptions const & options) options.isValid("-O") ? b2u(options.getOption("-O")) : "", options); } -bool CppuType::dumpFile( +void CppuType::dumpFile( OUString const & uri, OUString const & name, bool hpp, CppuOptions const & options) { @@ -422,7 +422,7 @@ bool CppuType::dumpFile( } bool exists = fileExists(u2b(fileUri)); if (exists && options.isValid("-G")) { - return false; + return; } FileStream out; out.createTempFile(getTempDir(u2b(fileUri))); @@ -449,7 +449,7 @@ bool CppuType::dumpFile( throw; } out.close(); - return makeValidTypeFile( + makeValidTypeFile( u2b(fileUri), u2b(tmpUri), exists && options.isValid("-Gc")); } |