From af494c65c19bcf5bff5cd25d68d9f46bfa96bfae Mon Sep 17 00:00:00 2001 From: Leon Scroggins III Date: Tue, 20 Sep 2022 17:05:18 -0400 Subject: [PATCH] Remove virtual from ~ComposerClientWriter This class has no other virtual methods, and it has no subclasses, so it's unclear why it had a virtual destructor. Bug: NA Test: make Change-Id: Ic87fe6596b6d796ca11822de53af2e4e521f127d --- .../hardware/graphics/composer3/ComposerClientWriter.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h b/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h index 1d81f7b602..775ae9fa88 100644 --- a/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h +++ b/graphics/composer/aidl/include/android/hardware/graphics/composer3/ComposerClientWriter.h @@ -59,13 +59,13 @@ using aidl::android::hardware::common::NativeHandle; namespace aidl::android::hardware::graphics::composer3 { -class ComposerClientWriter { +class ComposerClientWriter final { public: static constexpr std::optional kNoTimestamp = std::nullopt; ComposerClientWriter() { reset(); } - virtual ~ComposerClientWriter() { reset(); } + ~ComposerClientWriter() { reset(); } void reset() { mDisplayCommand.reset();