diff options
Diffstat (limited to 'vespalog/src')
-rw-r--r-- | vespalog/src/test/bufferedlogskiptest.cpp | 10 | ||||
-rw-r--r-- | vespalog/src/test/bufferedlogtest.cpp | 10 | ||||
-rw-r--r-- | vespalog/src/test/bufferedlogtest.logger1.cpp | 2 | ||||
-rw-r--r-- | vespalog/src/test/bufferedlogtest.logger2.cpp | 2 | ||||
-rw-r--r-- | vespalog/src/test/threads/testthreads.cpp | 2 | ||||
-rw-r--r-- | vespalog/src/vespa/log/bufferedlogger.h | 1 | ||||
-rw-r--r-- | vespalog/src/vespa/log/log.cpp | 2 | ||||
-rw-r--r-- | vespalog/src/vespa/log/log.h | 5 |
8 files changed, 9 insertions, 25 deletions
diff --git a/vespalog/src/test/bufferedlogskiptest.cpp b/vespalog/src/test/bufferedlogskiptest.cpp index 5b386d18e90..cb16b73fa24 100644 --- a/vespalog/src/test/bufferedlogskiptest.cpp +++ b/vespalog/src/test/bufferedlogskiptest.cpp @@ -1,18 +1,12 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#include <vespa/log/log.h> +#include <vespa/log/bufferedlogger.h> #include <fstream> #include <iostream> -#include <sstream> - -#include <sys/types.h> -#include <stdlib.h> #include <unistd.h> -#include <signal.h> -LOG_SETUP("bufferedlogskiptest", - "$Id$"); +LOG_SETUP("bufferedlogskiptest"); std::string readFile(const std::string& file) { std::ostringstream ost; diff --git a/vespalog/src/test/bufferedlogtest.cpp b/vespalog/src/test/bufferedlogtest.cpp index 0e27bd65988..97f70ebe004 100644 --- a/vespalog/src/test/bufferedlogtest.cpp +++ b/vespalog/src/test/bufferedlogtest.cpp @@ -1,21 +1,15 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#include <vespa/log/log.h> +#include <vespa/log/bufferedlogger.h> #include "bufferedlogtest.logger1.h" #include "bufferedlogtest.logger2.h" #include <fstream> #include <iostream> -#include <sstream> - -#include <sys/types.h> -#include <stdlib.h> #include <unistd.h> -#include <signal.h> -LOG_SETUP("bufferedlogtest", - "$Id$"); +LOG_SETUP("bufferedlogtest"); std::string readFile(const std::string& file) { std::ostringstream ost; diff --git a/vespalog/src/test/bufferedlogtest.logger1.cpp b/vespalog/src/test/bufferedlogtest.logger1.cpp index 992de19fea5..5af4b9489a4 100644 --- a/vespalog/src/test/bufferedlogtest.logger1.cpp +++ b/vespalog/src/test/bufferedlogtest.logger1.cpp @@ -1,6 +1,6 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#include <vespa/log/log.h> +#include <vespa/log/bufferedlogger.h> #include "bufferedlogtest.logger1.h" LOG_SETUP(".logger1"); diff --git a/vespalog/src/test/bufferedlogtest.logger2.cpp b/vespalog/src/test/bufferedlogtest.logger2.cpp index 688ecd153aa..ac6935c2c68 100644 --- a/vespalog/src/test/bufferedlogtest.logger2.cpp +++ b/vespalog/src/test/bufferedlogtest.logger2.cpp @@ -1,6 +1,6 @@ // Copyright 2016 Yahoo Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -#include <vespa/log/log.h> +#include <vespa/log/bufferedlogger.h> #include "bufferedlogtest.logger2.h" LOG_SETUP(".logger2"); diff --git a/vespalog/src/test/threads/testthreads.cpp b/vespalog/src/test/threads/testthreads.cpp index 056f5aad2a2..90b8c13e497 100644 --- a/vespalog/src/test/threads/testthreads.cpp +++ b/vespalog/src/test/threads/testthreads.cpp @@ -2,7 +2,7 @@ #include <vespa/fastos/app.h> #include <vespa/fastos/time.h> #include <vespa/fastos/thread.h> -#include <vespa/log/log.h> +#include <vespa/log/bufferedlogger.h> #include <iostream> using std::string; diff --git a/vespalog/src/vespa/log/bufferedlogger.h b/vespalog/src/vespa/log/bufferedlogger.h index 7ca8d799840..9ab02f7842f 100644 --- a/vespalog/src/vespa/log/bufferedlogger.h +++ b/vespalog/src/vespa/log/bufferedlogger.h @@ -198,7 +198,6 @@ public: /** Trim the buffer. Removing old messages if wanted. */ void trimCache(); - }; } // ns_log diff --git a/vespalog/src/vespa/log/log.cpp b/vespalog/src/vespa/log/log.cpp index 7e92008504e..c31d110d241 100644 --- a/vespalog/src/vespa/log/log.cpp +++ b/vespalog/src/vespa/log/log.cpp @@ -10,6 +10,7 @@ LOG_SETUP_INDIRECT(".log", "$Id$"); #include "log-target.h" #include "internal.h" #include "control-file.h" +#include "bufferedlogger.h" #include <vespa/fastos/thread.h> @@ -222,6 +223,7 @@ Logger::doLog(LogLevel level, const char *file, int line, const char *fmt, ...) actualSize = tryLog(sizeofPayload, level, file, line, fmt, args); va_end(args); } while (sizeofPayload < actualSize); + ns_log::BufferedLogger::logger.trimCache(); } void Logger::doLogCore(uint64_t timestamp, LogLevel level, diff --git a/vespalog/src/vespa/log/log.h b/vespalog/src/vespa/log/log.h index 6ecd69de792..53a0eec140c 100644 --- a/vespalog/src/vespa/log/log.h +++ b/vespalog/src/vespa/log/log.h @@ -38,14 +38,12 @@ static int log_dummmy __attribute__((unused)) = logger.setRcsId(x) do { \ if (logger.wants(ns_log::Logger::level)) { \ logger.doLog(ns_log::Logger::level, __FILE__, __LINE__, __VA_ARGS__); \ - ns_log::BufferedLogger::logger.trimCache(); \ } \ } while (0) #define VLOG(level, ...) \ do { \ if (logger.wants(level)) { \ logger.doLog(level, __FILE__, __LINE__, __VA_ARGS__); \ - ns_log::BufferedLogger::logger.trimCache(); \ } \ } while (0) #endif @@ -302,6 +300,3 @@ extern void log_abort(const char *message, #else #define LOG_ASSERT(expr) #endif // #ifndef NDEBUG - -#include <vespa/log/bufferedlogger.h> - |