remove some unused includes

odygrd · Dec 29, 2024 · b745cc9 · b745cc9
1 parent aecda26
commit b745cc9
Show file tree

Hide file tree

Showing 14 changed files with 10 additions and 24 deletions.
diff --git a/include/quill/Backend.h b/include/quill/Backend.h
@@ -16,7 +16,6 @@
 #include <csignal>
 #include <cstdint>
 #include <cstdlib>
-#include <initializer_list>
 #include <mutex>
 
 QUILL_BEGIN_NAMESPACE

diff --git a/include/quill/CsvWriter.h b/include/quill/CsvWriter.h
@@ -7,11 +7,9 @@
 
 #include "quill/Frontend.h"
 #include "quill/core/Attributes.h"
-#include "quill/core/LoggerBase.h"
 #include "quill/sinks/FileSink.h"
 #include "quill/sinks/Sink.h"
 
-#include <cstdint>
 #include <memory>
 #include <string>
 #include <utility>

diff --git a/include/quill/Frontend.h b/include/quill/Frontend.h
@@ -20,7 +20,6 @@
 
 #include <cassert>
 #include <cstddef>
-#include <cstdint>
 #include <initializer_list>
 #include <memory>
 #include <string>

diff --git a/include/quill/backend/BackendManager.h b/include/quill/backend/BackendManager.h
@@ -25,7 +25,7 @@ namespace detail
 /**
  * Provides access to common collection class that are used by both the frontend and the backend
  * components of the logging system
- * There should only be only active active instance of this class which is achieved by the
+ * There should only be only active instance of this class which is achieved by the
  * LogSystemManagerSingleton
  */
 class BackendManager

diff --git a/include/quill/backend/SignalHandler.h b/include/quill/backend/SignalHandler.h
@@ -89,19 +89,19 @@ class SignalHandlerContext
   }
 
   /***/
-  QUILL_NODISCARD LoggerBase* get_logger() noexcept
+  QUILL_NODISCARD static LoggerBase* get_logger() noexcept
   {
     LoggerBase* logger_base{nullptr};
 
-    if (!SignalHandlerContext::instance().logger_name.empty())
+    if (!instance().logger_name.empty())
     {
-      logger_base = detail::LoggerManager::instance().get_logger(SignalHandlerContext::instance().logger_name);
+      logger_base = LoggerManager::instance().get_logger(instance().logger_name);
     }
 
     // This also checks if the logger was found above
     if (!logger_base || !logger_base->is_valid_logger())
     {
-      logger_base = detail::LoggerManager::instance().get_valid_logger(SignalHandlerContext::excluded_logger_name_substr);
+      logger_base = LoggerManager::instance().get_valid_logger(excluded_logger_name_substr);
     }
 
     return logger_base;

diff --git a/include/quill/backend/TimestampFormatter.h b/include/quill/backend/TimestampFormatter.h
@@ -34,7 +34,7 @@ namespace detail
  * 2) %Qus - Microseconds
  * 3) %Qns - Nanoseconds
  * @note %Qms, %Qus, %Qns specifiers are mutually exclusive
- * e.g given : "%I:%M.%Qms%p" the output would be "03:21.343PM"
+ * e.g. given : "%I:%M.%Qms%p" the output would be "03:21.343PM"
  */
 class TimestampFormatter
 {

diff --git a/include/quill/backend/TransitEvent.h b/include/quill/backend/TransitEvent.h
@@ -87,7 +87,7 @@ struct TransitEvent
 
   uint64_t timestamp{0};
   MacroMetadata const* macro_metadata{nullptr};
-  detail::LoggerBase* logger_base{nullptr};
+  LoggerBase* logger_base{nullptr};
   std::unique_ptr<FormatBuffer> formatted_msg{std::make_unique<FormatBuffer>()}; /** buffer for message **/
   std::unique_ptr<std::vector<std::pair<std::string, std::string>>> named_args; /** A unique ptr to save space as named args feature is not always used */
   std::atomic<bool>* flush_flag{nullptr}; /** This is only used in the case of Event::Flush **/

diff --git a/include/quill/backend/TransitEventBuffer.h b/include/quill/backend/TransitEventBuffer.h
@@ -11,10 +11,6 @@
 #include "quill/core/Attributes.h"
 #include "quill/core/MathUtilities.h"
 
-#include <cassert>
-#include <cstdint>
-#include <vector>
-
 QUILL_BEGIN_NAMESPACE
 
 namespace detail

diff --git a/include/quill/backend/Utf8Conv.h b/include/quill/backend/Utf8Conv.h
@@ -31,8 +31,8 @@ namespace detail
 /**
  * @brief Convert a wide Unicode string to a UTF-8 encoded string.
  *
- * @param input_string_data Pointer to the wide string data.
- * @param input_string_length Length of the wide string.
+ * @param data Pointer to the wide string data.
+ * @param wide_str_len Length of the wide string.
  * @return The UTF-8 encoded string.
  *
  * @remarks If the input string is empty or the conversion fails, an empty string is returned.

diff --git a/include/quill/core/InlinedVector.h b/include/quill/core/InlinedVector.h
@@ -6,10 +6,8 @@
 
 #pragma once
 
-#include <cassert>
 #include <cstddef>
 #include <cstdint>
-#include <cstring>
 #include <type_traits>
 
 #include "quill/core/Attributes.h"

diff --git a/include/quill/core/MacroMetadata.h b/include/quill/core/MacroMetadata.h
@@ -10,7 +10,6 @@
 #include "quill/core/Common.h"
 #include "quill/core/LogLevel.h"
 
-#include <cassert>
 #include <cstddef>
 #include <cstdint>
 #include <string_view>

diff --git a/include/quill/core/PatternFormatterOptions.h b/include/quill/core/PatternFormatterOptions.h
@@ -8,7 +8,7 @@
 
 #include "quill/core/Attributes.h"
 #include "quill/core/Common.h"
-#include <cstdint>
+
 #include <string>
 
 QUILL_BEGIN_NAMESPACE

diff --git a/include/quill/core/UnboundedSPSCQueue.h b/include/quill/core/UnboundedSPSCQueue.h
@@ -9,13 +9,11 @@
 #include "quill/core/Attributes.h"
 #include "quill/core/BoundedSPSCQueue.h"
 #include "quill/core/Common.h"
-#include "quill/core/MathUtilities.h"
 #include "quill/core/QuillError.h"
 
 #include <atomic>
 #include <cassert>
 #include <cstddef>
-#include <cstdint>
 #include <string>
 
 QUILL_BEGIN_NAMESPACE

diff --git a/include/quill/sinks/ConsoleSink.h b/include/quill/sinks/ConsoleSink.h
@@ -8,7 +8,6 @@
 
 #include "quill/core/Attributes.h"
 #include "quill/core/LogLevel.h"
-#include "quill/core/QuillError.h"
 #include "quill/sinks/StreamSink.h"
 
 #include <array>