From: Kefu Chai <kchai@redhat.com>
Date: Sat, 28 Nov 2020 11:15:34 +0000 (+0800)
Subject: mon/ConfigKeyService: move private member variable declaration to the end of class
X-Git-Tag: v16.1.0~361^2~5
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=7f366169f050cba61919c0436be5d5ecab8d4dbc;p=ceph-ci.git

mon/ConfigKeyService: move private member variable declaration to the end of class

to follow https://google.github.io/styleguide/cppguide.html

Signed-off-by: Kefu Chai <kchai@redhat.com>
---

diff --git a/src/mon/ConfigKeyService.h b/src/mon/ConfigKeyService.h
index f40062046a6..e85753daecb 100644
--- a/src/mon/ConfigKeyService.h
+++ b/src/mon/ConfigKeyService.h
@@ -22,25 +22,6 @@ class Monitor;
 
 class ConfigKeyService : public QuorumService
 {
-  Paxos *paxos;
-
-  int store_get(const std::string &key, ceph::buffer::list &bl);
-  void store_put(const std::string &key, ceph::buffer::list &bl, Context *cb = NULL);
-  void store_delete(MonitorDBStore::TransactionRef t, const std::string &key);
-  void store_delete(const std::string &key, Context *cb = NULL);
-  void store_delete_prefix(
-      MonitorDBStore::TransactionRef t,
-      const std::string &prefix);
-  void store_list(std::stringstream &ss);
-  void store_dump(std::stringstream &ss, const std::string& prefix);
-  bool store_exists(const std::string &key);
-  bool store_has_prefix(const std::string &prefix);
-
-  static const std::string STORE_PREFIX;
-
-protected:
-  void service_shutdown() override { }
-
 public:
   ConfigKeyService(Monitor *m, Paxos *p) :
     QuorumService(m),
@@ -80,6 +61,24 @@ public:
   /**
    * @} // ConfigKeyService_Inherited_h
    */
+protected:
+  void service_shutdown() override { }
+private:
+  Paxos *paxos;
+
+  int store_get(const std::string &key, ceph::buffer::list &bl);
+  void store_put(const std::string &key, ceph::buffer::list &bl, Context *cb = NULL);
+  void store_delete(MonitorDBStore::TransactionRef t, const std::string &key);
+  void store_delete(const std::string &key, Context *cb = NULL);
+  void store_delete_prefix(
+      MonitorDBStore::TransactionRef t,
+      const std::string &prefix);
+  void store_list(std::stringstream &ss);
+  void store_dump(std::stringstream &ss, const std::string& prefix);
+  bool store_exists(const std::string &key);
+  bool store_has_prefix(const std::string &prefix);
+
+  static const std::string STORE_PREFIX;
 };
 
 #endif // CEPH_MON_CONFIG_KEY_SERVICE_H