From 30e5d90832f5e827a17db4f16f41b373a2524ee0 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sat, 15 May 2021 12:18:07 +0800 Subject: [PATCH] crimson/os: define SeastoreCollection as a class to be consistent with the forward declaration. C++ standard does not differentiate class from struct in this perspective. but Clang warngs at seeing it. so silence the warning. Signed-off-by: Kefu Chai --- src/crimson/os/seastore/seastore.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/crimson/os/seastore/seastore.cc b/src/crimson/os/seastore/seastore.cc index cf9a6cb02b7de..b7de87528f18b 100644 --- a/src/crimson/os/seastore/seastore.cc +++ b/src/crimson/os/seastore/seastore.cc @@ -33,7 +33,8 @@ SeaStore::~SeaStore() { perf_service->remove_from_collection(); } -struct SeastoreCollection final : public FuturizedCollection { +class SeastoreCollection final : public FuturizedCollection { +public: template SeastoreCollection(T&&... args) : FuturizedCollection(std::forward(args)...) {} -- 2.39.5