diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2014-09-15 17:06:07 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2014-09-15 17:12:07 +0200 |
| commit | c94f43c9da74dbcc12f35e13b64c68c54be35421 (patch) | |
| tree | 9e88bec68060ccb4ec18b6c61e87c9fdbcc8c680 /o3tl | |
| parent | bdca9f352bd6732e04d86930fefe49f7faaf72b8 (diff) | |
Mark o3tl::heap_ptr as SAL_WARN_UNUSED
...to make it more straightforward to fix blunder as in
a88b2d841856026ea1a0539d670e07fafd296c6e "Resolves: fdo#81039 crash on adding
new macro library."
Conceivably, instances of heap_ptr could legitimately be created only for ctor/
dtor effect, but then again uses of heap_ptr should arguably be replaced with
std::unique_ptr anyway.
Change-Id: I9c765b8d265f3c26d459ec276e335eeebc8fee91
Diffstat (limited to 'o3tl')
| -rw-r--r-- | o3tl/qa/test-heap_ptr.cxx | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/o3tl/qa/test-heap_ptr.cxx b/o3tl/qa/test-heap_ptr.cxx index f863b6cd3f8b..4fbfe1957f7a 100644 --- a/o3tl/qa/test-heap_ptr.cxx +++ b/o3tl/qa/test-heap_ptr.cxx @@ -138,6 +138,7 @@ class heap_ptr_test : public CppUnit::TestFixture { heap_ptr<Help> t_raii( new Help(55) ); + (void)t_raii; CPPUNIT_ASSERT_MESSAGE("raii1", Help::InstanceCount_() == nHelpCount + 1); } CPPUNIT_ASSERT_MESSAGE("raii2", Help::InstanceCount_() == nHelpCount); |