summaryrefslogtreecommitdiff
path: root/codemaker
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2018-01-19 09:13:03 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-01-19 13:26:27 +0100
commitbe8c414567f49242164b1fdfb12764b16be355c1 (patch)
treec1f505272ec27e5e069f5d6964dc4f7b20150a5a /codemaker
parentbe94207ecb88a9005ee6624e354d70c9613d7653 (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.cxx8
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"));
}