Browse Source

[3591] Logging::applyDefaultConfiguration() removed.

Tomek Mrugalski 10 years ago
parent
commit
bf2544528e
3 changed files with 7 additions and 23 deletions
  1. 7 7
      src/lib/dhcpsrv/daemon.cc
  2. 0 8
      src/lib/dhcpsrv/logging.cc
  3. 0 8
      src/lib/dhcpsrv/logging.h

+ 7 - 7
src/lib/dhcpsrv/daemon.cc

@@ -61,15 +61,11 @@ void Daemon::configureLogger(const isc::data::ConstElementPtr& log_config,
                              const ConfigurationPtr& storage,
                              const ConfigurationPtr& storage,
                              bool verbose) {
                              bool verbose) {
 
 
-    // This is utility class that translates JSON structures into formats
-    // understandable by log4cplus.
-    LogConfigParser parser(storage);
-
     if (!log_config) {
     if (!log_config) {
         // There was no logger configuration. Let's clear any config
         // There was no logger configuration. Let's clear any config
         // and revert to the default.
         // and revert to the default.
 
 
-        parser.applyDefaultConfiguration(verbose); // Set up default logging
+        isc::log::setDefaultLoggingOutput(verbose); // Set up default logging
         return;
         return;
     }
     }
 
 
@@ -80,10 +76,14 @@ void Daemon::configureLogger(const isc::data::ConstElementPtr& log_config,
         // array in it. Let's clear any old logging configuration
         // array in it. Let's clear any old logging configuration
         // we may have and revert to the default.
         // we may have and revert to the default.
 
 
-        parser.applyDefaultConfiguration(verbose); // Set up default logging
+        isc::log::setDefaultLoggingOutput(verbose); // Set up default logging
         return;
         return;
     }
     }
 
 
+    // This is utility class that translates JSON structures into formats
+    // understandable by log4cplus.
+    LogConfigParser parser(storage);
+
     // Translate JSON structures into log4cplus formats
     // Translate JSON structures into log4cplus formats
     parser.parseConfiguration(loggers, verbose);
     parser.parseConfiguration(loggers, verbose);
 
 
@@ -101,7 +101,7 @@ void Daemon::loggerInit(const char* name, bool verbose) {
                          NULL);
                          NULL);
 
 
     // Apply default configuration (log INFO or DEBUG to stdout)
     // Apply default configuration (log INFO or DEBUG to stdout)
-    LogConfigParser::applyDefaultConfiguration(verbose);
+    isc::log::setDefaultLoggingOutput(verbose);
 }
 }
 
 
 };
 };

+ 0 - 8
src/lib/dhcpsrv/logging.cc

@@ -213,13 +213,5 @@ void LogConfigParser::applyConfiguration() {
     manager.process(specs.begin(), specs.end());
     manager.process(specs.begin(), specs.end());
 }
 }
 
 
-void LogConfigParser::applyDefaultConfiguration(bool verbose) {
-    LoggerSpecification spec(isc::log::getRootLoggerName(),
-                             (verbose?isc::log::DEBUG : isc::log::INFO),
-                             (verbose?99:0));
-
-    setDefaultLoggingOutput(verbose);
-}
-
 } // namespace isc::dhcp
 } // namespace isc::dhcp
 } // namespace isc
 } // namespace isc

+ 0 - 8
src/lib/dhcpsrv/logging.h

@@ -67,14 +67,6 @@ public:
     /// @brief Applies stored configuration
     /// @brief Applies stored configuration
     void applyConfiguration();
     void applyConfiguration();
 
 
-    /// @brief Configures default logging
-    ///
-    /// This method is static, so it can be called in the initial phases of
-    /// the process start-up.
-    ///
-    /// @param verbose specifies verbose mode (true forces DEBUG, debuglevel = 99)
-    static void applyDefaultConfiguration(bool verbose = false);
-
 private:
 private:
 
 
     /// @brief Parses one JSON structure in Logging/loggers" array
     /// @brief Parses one JSON structure in Logging/loggers" array