commit 2e7997340fb984d25c76b5186d543cbbbdbd77e4
author Brian Silverman <brians> Thu Jun 05 21:50:19 2014 -0700
committer Brian Silverman <brians> Thu Jun 05 21:50:19 2014 -0700
tree 081d21921ed91678303f5515631690d17ee987fb
parent 407ca821f6ad596bd00d29efffe92c5c4d4d2189

sanify logging during startup

aos/common/logging/logging_impl.cc

diff --git a/aos/common/logging/logging_impl.cc b/aos/common/logging/logging_impl.cc
index 8aa66f5..43a20e0 100644
--- a/aos/common/logging/logging_impl.cc
+++ b/aos/common/logging/logging_impl.cc
@@ -20,6 +20,10 @@
 // apply here (mostly the parts about being able to use LOG) because this is the
 // root one.
 class RootLogImplementation : public LogImplementation {
+ public:
+  void have_other_implementation() { only_implementation_ = false; }
+
+ private:
   virtual void set_next(LogImplementation *) {
     LOG(FATAL, "can't have a next logger from here\n");
   }
@@ -29,10 +33,16 @@
     LogMessage message;
     internal::FillInMessage(level, format, ap, &message);
     internal::PrintMessage(stderr, message);
-    fputs("root logger got used, see stderr for message\n", stdout);
+    if (!only_implementation_) {
+      fputs("root logger got used, see stderr for message\n", stdout);
+    }
   }
+
+  bool only_implementation_ = true;
 };
 
+RootLogImplementation *root_implementation = nullptr;
+
 void SetGlobalImplementation(LogImplementation *implementation) {
   Context *context = Context::Get();
 
@@ -45,7 +55,7 @@
 }
 
 void *DoInit() {
-  SetGlobalImplementation(new RootLogImplementation());
+  SetGlobalImplementation(root_implementation = new RootLogImplementation());
 
   if (pthread_atfork(NULL /*prepare*/, NULL /*parent*/,
                      NewContext /*child*/) != 0) {
@@ -295,6 +305,7 @@
     implementation->set_next(old);
   }
   SetGlobalImplementation(implementation);
+  root_implementation->have_other_implementation();
 }
 
 void Init() {