From: Kefu Chai <kchai@redhat.com>
Date: Fri, 24 Aug 2018 16:04:12 +0000 (+0800)
Subject: common/config: cleanup config_fwd.h
X-Git-Tag: v14.0.1~345^2~2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=283c739b1abd8342077f6b6607a5138ddfa17064;p=ceph.git

common/config: cleanup config_fwd.h

* define an alias for ceph::common::ConfigProxy
* remove obs bits from config_fwd.h, as the users of obs will always
  need to include config_obs.h anyway.

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

diff --git a/src/common/config.h b/src/common/config.h
index 503e57b14739a..76886cd2d1d8d 100644
--- a/src/common/config.h
+++ b/src/common/config.h
@@ -23,7 +23,6 @@
 #include "log/SubsystemMap.h"
 #include "common/options.h"
 #include "common/subsys_types.h"
-#include "common/config_fwd.h"
 #include "common/config_tracker.h"
 #include "common/config_values.h"
 
diff --git a/src/common/config_fwd.h b/src/common/config_fwd.h
index 7b7ff28e5366e..817d1a0f8b033 100644
--- a/src/common/config_fwd.h
+++ b/src/common/config_fwd.h
@@ -2,12 +2,11 @@
 
 #pragma once
 
-#include "lock_policy.h"
-
-namespace ceph {
-template<class ConfigProxy> class md_config_obs_impl;
+#ifdef WITH_SEASTAR
+namespace ceph::common {
+  class ConfigProxy;
 }
-
-struct md_config_t;
+using ConfigProxy = ceph::common::ConfigProxy;
+#else
 class ConfigProxy;
-using md_config_obs_t = ceph::md_config_obs_impl<ConfigProxy>;
+#endif
diff --git a/src/common/config_obs.h b/src/common/config_obs.h
index 0cbbdb37397a7..20d12ad83343a 100644
--- a/src/common/config_obs.h
+++ b/src/common/config_obs.h
@@ -44,4 +44,7 @@ public:
 				    const std::set<int>& changed) { }
 };
 }
+
+using md_config_obs_t = ceph::md_config_obs_impl<ConfigProxy>;
+
 #endif
diff --git a/src/common/config_proxy.h b/src/common/config_proxy.h
index d6f75b10b87f4..e9b6b39b6263a 100644
--- a/src/common/config_proxy.h
+++ b/src/common/config_proxy.h
@@ -4,7 +4,6 @@
 
 #include <type_traits>
 #include "common/config.h"
-#include "common/config_fwd.h"
 #include "common/config_obs.h"
 #include "common/config_obs_mgr.h"
 #include "common/Mutex.h"
@@ -17,6 +16,7 @@ class ConfigProxy {
    * The current values of all settings described by the schema
    */
   ConfigValues values;
+  using md_config_obs_t = ceph::md_config_obs_impl<ConfigProxy>;
   ObserverMgr<md_config_obs_t> obs_mgr;
   md_config_t config;
   /** A lock that protects the md_config_t internals. It is
diff --git a/src/common/config_values.h b/src/common/config_values.h
index b5f10b6cb47ef..2614d2ef15978 100644
--- a/src/common/config_values.h
+++ b/src/common/config_values.h
@@ -8,7 +8,6 @@
 #include <string>
 #include <utility>
 
-#include "common/config_fwd.h"
 #include "common/entity_name.h"
 #include "common/options.h"
 #include "log/SubsystemMap.h"
diff --git a/src/mon/MonMap.h b/src/mon/MonMap.h
index 5b6e2702c5023..0bde9f014ba38 100644
--- a/src/mon/MonMap.h
+++ b/src/mon/MonMap.h
@@ -15,11 +15,15 @@
 #ifndef CEPH_MONMAP_H
 #define CEPH_MONMAP_H
 
-#include "include/err.h"
 
-#include "msg/Message.h"
+#include "common/config_fwd.h"
+
+#include "include/err.h"
 #include "include/types.h"
+
 #include "mon/mon_types.h"
+#include "msg/Message.h"
+
 
 namespace ceph {
   class Formatter;