From: Philippe Coval <philippe.coval@osg.samsung.com>
Date: Mon, 13 Mar 2017 08:39:46 +0000 (+0100)
Subject: resource: Disable logging if not enabled
X-Git-Tag: 1.3.0~416
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=be77876f05d7e18bc5838b5b8ef13d33d951e16e;hp=293024229f8aa721c3ddff5469c30849d28cfbfa;p=platform%2Fupstream%2Fiotivity.git

resource: Disable logging if not enabled

Curently without logging (LOGGING=False or undefined)
some trace messages are appearing:

  0:
  In entity handler wrapper:

Now with this change those log lines are sent to null stream.

Change-Id: I26892c7ca6efca025c24e762305d59589ca26543
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/17869
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
---

diff --git a/resource/oc_logger/cpp/oc_ostream_logger.cpp b/resource/oc_logger/cpp/oc_ostream_logger.cpp
index 8134940..270a4c1 100644
--- a/resource/oc_logger/cpp/oc_ostream_logger.cpp
+++ b/resource/oc_logger/cpp/oc_ostream_logger.cpp
@@ -67,7 +67,14 @@ try
  auto *target = reinterpret_cast<std::ostream *>(world);
 
  if(nullptr == world)
+ {
+#ifdef TB_LOG
   target = &std::cout;
+#else
+  static std::ostream nullstream(0);
+  target = &nullstream;
+#endif
+ }
 
  oc_ostream_logger_ctx *my_ctx = new oc_ostream_logger_ctx(target);
 
diff --git a/resource/src/OCApi.cpp b/resource/src/OCApi.cpp
index 529573f..bc5c969 100644
--- a/resource/src/OCApi.cpp
+++ b/resource/src/OCApi.cpp
@@ -28,7 +28,12 @@ namespace OC
 {
     std::ostream& oclog()
     {
+#ifdef TB_LOG
         return std::cout;
+#else
+        static std::ostream nullstream(0);
+        return nullstream;
+#endif
     }
 }
 #endif