Patch release 0.8.1

Remove etl::details namespace.

Author: thebashpotato

CHANGELOG

@@ -3,6 +3,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+
+## [0.8.1] - 2024-12-22
+
+### Changed
+
+- Removed internal `etl::detail` namespace, as it was causing issues with `SourceCodeLocation`
+  when using the RUNTIME_INFO macro.
+
 ## [0.8.0] - 2024-12-17
 
 ### Added

etl/examples/blackjack/blackjack.hpp

@@ -29,7 +29,7 @@ class Error : public etl::BaseError
     {
     }
 
-    Error(const std::string_view &msg, const etl::detail::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
+    Error(const std::string_view &msg, const etl::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
     {
     }
 };

etl/include/etl.hpp

@@ -42,9 +42,9 @@ namespace etl
 
 constexpr auto VERSION_MAJOR = 0;
 constexpr auto VERSION_MINOR = 8;
-constexpr auto VERSION_PATCH = 0;
+constexpr auto VERSION_PATCH = 1;
 constexpr auto VERSION = (VERSION_MAJOR * 10000) + (VERSION_MINOR * 100) + VERSION_PATCH;
-constexpr std::string_view VERSION_STRING = "0.8.0";
+constexpr std::string_view VERSION_STRING = "0.8.1";
 
 /// @brief Ditch those old C style for loops and iterate over your enums safely with ranged for loops.
 ///
@@ -444,11 +444,6 @@ template <typename Tag, typename FundamentalType> class TaggedFundamental
     }
 };
 
-/// @brief User of etl should not use anything in the detail namespace, SourceCodeLocation location type only has to be 
-/// used in one of the constructors of an error class which implements etl::BaseError
-namespace detail 
-{
-
 /// @brief Holds useful runtime source code location information for use in Errors.
 ///
 /// @details Should not be used directly, rather the user should pass the `etl::RUNTIME_INFO`
@@ -500,15 +495,13 @@ class SourceCodeLocation
     std::string m_func;
 };
 
-} // namespace detail
-
 /// @brief Wrapper macro which constructs an instance of SourceCodeLocation in-place
 /// and guarantees that the accurate file, function name, and line number will be
 /// reported.
 #ifdef __linux__
-#define RUNTIME_INFO detail::SourceCodeLocation(__FILE__, __LINE__, static_cast<const char *>(__PRETTY_FUNCTION__))
+#define RUNTIME_INFO SourceCodeLocation(__FILE__, __LINE__, static_cast<const char *>(__PRETTY_FUNCTION__))
 #else
-#define RUNTIME_INFO detail::SourceCodeLocation(__FILE__, __LINE__, static_cast<const char *>(__func__))
+#define RUNTIME_INFO SourceCodeLocation(__FILE__, __LINE__, static_cast<const char *>(__func__))
 #endif
 
 /// @brief Base Abstract Error for use with the Result and DynError types.
@@ -531,7 +524,7 @@ class BaseError
     ///
     /// @param `msg` the error message
     /// @param `slc` the source code location object
-    BaseError(const std::string_view &msg, const detail::SourceCodeLocation &slc) noexcept : m_msg(msg)
+    BaseError(const std::string_view &msg, const SourceCodeLocation &slc) noexcept : m_msg(msg)
     {
         m_info.append("Error: ")
             .append(msg)

etl/tests/result_test.cpp

@@ -14,7 +14,7 @@ class GenericError : public etl::BaseError
     {
     }
 
-    GenericError(const std::string_view &msg, const etl::detail::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
+    GenericError(const std::string_view &msg, const etl::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
     {
     }
 };
@@ -26,7 +26,7 @@ class OtherError : public etl::BaseError
     {
     }
 
-    OtherError(const std::string_view &msg, const etl::detail::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
+    OtherError(const std::string_view &msg, const etl::SourceCodeLocation &slc) noexcept : etl::BaseError(msg, slc)
     {
     }
 };

etl/tests/version_test.cpp

@@ -3,12 +3,12 @@
 
 TEST(EtlVersionTests, EnsureVersionStringIsCorrect)
 {
-    ASSERT_EQ(etl::VERSION_STRING, "0.8.0");
+    ASSERT_EQ(etl::VERSION_STRING, "0.8.1");
 }
 
 TEST(EtlVersionTests, EnsureVersionIntIsCorrect)
 {
     ASSERT_EQ(etl::VERSION_MAJOR, 0);
     ASSERT_EQ(etl::VERSION_MINOR, 8);
-    ASSERT_EQ(etl::VERSION_PATCH, 0);
+    ASSERT_EQ(etl::VERSION_PATCH, 1);
 }