From b88c91dd3d03f1a452cdd48f0db4e010cb150753 Mon Sep 17 00:00:00 2001
From: Yuri Kunde Schlesner <yuriks@yuriks.net>
Date: Tue, 12 May 2015 02:19:44 -0300
Subject: Common: Remove async logging

It provided a large increase in complexity of the logging system while
having a negligible performance impact: the usage patterns of the ring
buffer meant that each log contended with the logging thread, causing
it to effectively act as a synchronous extra buffering.

Also removed some broken code related to filtering of subclasses which
was broken since it was introduced. (Which means no one ever used that
feature anyway, since, 8 months later, no one ever complained.)
---
 src/common/logging/backend.h | 92 ++++++--------------------------------------
 1 file changed, 12 insertions(+), 80 deletions(-)

(limited to 'src/common/logging/backend.h')

diff --git a/src/common/logging/backend.h b/src/common/logging/backend.h
index 3114f864..c1f4d08e 100644
--- a/src/common/logging/backend.h
+++ b/src/common/logging/backend.h
@@ -4,17 +4,17 @@
 
 #pragma once
 
+#include <chrono>
 #include <cstdarg>
-#include <memory>
-#include <vector>
+#include <string>
+#include <utility>
 
-#include "common/concurrent_ring_buffer.h"
-
-#include "common/logging/filter.h"
 #include "common/logging/log.h"
 
 namespace Log {
 
+class Filter;
+
 /**
  * A log entry. Log entries are store in a structured format to permit more varied output
  * formatting on different frontends, as well as facilitating filtering and aggregation.
@@ -48,89 +48,21 @@ struct Entry {
     }
 };
 
-struct ClassInfo {
-    Class log_class;
-
-    /**
-        * Total number of (direct or indirect) sub classes this class has. If any, they follow in
-        * sequence after this class in the class list.
-        */
-    unsigned int num_children = 0;
-
-    ClassInfo(Class log_class) : log_class(log_class) {}
-};
-
 /**
- * Logging management class. This class has the dual purpose of acting as an exchange point between
- * the logging clients and the log outputter, as well as containing reflection info about available
- * log classes.
+ * Returns the name of the passed log class as a C-string. Subclasses are separated by periods
+ * instead of underscores as in the enumeration.
  */
-class Logger {
-private:
-    using Buffer = Common::ConcurrentRingBuffer<Entry, 16 * 1024 / sizeof(Entry)>;
-
-public:
-    static const size_t QUEUE_CLOSED = Buffer::QUEUE_CLOSED;
-
-    Logger();
-
-    /**
-     * Returns a list of all vector classes and subclasses. The sequence returned is a pre-order of
-     * classes and subclasses, which together with the `num_children` field in ClassInfo, allows
-     * you to recover the hierarchy.
-     */
-    const std::vector<ClassInfo>& GetClasses() const { return all_classes; }
+const char* GetLogClassName(Class log_class);
 
-    /**
-     * Returns the name of the passed log class as a C-string. Subclasses are separated by periods
-     * instead of underscores as in the enumeration.
-     */
-    static const char* GetLogClassName(Class log_class);
-
-    /**
-     * Returns the name of the passed log level as a C-string.
-     */
-    static const char* GetLevelName(Level log_level);
-
-    /**
-     * Appends a messages to the log buffer.
-     * @note This function is thread safe.
-     */
-    void LogMessage(Entry entry);
-
-    /**
-     * Retrieves a batch of messages from the log buffer, blocking until they are available.
-     * @note This function is thread safe.
-     *
-     * @param out_buffer Destination buffer that will receive the log entries.
-     * @param buffer_len The maximum size of `out_buffer`.
-     * @return The number of entries stored. In case the logger is shutting down, `QUEUE_CLOSED` is
-     *         returned, no entries are stored and the logger should shutdown.
-     */
-    size_t GetEntries(Entry* out_buffer, size_t buffer_len);
-
-    /**
-     * Initiates a shutdown of the logger. This will indicate to log output clients that they
-     * should shutdown.
-     */
-    void Close() { ring_buffer.Close(); }
-
-    /**
-     * Returns true if Close() has already been called on the Logger.
-     */
-    bool IsClosed() const { return ring_buffer.IsClosed(); }
-
-private:
-    Buffer ring_buffer;
-    std::vector<ClassInfo> all_classes;
-};
+/**
+ * Returns the name of the passed log level as a C-string.
+ */
+const char* GetLevelName(Level log_level);
 
 /// Creates a log entry by formatting the given source location, and message.
 Entry CreateEntry(Class log_class, Level log_level,
                         const char* filename, unsigned int line_nr, const char* function,
                         const char* format, va_list args);
-/// Initializes the default Logger.
-std::shared_ptr<Logger> InitGlobalLogger();
 
 void SetFilter(Filter* filter);
 
-- 
cgit v1.2.3