Don't call the destructor on the base class.
Not only is this unnecessary (since the base-class destructor is
called automatically), but it is probably undefined behaviour, as
it leads to the base-class destructor being called twice.
Although clang doesn't complain about this, gcc gives a compilation
error (albeit with a highly confusing message).
Test: No compilation error from gcc.
Change-Id: I5043630335d0b09b08d4272794f292ed98541b87
Signed-off-by: Alec Edgington <alec.m.edgington@gmail.com>
diff --git a/base/HidlSupport.cpp b/base/HidlSupport.cpp
index a69faa2..eb4368f 100644
--- a/base/HidlSupport.cpp
+++ b/base/HidlSupport.cpp
@@ -315,9 +315,7 @@
: hidl_memory(name, std::move(handle), size) {}
// it's required to have at least one out-of-line method to avoid weak vtable
-HidlMemory::~HidlMemory() {
- hidl_memory::~hidl_memory();
-}
+HidlMemory::~HidlMemory() {}
} // namespace hardware
} // namespace android