[moco.log] Add source-generic configure method (#3984)
author박종현/On-Device Lab(SR)/Staff Engineer/삼성전자 <jh1302.park@samsung.com>
Wed, 26 Jun 2019 07:59:56 +0000 (16:59 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Wed, 26 Jun 2019 07:59:56 +0000 (16:59 +0900)
moco::LoggerConfig now has source-generic configure method (not used
yet).

Signed-off-by: Jonghyun Park <jh1302.park@samsung.com>
contrib/moco-log/include/moco/Log.h
contrib/moco-log/src/Log.cpp

index 04fc4dc..3a6a1ad 100644 (file)
@@ -46,6 +46,8 @@ public:
   LoggerConfig();
 
 public:
+  // TODO Mark this method as override (or final) after hermes migration
+  void configure(const hermes::Source *, hermes::Source::Setting &) const;
   void configure(const Logger *, hermes::Source::Setting &) const;
 
 private:
index 52f8e2e..63dd20b 100644 (file)
@@ -57,6 +57,15 @@ LoggerConfig::LoggerConfig()
   _enabled = safecast<bool>(std::getenv("MOCO_LOG"), false);
 }
 
+void LoggerConfig::configure(const hermes::Source *source, hermes::Source::Setting &setting) const
+{
+  // Let's ignore hermes::Sources if that is not a moco logger
+  if (auto logger = dynamic_cast<const Logger *>(source))
+  {
+    configure(logger, setting);
+  }
+}
+
 void LoggerConfig::configure(const Logger *, hermes::Source::Setting &setting) const
 {
   if (_enabled)