RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-01-04 02:02:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

common/config: change md_config_obs_impl's template param to ConfigProxy

Browse Source 
we were using LockPolicy as this tempalte class's template parameter,
but turns out ConfigProxy is a better parameter for it. as ConfigProxy is
"closer" to md_config_obs_impl than LockPolicy is.

Signed-off-by: Kefu Chai <kchai@redhat.com>
This commit is contained in:
Kefu Chai 2018-07-18 15:57:46 +08:00
parent 7909c08fe0
commit 8435aa9bda
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/common/config_fwd.h
View File

@ -5,10 +5,10 @@
#include "lock_policy.h"
namespace ceph::internal {
template<LockPolicy lp> class md_config_obs_impl;
template<class ConfigProxy> class md_config_obs_impl;
}
class md_config_t;
using md_config_obs_t =
ceph::internal::md_config_obs_impl<ceph::internal::LockPolicy::MUTEX>;
struct md_config_t;
class ConfigProxy;
using md_config_obs_t =
ceph::internal::md_config_obs_impl<ConfigProxy>;

2
src/common/config_obs.h
View File

@ -28,7 +28,7 @@ namespace ceph::internal {
* Subscribe for configuration changes by calling the md_config_t::add_observer() method
* and unsubscribe using md_config_t::remove_observer().
*/
template<LockPolicy lp>
template<class ConfigProxy>
class md_config_obs_impl {
public:
virtual ~md_config_obs_impl() {}