From: Kefu Chai <kchai@redhat.com>
Date: Sat, 15 May 2021 04:18:07 +0000 (+0800)
Subject: crimson/os: define SeastoreCollection as a class
X-Git-Tag: v17.1.0~1936^2~7
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=30e5d90832f5e827a17db4f16f41b373a2524ee0;p=ceph-ci.git

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 <kchai@redhat.com>
---

diff --git a/src/crimson/os/seastore/seastore.cc b/src/crimson/os/seastore/seastore.cc
index cf9a6cb02b7..b7de87528f1 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 <typename... T>
   SeastoreCollection(T&&... args) :
     FuturizedCollection(std::forward<T>(args)...) {}