summaryrefslogtreecommitdiffstats
path: root/codemaker/source/cppumaker
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2017-01-31 14:46:38 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2017-02-01 12:15:22 +0000
commit2489000d3fd66319a8355fd4e37cfdfda47296d0 (patch)
treecaad79e7b5bec3863604b20190b682c0d73d2b25 /codemaker/source/cppumaker
parentgpg4libre: Support building --with-system-gpgme (Linux only so far) (diff)
downloadcore-2489000d3fd66319a8355fd4e37cfdfda47296d0.tar.gz
core-2489000d3fd66319a8355fd4e37cfdfda47296d0.zip
loplugin:useuniqueptr extend to check local vars
just the simple and obvious case for now, of a local var being allocated and deleted inside a single local block, and the delete happening at the end of the block Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'codemaker/source/cppumaker')
-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 47aab6c356c9..6432baee05d6 100644
--- a/codemaker/source/cppumaker/cpputype.cxx
+++ b/codemaker/source/cppumaker/cpputype.cxx
@@ -3359,9 +3359,9 @@ void includeExceptions(
if (node->present) {
includes.add(node->name);
} else {
- for (codemaker::ExceptionTreeNode* pChild : node->children)
+ for (std::unique_ptr<codemaker::ExceptionTreeNode> const & pChild : node->children)
{
- includeExceptions(includes, pChild);
+ includeExceptions(includes, pChild.get());
}
}
}
@@ -3713,9 +3713,9 @@ void ServiceType::dumpCatchClauses(
out << indent() << "throw;\n";
dec();
} else {
- for (codemaker::ExceptionTreeNode* pChild : node->children)
+ for (std::unique_ptr<codemaker::ExceptionTreeNode> const & pChild : node->children)
{
- dumpCatchClauses(out, pChild);
+ dumpCatchClauses(out, pChild.get());
}
}
}