aboutsummaryrefslogtreecommitdiff
path: root/src/zencore/include
diff options
context:
space:
mode:
Diffstat (limited to 'src/zencore/include')
-rw-r--r--src/zencore/include/zencore/base64.h4
-rw-r--r--src/zencore/include/zencore/blockingqueue.h2
-rw-r--r--src/zencore/include/zencore/compactbinaryfile.h1
-rw-r--r--src/zencore/include/zencore/compactbinaryvalue.h24
-rw-r--r--src/zencore/include/zencore/filesystem.h8
-rw-r--r--src/zencore/include/zencore/hashutils.h4
-rw-r--r--src/zencore/include/zencore/iobuffer.h37
-rw-r--r--src/zencore/include/zencore/jobqueue.h7
-rw-r--r--src/zencore/include/zencore/logbase.h137
-rw-r--r--src/zencore/include/zencore/logging.h210
-rw-r--r--src/zencore/include/zencore/logging/ansicolorsink.h33
-rw-r--r--src/zencore/include/zencore/logging/asyncsink.h30
-rw-r--r--src/zencore/include/zencore/logging/formatter.h20
-rw-r--r--src/zencore/include/zencore/logging/helpers.h122
-rw-r--r--src/zencore/include/zencore/logging/logger.h74
-rw-r--r--src/zencore/include/zencore/logging/logmsg.h66
-rw-r--r--src/zencore/include/zencore/logging/memorybuffer.h11
-rw-r--r--src/zencore/include/zencore/logging/messageonlyformatter.h22
-rw-r--r--src/zencore/include/zencore/logging/msvcsink.h30
-rw-r--r--src/zencore/include/zencore/logging/nullsink.h17
-rw-r--r--src/zencore/include/zencore/logging/registry.h74
-rw-r--r--src/zencore/include/zencore/logging/sink.h34
-rw-r--r--src/zencore/include/zencore/logging/tracesink.h27
-rw-r--r--src/zencore/include/zencore/md5.h2
-rw-r--r--src/zencore/include/zencore/meta.h1
-rw-r--r--src/zencore/include/zencore/mpscqueue.h20
-rw-r--r--src/zencore/include/zencore/process.h43
-rw-r--r--src/zencore/include/zencore/sentryintegration.h9
-rw-r--r--src/zencore/include/zencore/sharedbuffer.h13
-rw-r--r--src/zencore/include/zencore/string.h47
-rw-r--r--src/zencore/include/zencore/system.h39
-rw-r--r--src/zencore/include/zencore/testing.h7
-rw-r--r--src/zencore/include/zencore/testutils.h27
-rw-r--r--src/zencore/include/zencore/thread.h16
-rw-r--r--src/zencore/include/zencore/trace.h1
-rw-r--r--src/zencore/include/zencore/varint.h1
-rw-r--r--src/zencore/include/zencore/xxhash.h2
-rw-r--r--src/zencore/include/zencore/zencore.h34
38 files changed, 1007 insertions, 249 deletions
diff --git a/src/zencore/include/zencore/base64.h b/src/zencore/include/zencore/base64.h
index 4d78b085f..08d9f3043 100644
--- a/src/zencore/include/zencore/base64.h
+++ b/src/zencore/include/zencore/base64.h
@@ -11,7 +11,11 @@ struct Base64
template<typename CharType>
static uint32_t Encode(const uint8_t* Source, uint32_t Length, CharType* Dest);
+ template<typename CharType>
+ static bool Decode(const CharType* Source, uint32_t Length, uint8_t* Dest, uint32_t& OutLength);
+
static inline constexpr int32_t GetEncodedDataSize(uint32_t Size) { return ((Size + 2) / 3) * 4; }
+ static inline constexpr int32_t GetMaxDecodedDataSize(uint32_t Length) { return (Length / 4) * 3; }
};
} // namespace zen
diff --git a/src/zencore/include/zencore/blockingqueue.h b/src/zencore/include/zencore/blockingqueue.h
index d205b6ec3..6ac43b1ac 100644
--- a/src/zencore/include/zencore/blockingqueue.h
+++ b/src/zencore/include/zencore/blockingqueue.h
@@ -2,6 +2,8 @@
#pragma once
+#include <zencore/zencore.h> // For ZEN_ASSERT
+
#include <atomic>
#include <chrono>
#include <condition_variable>
diff --git a/src/zencore/include/zencore/compactbinaryfile.h b/src/zencore/include/zencore/compactbinaryfile.h
index 00c37e941..33f3e7bea 100644
--- a/src/zencore/include/zencore/compactbinaryfile.h
+++ b/src/zencore/include/zencore/compactbinaryfile.h
@@ -1,4 +1,5 @@
// Copyright Epic Games, Inc. All Rights Reserved.
+#pragma once
#include <zencore/compactbinary.h>
#include <zencore/iohash.h>
diff --git a/src/zencore/include/zencore/compactbinaryvalue.h b/src/zencore/include/zencore/compactbinaryvalue.h
index aa2d2821d..4ce8009b8 100644
--- a/src/zencore/include/zencore/compactbinaryvalue.h
+++ b/src/zencore/include/zencore/compactbinaryvalue.h
@@ -128,17 +128,21 @@ CbValue::AsString(CbFieldError* OutError, std::string_view Default) const
uint32_t ValueSizeByteCount;
const uint64_t ValueSize = ReadVarUInt(Chars, ValueSizeByteCount);
- if (OutError)
+ if (ValueSize >= (uint64_t(1) << 31))
{
- if (ValueSize >= (uint64_t(1) << 31))
+ if (OutError)
{
*OutError = CbFieldError::RangeError;
- return Default;
}
+ return Default;
+ }
+
+ if (OutError)
+ {
*OutError = CbFieldError::None;
}
- return std::string_view(Chars + ValueSizeByteCount, int32_t(ValueSize));
+ return std::string_view(Chars + ValueSizeByteCount, size_t(ValueSize));
}
inline std::u8string_view
@@ -148,17 +152,21 @@ CbValue::AsU8String(CbFieldError* OutError, std::u8string_view Default) const
uint32_t ValueSizeByteCount;
const uint64_t ValueSize = ReadVarUInt(Chars, ValueSizeByteCount);
- if (OutError)
+ if (ValueSize >= (uint64_t(1) << 31))
{
- if (ValueSize >= (uint64_t(1) << 31))
+ if (OutError)
{
*OutError = CbFieldError::RangeError;
- return Default;
}
+ return Default;
+ }
+
+ if (OutError)
+ {
*OutError = CbFieldError::None;
}
- return std::u8string_view(Chars + ValueSizeByteCount, int32_t(ValueSize));
+ return std::u8string_view(Chars + ValueSizeByteCount, size_t(ValueSize));
}
inline uint64_t
diff --git a/src/zencore/include/zencore/filesystem.h b/src/zencore/include/zencore/filesystem.h
index 16e2b59f8..6dc159a83 100644
--- a/src/zencore/include/zencore/filesystem.h
+++ b/src/zencore/include/zencore/filesystem.h
@@ -187,6 +187,14 @@ void ScanFile(void* NativeHandle,
void WriteFile(void* NativeHandle, const void* Data, uint64_t Size, uint64_t FileOffset, uint64_t ChunkSize, std::error_code& Ec);
void ReadFile(void* NativeHandle, void* Data, uint64_t Size, uint64_t FileOffset, uint64_t ChunkSize, std::error_code& Ec);
+// Interface for sub-file range cloning on filesystems that support copy-on-write.
+// GetCloneQueryInterface() returns nullptr on platforms without range clone support.
+//
+// Platform capabilities:
+// Windows (ReFS) - True CoW range cloning via FSCTL_DUPLICATE_EXTENTS_TO_FILE.
+// Linux (Btrfs/XFS) - True CoW range cloning via FICLONERANGE ioctl.
+// macOS (APFS) - Not implemented. No sub-file range clone API exists.
+// Whole-file CoW cloning is available via TryCloneFile (clonefile syscall).
class CloneQueryInterface
{
public:
diff --git a/src/zencore/include/zencore/hashutils.h b/src/zencore/include/zencore/hashutils.h
index 4e877e219..8abfd4b6e 100644
--- a/src/zencore/include/zencore/hashutils.h
+++ b/src/zencore/include/zencore/hashutils.h
@@ -2,6 +2,10 @@
#pragma once
+#include <cstddef>
+#include <functional>
+#include <type_traits>
+
namespace zen {
template<typename T>
diff --git a/src/zencore/include/zencore/iobuffer.h b/src/zencore/include/zencore/iobuffer.h
index 182768ff6..82c201edd 100644
--- a/src/zencore/include/zencore/iobuffer.h
+++ b/src/zencore/include/zencore/iobuffer.h
@@ -426,22 +426,39 @@ private:
class IoBufferBuilder
{
public:
- static IoBuffer MakeFromFile(const std::filesystem::path& FileName, uint64_t Offset = 0, uint64_t Size = ~0ull);
- static IoBuffer MakeFromTemporaryFile(const std::filesystem::path& FileName);
- static IoBuffer MakeFromFileHandle(void* FileHandle, uint64_t Offset = 0, uint64_t Size = ~0ull);
- /** Make sure buffer data is memory resident, but avoid memory mapping data from files
- */
- static IoBuffer ReadFromFileMaybe(const IoBuffer& InBuffer);
- inline static IoBuffer MakeFromMemory(MemoryView Memory) { return IoBuffer(IoBuffer::Wrap, Memory.GetData(), Memory.GetSize()); }
- inline static IoBuffer MakeCloneFromMemory(const void* Ptr, size_t Sz)
+ static IoBuffer MakeFromFile(const std::filesystem::path& FileName,
+ uint64_t Offset = 0,
+ uint64_t Size = ~0ull,
+ ZenContentType ContentType = ZenContentType::kBinary);
+ static IoBuffer MakeFromTemporaryFile(const std::filesystem::path& FileName, ZenContentType ContentType = ZenContentType::kBinary);
+ static IoBuffer MakeFromFileHandle(void* FileHandle,
+ uint64_t Offset = 0,
+ uint64_t Size = ~0ull,
+ ZenContentType ContentType = ZenContentType::kBinary);
+ inline static IoBuffer MakeFromMemory(MemoryView Memory, ZenContentType ContentType = ZenContentType::kBinary)
+ {
+ IoBuffer NewBuffer(IoBuffer::Wrap, Memory.GetData(), Memory.GetSize());
+ NewBuffer.SetContentType(ContentType);
+ return NewBuffer;
+ }
+ inline static IoBuffer MakeCloneFromMemory(const void* Ptr, size_t Sz, ZenContentType ContentType = ZenContentType::kBinary)
{
if (Sz)
{
- return IoBuffer(IoBuffer::Clone, Ptr, Sz);
+ IoBuffer NewBuffer(IoBuffer::Clone, Ptr, Sz);
+ NewBuffer.SetContentType(ContentType);
+ return NewBuffer;
}
return {};
}
- inline static IoBuffer MakeCloneFromMemory(MemoryView Memory) { return MakeCloneFromMemory(Memory.GetData(), Memory.GetSize()); }
+ inline static IoBuffer MakeCloneFromMemory(MemoryView Memory, ZenContentType ContentType = ZenContentType::kBinary)
+ {
+ return MakeCloneFromMemory(Memory.GetData(), Memory.GetSize(), ContentType);
+ }
+
+ /** Make sure buffer data is memory resident, but avoid memory mapping data from files
+ */
+ static IoBuffer ReadFromFileMaybe(const IoBuffer& InBuffer);
};
void iobuffer_forcelink();
diff --git a/src/zencore/include/zencore/jobqueue.h b/src/zencore/include/zencore/jobqueue.h
index d348bd021..6631e5766 100644
--- a/src/zencore/include/zencore/jobqueue.h
+++ b/src/zencore/include/zencore/jobqueue.h
@@ -25,7 +25,11 @@ public:
virtual bool IsCancelled() const = 0;
virtual void ReportMessage(std::string_view Message) = 0;
// virtual void ReportProgress(std::string_view CurrentOp, uint32_t CurrentOpPercentComplete) = 0;
- virtual void ReportProgress(std::string_view CurrentOp, std::string_view Details, ptrdiff_t TotalCount, ptrdiff_t RemainingCount) = 0;
+ virtual void ReportProgress(std::string_view CurrentOp,
+ std::string_view Details,
+ ptrdiff_t TotalCount,
+ ptrdiff_t RemainingCount,
+ uint64_t ElapsedTimeMs) = 0;
};
class JobError : public std::runtime_error
@@ -62,6 +66,7 @@ public:
std::string CurrentOpDetails;
ptrdiff_t TotalCount;
ptrdiff_t RemainingCount;
+ uint64_t ProgressElapsedTimeMs;
std::vector<std::string> Messages;
std::string AbortReason;
};
diff --git a/src/zencore/include/zencore/logbase.h b/src/zencore/include/zencore/logbase.h
index 00af68b0a..046e96db3 100644
--- a/src/zencore/include/zencore/logbase.h
+++ b/src/zencore/include/zencore/logbase.h
@@ -2,98 +2,113 @@
#pragma once
-#include <string_view>
-
-#define ZEN_LOG_LEVEL_TRACE 0
-#define ZEN_LOG_LEVEL_DEBUG 1
-#define ZEN_LOG_LEVEL_INFO 2
-#define ZEN_LOG_LEVEL_WARN 3
-#define ZEN_LOG_LEVEL_ERROR 4
-#define ZEN_LOG_LEVEL_CRITICAL 5
-#define ZEN_LOG_LEVEL_OFF 6
-
-#define ZEN_LEVEL_NAME_TRACE std::string_view("trace", 5)
-#define ZEN_LEVEL_NAME_DEBUG std::string_view("debug", 5)
-#define ZEN_LEVEL_NAME_INFO std::string_view("info", 4)
-#define ZEN_LEVEL_NAME_WARNING std::string_view("warning", 7)
-#define ZEN_LEVEL_NAME_ERROR std::string_view("error", 5)
-#define ZEN_LEVEL_NAME_CRITICAL std::string_view("critical", 8)
-#define ZEN_LEVEL_NAME_OFF std::string_view("off", 3)
+#include <zenbase/refcount.h>
-namespace zen::logging::level {
+#include <atomic>
+#include <string_view>
+namespace zen::logging {
enum LogLevel : int
{
- Trace = ZEN_LOG_LEVEL_TRACE,
- Debug = ZEN_LOG_LEVEL_DEBUG,
- Info = ZEN_LOG_LEVEL_INFO,
- Warn = ZEN_LOG_LEVEL_WARN,
- Err = ZEN_LOG_LEVEL_ERROR,
- Critical = ZEN_LOG_LEVEL_CRITICAL,
- Off = ZEN_LOG_LEVEL_OFF,
+ Trace,
+ Debug,
+ Info,
+ Warn,
+ Err,
+ Critical,
+ Off,
LogLevelCount
};
LogLevel ParseLogLevelString(std::string_view String);
std::string_view ToStringView(LogLevel Level);
-} // namespace zen::logging::level
+void SetLogLevel(LogLevel NewLogLevel);
+LogLevel GetLogLevel();
-namespace zen::logging {
+struct SourceLocation
+{
+ constexpr SourceLocation() = default;
+ constexpr SourceLocation(const char* InFilename, int InLine) : Filename(InFilename), Line(InLine) {}
-void SetLogLevel(level::LogLevel NewLogLevel);
-level::LogLevel GetLogLevel();
+ constexpr operator bool() const noexcept { return Line != 0; }
-} // namespace zen::logging
+ const char* Filename{nullptr};
+ int Line{0};
+};
-namespace spdlog {
-class logger;
-}
+/** This encodes the constant parts of a log message which can be emitted once
+ * and then referred to by log events.
+ *
+ * It's *critical* that instances of this struct are permanent and never
+ * destroyed, as log messages will refer to them by pointer. The easiest way
+ * to ensure this is to create them as function-local statics.
+ *
+ * The logging macros already do this for you so this should not be something
+ * you normally would need to worry about.
+ */
+struct LogPoint
+{
+ SourceLocation Location;
+ LogLevel Level;
+ std::string_view FormatString;
+};
-namespace zen::logging {
+class Logger;
-struct SourceLocation
+/** This is the base class for all loggers
+
+ There's only one derived type - `Logger`. The reason this
+ exists is just to try and hide the full Logger definition
+ but once that is lightweight enough it may not be necessary
+ to have this split
+ */
+class LoggerBase : public TRefCounted<Logger>
{
- constexpr SourceLocation() = default;
- constexpr SourceLocation(const char* filename_in, int line_in, const char* funcname_in)
- : filename(filename_in)
- , line(line_in)
- , funcname(funcname_in)
- {
- }
-
- constexpr bool empty() const noexcept { return line == 0; }
-
- // IMPORTANT NOTE: the layout of this class must match the spdlog::source_loc class
- // since we currently pass a pointer to it into spdlog after casting it to
- // spdlog::source_loc*
- //
- // This is intended to be an intermediate state, before we (probably) transition off
- // spdlog entirely
-
- const char* filename{nullptr};
- int line{0};
- const char* funcname{nullptr};
+public:
+ bool ShouldLog(LogLevel InLevel) const { return InLevel >= m_Level.load(std::memory_order_relaxed); }
+
+ void SetLevel(LogLevel InLevel) { m_Level.store(InLevel, std::memory_order_relaxed); }
+ LogLevel GetLevel() const { return m_Level.load(std::memory_order_relaxed); }
+
+ void SetFlushLevel(LogLevel InLevel) { m_FlushLevel.store(InLevel, std::memory_order_relaxed); }
+ LogLevel GetFlushLevel() const { return m_FlushLevel.load(std::memory_order_relaxed); }
+
+protected:
+ std::atomic<LogLevel> m_Level{Info};
+ std::atomic<LogLevel> m_FlushLevel{Off};
};
} // namespace zen::logging
namespace zen {
+// Lightweight non-owning handle to a Logger. Loggers are owned by the Registry
+// via Ref<Logger>; LoggerRef exists as a cheap (raw pointer) handle that can be
+// stored in members and passed through logging macros without requiring the
+// complete Logger type or incurring refcount overhead on every log call.
struct LoggerRef
{
LoggerRef() = default;
- LoggerRef(spdlog::logger& InLogger) : SpdLogger(&InLogger) {}
+ explicit LoggerRef(logging::Logger& InLogger);
+ // This exists so that logging macros can pass LoggerRef or LogCategory
+ // to ZEN_LOG without needing to know which one it is
LoggerRef Logger() { return *this; }
- bool ShouldLog(int Level) const;
- inline operator bool() const { return SpdLogger != nullptr; }
+ inline operator bool() const { return m_Logger != nullptr; }
+
+ inline logging::Logger* operator->() const;
+ inline logging::Logger& operator*() const;
+
+ bool ShouldLog(logging::LogLevel Level) const { return m_Logger && m_Logger->ShouldLog(Level); }
+ void SetLogLevel(logging::LogLevel NewLogLevel) { m_Logger->SetLevel(NewLogLevel); }
+ logging::LogLevel GetLogLevel() { return m_Logger->GetLevel(); }
- void SetLogLevel(logging::level::LogLevel NewLogLevel);
- logging::level::LogLevel GetLogLevel();
+ void Flush();
- spdlog::logger* SpdLogger = nullptr;
+private:
+ logging::LoggerBase* m_Logger = nullptr;
};
} // namespace zen
diff --git a/src/zencore/include/zencore/logging.h b/src/zencore/include/zencore/logging.h
index 74a44d028..dc5e05656 100644
--- a/src/zencore/include/zencore/logging.h
+++ b/src/zencore/include/zencore/logging.h
@@ -9,16 +9,9 @@
#if ZEN_PLATFORM_WINDOWS
# define ZEN_LOG_SECTION(Id) ZEN_DATA_SECTION(Id)
-# pragma section(".zlog$f", read)
# pragma section(".zlog$l", read)
-# pragma section(".zlog$m", read)
-# pragma section(".zlog$s", read)
-# define ZEN_DECLARE_FUNCTION static constinit ZEN_LOG_SECTION(".zlog$f") char FuncName[] = __FUNCTION__;
-# define ZEN_LOG_FUNCNAME FuncName
#else
# define ZEN_LOG_SECTION(Id)
-# define ZEN_DECLARE_FUNCTION
-# define ZEN_LOG_FUNCNAME static_cast<const char*>(__func__)
#endif
namespace zen::logging {
@@ -37,34 +30,29 @@ LoggerRef ErrorLog();
void SetErrorLog(std::string_view LoggerId);
LoggerRef Get(std::string_view Name);
-void ConfigureLogLevels(level::LogLevel Level, std::string_view Loggers);
+void ConfigureLogLevels(LogLevel Level, std::string_view Loggers);
void RefreshLogLevels();
-void RefreshLogLevels(level::LogLevel DefaultLevel);
-
+void RefreshLogLevels(LogLevel DefaultLevel);
+
+/** LogCategory allows for the creation of log categories that can be used with
+ * the logging macros just like a logger reference. The main purpose of this is
+ * to allow for static log categories in global scope where we can't actually
+ * go ahead and instantiate a logger immediately because the logging system may
+ * not be initialized yet.
+ */
struct LogCategory
{
- inline LogCategory(std::string_view InCategory) : CategoryName(InCategory) {}
-
- inline zen::LoggerRef Logger()
- {
- if (LoggerRef)
- {
- return LoggerRef;
- }
+ inline LogCategory(std::string_view InCategory) : m_CategoryName(InCategory) {}
- LoggerRef = zen::logging::Get(CategoryName);
- return LoggerRef;
- }
+ LoggerRef Logger();
- std::string CategoryName;
- zen::LoggerRef LoggerRef;
+private:
+ std::string m_CategoryName;
+ LoggerRef m_LoggerRef;
};
-void EmitConsoleLogMessage(int LogLevel, std::string_view Format, fmt::format_args Args);
-void EmitLogMessage(LoggerRef& Logger, int LogLevel, std::string_view Message);
-void EmitLogMessage(LoggerRef& Logger, const SourceLocation& Location, int LogLevel, std::string_view Message);
-void EmitLogMessage(LoggerRef& Logger, int LogLevel, std::string_view Format, fmt::format_args Args);
-void EmitLogMessage(LoggerRef& Logger, const SourceLocation& Location, int LogLevel, std::string_view Format, fmt::format_args Args);
+void EmitConsoleLogMessage(const LogPoint& Lp, fmt::format_args Args);
+void EmitLogMessage(LoggerRef& Logger, const LogPoint& Lp, fmt::format_args Args);
template<typename... T>
auto
@@ -79,14 +67,17 @@ namespace zen {
extern LoggerRef TheDefaultLogger;
+/**
+ * This is the default logger, which any ZEN_INFO et al will get if there's
+ * no Log() function declared in the current scope.
+ *
+ * Typically, classes which want to log to its own channel will declare a Log()
+ * member function which returns a LoggerRef created at construction time.
+ */
inline LoggerRef
Log()
{
- if (TheDefaultLogger)
- {
- return TheDefaultLogger;
- }
- return zen::logging::ConsoleLog();
+ return TheDefaultLogger;
}
using logging::ConsoleLog;
@@ -98,12 +89,6 @@ using zen::ConsoleLog;
using zen::ErrorLog;
using zen::Log;
-inline consteval bool
-LogIsErrorLevel(int LogLevel)
-{
- return (LogLevel == zen::logging::level::Err || LogLevel == zen::logging::level::Critical);
-};
-
#if ZEN_BUILD_DEBUG
# define ZEN_CHECK_FORMAT_STRING(fmtstr, ...) \
while (false) \
@@ -117,75 +102,66 @@ LogIsErrorLevel(int LogLevel)
}
#endif
-#define ZEN_LOG_WITH_LOCATION(InLogger, InLevel, fmtstr, ...) \
- do \
- { \
- using namespace std::literals; \
- ZEN_DECLARE_FUNCTION \
- static constinit ZEN_LOG_SECTION(".zlog$s") char FileName[] = __FILE__; \
- static constinit ZEN_LOG_SECTION(".zlog$m") char FormatString[] = fmtstr; \
- static constinit ZEN_LOG_SECTION(".zlog$l") zen::logging::SourceLocation Location{FileName, __LINE__, ZEN_LOG_FUNCNAME}; \
- zen::LoggerRef Logger = InLogger; \
- ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
- if (Logger.ShouldLog(InLevel)) \
- { \
- zen::logging::EmitLogMessage(Logger, \
- Location, \
- InLevel, \
- std::string_view(FormatString, sizeof FormatString - 1), \
- zen::logging::LogCaptureArguments(__VA_ARGS__)); \
- } \
+#define ZEN_LOG_WITH_LOCATION(InLogger, InLevel, fmtstr, ...) \
+ do \
+ { \
+ using namespace std::literals; \
+ static constinit ZEN_LOG_SECTION(".zlog$l") \
+ zen::logging::LogPoint LogPoint{zen::logging::SourceLocation{__FILE__, __LINE__}, InLevel, std::string_view(fmtstr)}; \
+ zen::LoggerRef Logger = InLogger; \
+ ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
+ if (Logger.ShouldLog(InLevel)) \
+ { \
+ zen::logging::EmitLogMessage(Logger, LogPoint, zen::logging::LogCaptureArguments(__VA_ARGS__)); \
+ } \
} while (false);
-#define ZEN_LOG(InLogger, InLevel, fmtstr, ...) \
- do \
- { \
- using namespace std::literals; \
- static constinit ZEN_LOG_SECTION(".zlog$m") char FormatString[] = fmtstr; \
- zen::LoggerRef Logger = InLogger; \
- ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
- if (Logger.ShouldLog(InLevel)) \
- { \
- zen::logging::EmitLogMessage(Logger, \
- InLevel, \
- std::string_view(FormatString, sizeof FormatString - 1), \
- zen::logging::LogCaptureArguments(__VA_ARGS__)); \
- } \
+#define ZEN_LOG(InLogger, InLevel, fmtstr, ...) \
+ do \
+ { \
+ using namespace std::literals; \
+ static constinit ZEN_LOG_SECTION(".zlog$l") zen::logging::LogPoint LogPoint{{}, InLevel, std::string_view(fmtstr)}; \
+ zen::LoggerRef Logger = InLogger; \
+ ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
+ if (Logger.ShouldLog(InLevel)) \
+ { \
+ zen::logging::EmitLogMessage(Logger, LogPoint, zen::logging::LogCaptureArguments(__VA_ARGS__)); \
+ } \
} while (false);
#define ZEN_DEFINE_LOG_CATEGORY_STATIC(Category, Name) \
static zen::logging::LogCategory Category { Name }
-#define ZEN_LOG_TRACE(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::level::Trace, fmtstr, ##__VA_ARGS__)
-#define ZEN_LOG_DEBUG(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::level::Debug, fmtstr, ##__VA_ARGS__)
-#define ZEN_LOG_INFO(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::level::Info, fmtstr, ##__VA_ARGS__)
-#define ZEN_LOG_WARN(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::level::Warn, fmtstr, ##__VA_ARGS__)
-#define ZEN_LOG_ERROR(Category, fmtstr, ...) ZEN_LOG_WITH_LOCATION(Category.Logger(), zen::logging::level::Err, fmtstr, ##__VA_ARGS__)
-#define ZEN_LOG_CRITICAL(Category, fmtstr, ...) \
- ZEN_LOG_WITH_LOCATION(Category.Logger(), zen::logging::level::Critical, fmtstr, ##__VA_ARGS__)
-
-#define ZEN_TRACE(fmtstr, ...) ZEN_LOG(Log(), zen::logging::level::Trace, fmtstr, ##__VA_ARGS__)
-#define ZEN_DEBUG(fmtstr, ...) ZEN_LOG(Log(), zen::logging::level::Debug, fmtstr, ##__VA_ARGS__)
-#define ZEN_INFO(fmtstr, ...) ZEN_LOG(Log(), zen::logging::level::Info, fmtstr, ##__VA_ARGS__)
-#define ZEN_WARN(fmtstr, ...) ZEN_LOG(Log(), zen::logging::level::Warn, fmtstr, ##__VA_ARGS__)
-#define ZEN_ERROR(fmtstr, ...) ZEN_LOG_WITH_LOCATION(Log(), zen::logging::level::Err, fmtstr, ##__VA_ARGS__)
-#define ZEN_CRITICAL(fmtstr, ...) ZEN_LOG_WITH_LOCATION(Log(), zen::logging::level::Critical, fmtstr, ##__VA_ARGS__)
-
-#define ZEN_CONSOLE_LOG(InLevel, fmtstr, ...) \
- do \
- { \
- using namespace std::literals; \
- ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
- zen::logging::EmitConsoleLogMessage(InLevel, fmtstr, zen::logging::LogCaptureArguments(__VA_ARGS__)); \
+#define ZEN_LOG_TRACE(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::Trace, fmtstr, ##__VA_ARGS__)
+#define ZEN_LOG_DEBUG(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::Debug, fmtstr, ##__VA_ARGS__)
+#define ZEN_LOG_INFO(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::Info, fmtstr, ##__VA_ARGS__)
+#define ZEN_LOG_WARN(Category, fmtstr, ...) ZEN_LOG(Category.Logger(), zen::logging::Warn, fmtstr, ##__VA_ARGS__)
+#define ZEN_LOG_ERROR(Category, fmtstr, ...) ZEN_LOG_WITH_LOCATION(Category.Logger(), zen::logging::Err, fmtstr, ##__VA_ARGS__)
+#define ZEN_LOG_CRITICAL(Category, fmtstr, ...) ZEN_LOG_WITH_LOCATION(Category.Logger(), zen::logging::Critical, fmtstr, ##__VA_ARGS__)
+
+#define ZEN_TRACE(fmtstr, ...) ZEN_LOG(Log(), zen::logging::Trace, fmtstr, ##__VA_ARGS__)
+#define ZEN_DEBUG(fmtstr, ...) ZEN_LOG(Log(), zen::logging::Debug, fmtstr, ##__VA_ARGS__)
+#define ZEN_INFO(fmtstr, ...) ZEN_LOG(Log(), zen::logging::Info, fmtstr, ##__VA_ARGS__)
+#define ZEN_WARN(fmtstr, ...) ZEN_LOG(Log(), zen::logging::Warn, fmtstr, ##__VA_ARGS__)
+#define ZEN_ERROR(fmtstr, ...) ZEN_LOG_WITH_LOCATION(Log(), zen::logging::Err, fmtstr, ##__VA_ARGS__)
+#define ZEN_CRITICAL(fmtstr, ...) ZEN_LOG_WITH_LOCATION(Log(), zen::logging::Critical, fmtstr, ##__VA_ARGS__)
+
+#define ZEN_CONSOLE_LOG(InLevel, fmtstr, ...) \
+ do \
+ { \
+ using namespace std::literals; \
+ static constinit ZEN_LOG_SECTION(".zlog$l") zen::logging::LogPoint LogPoint{{}, InLevel, std::string_view(fmtstr)}; \
+ ZEN_CHECK_FORMAT_STRING(fmtstr##sv, ##__VA_ARGS__); \
+ zen::logging::EmitConsoleLogMessage(LogPoint, zen::logging::LogCaptureArguments(__VA_ARGS__)); \
} while (false)
-#define ZEN_CONSOLE(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Info, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_TRACE(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Trace, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_DEBUG(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Debug, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_INFO(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Info, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_WARN(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Warn, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_ERROR(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Err, fmtstr, ##__VA_ARGS__)
-#define ZEN_CONSOLE_CRITICAL(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::level::Critical, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Info, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_TRACE(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Trace, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_DEBUG(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Debug, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_INFO(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Info, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_WARN(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Warn, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_ERROR(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Err, fmtstr, ##__VA_ARGS__)
+#define ZEN_CONSOLE_CRITICAL(fmtstr, ...) ZEN_CONSOLE_LOG(zen::logging::Critical, fmtstr, ##__VA_ARGS__)
//////////////////////////////////////////////////////////////////////////
@@ -240,28 +216,28 @@ std::string_view EmitActivitiesForLogging(StringBuilderBase& OutString);
#define ZEN_LOG_SCOPE(...) ScopedLazyActivity $Activity##__LINE__([&](StringBuilderBase& Out) { Out << fmt::format(__VA_ARGS__); })
-#define ZEN_SCOPED_WARN(fmtstr, ...) \
- do \
- { \
- ExtendableStringBuilder<256> ScopeString; \
- const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
- ZEN_LOG(Log(), zen::logging::level::Warn, fmtstr "{}", ##__VA_ARGS__, Scopes); \
+#define ZEN_SCOPED_WARN(fmtstr, ...) \
+ do \
+ { \
+ ExtendableStringBuilder<256> ScopeString; \
+ const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
+ ZEN_LOG(Log(), zen::logging::Warn, fmtstr "{}", ##__VA_ARGS__, Scopes); \
} while (false)
-#define ZEN_SCOPED_ERROR(fmtstr, ...) \
- do \
- { \
- ExtendableStringBuilder<256> ScopeString; \
- const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
- ZEN_LOG_WITH_LOCATION(Log(), zen::logging::level::Err, fmtstr "{}", ##__VA_ARGS__, Scopes); \
+#define ZEN_SCOPED_ERROR(fmtstr, ...) \
+ do \
+ { \
+ ExtendableStringBuilder<256> ScopeString; \
+ const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
+ ZEN_LOG_WITH_LOCATION(Log(), zen::logging::Err, fmtstr "{}", ##__VA_ARGS__, Scopes); \
} while (false)
-#define ZEN_SCOPED_CRITICAL(fmtstr, ...) \
- do \
- { \
- ExtendableStringBuilder<256> ScopeString; \
- const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
- ZEN_LOG_WITH_LOCATION(Log(), zen::logging::level::Critical, fmtstr "{}", ##__VA_ARGS__, Scopes); \
+#define ZEN_SCOPED_CRITICAL(fmtstr, ...) \
+ do \
+ { \
+ ExtendableStringBuilder<256> ScopeString; \
+ const std::string_view Scopes = EmitActivitiesForLogging(ScopeString); \
+ ZEN_LOG_WITH_LOCATION(Log(), zen::logging::Critical, fmtstr "{}", ##__VA_ARGS__, Scopes); \
} while (false)
ScopedActivityBase* GetThreadActivity();
diff --git a/src/zencore/include/zencore/logging/ansicolorsink.h b/src/zencore/include/zencore/logging/ansicolorsink.h
new file mode 100644
index 000000000..5060a8393
--- /dev/null
+++ b/src/zencore/include/zencore/logging/ansicolorsink.h
@@ -0,0 +1,33 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+#include <memory>
+
+namespace zen::logging {
+
+enum class ColorMode
+{
+ On,
+ Off,
+ Auto
+};
+
+class AnsiColorStdoutSink : public Sink
+{
+public:
+ explicit AnsiColorStdoutSink(ColorMode Mode = ColorMode::Auto);
+ ~AnsiColorStdoutSink() override;
+
+ void Log(const LogMessage& Msg) override;
+ void Flush() override;
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter) override;
+
+private:
+ struct Impl;
+ std::unique_ptr<Impl> m_Impl;
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/asyncsink.h b/src/zencore/include/zencore/logging/asyncsink.h
new file mode 100644
index 000000000..c49a1ccce
--- /dev/null
+++ b/src/zencore/include/zencore/logging/asyncsink.h
@@ -0,0 +1,30 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+#include <memory>
+#include <vector>
+
+namespace zen::logging {
+
+class AsyncSink : public Sink
+{
+public:
+ explicit AsyncSink(std::vector<SinkPtr> InSinks);
+ ~AsyncSink() override;
+
+ AsyncSink(const AsyncSink&) = delete;
+ AsyncSink& operator=(const AsyncSink&) = delete;
+
+ void Log(const LogMessage& Msg) override;
+ void Flush() override;
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter) override;
+
+private:
+ struct Impl;
+ std::unique_ptr<Impl> m_Impl;
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/formatter.h b/src/zencore/include/zencore/logging/formatter.h
new file mode 100644
index 000000000..11904d71d
--- /dev/null
+++ b/src/zencore/include/zencore/logging/formatter.h
@@ -0,0 +1,20 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/logmsg.h>
+#include <zencore/logging/memorybuffer.h>
+
+#include <memory>
+
+namespace zen::logging {
+
+class Formatter
+{
+public:
+ virtual ~Formatter() = default;
+ virtual void Format(const LogMessage& Msg, MemoryBuffer& Dest) = 0;
+ virtual std::unique_ptr<Formatter> Clone() const = 0;
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/helpers.h b/src/zencore/include/zencore/logging/helpers.h
new file mode 100644
index 000000000..ce021e1a5
--- /dev/null
+++ b/src/zencore/include/zencore/logging/helpers.h
@@ -0,0 +1,122 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logbase.h>
+#include <zencore/logging/memorybuffer.h>
+
+#include <chrono>
+#include <ctime>
+#include <string_view>
+
+namespace zen::logging::helpers {
+
+inline void
+AppendStringView(std::string_view Sv, MemoryBuffer& Dest)
+{
+ Dest.append(Sv.data(), Sv.data() + Sv.size());
+}
+
+inline void
+AppendInt(int N, MemoryBuffer& Dest)
+{
+ fmt::format_int Formatted(N);
+ Dest.append(Formatted.data(), Formatted.data() + Formatted.size());
+}
+
+inline void
+Pad2(int N, MemoryBuffer& Dest)
+{
+ if (N >= 0 && N < 100)
+ {
+ Dest.push_back(static_cast<char>('0' + N / 10));
+ Dest.push_back(static_cast<char>('0' + N % 10));
+ }
+ else
+ {
+ fmt::format_int Formatted(N);
+ Dest.append(Formatted.data(), Formatted.data() + Formatted.size());
+ }
+}
+
+inline void
+Pad3(uint32_t N, MemoryBuffer& Dest)
+{
+ if (N < 1000)
+ {
+ Dest.push_back(static_cast<char>('0' + N / 100));
+ Dest.push_back(static_cast<char>('0' + (N / 10) % 10));
+ Dest.push_back(static_cast<char>('0' + N % 10));
+ }
+ else
+ {
+ AppendInt(static_cast<int>(N), Dest);
+ }
+}
+
+inline void
+PadUint(size_t N, unsigned int Width, MemoryBuffer& Dest)
+{
+ fmt::format_int Formatted(N);
+ auto StrLen = static_cast<unsigned int>(Formatted.size());
+ if (Width > StrLen)
+ {
+ for (unsigned int Pad = 0; Pad < Width - StrLen; ++Pad)
+ {
+ Dest.push_back('0');
+ }
+ }
+ Dest.append(Formatted.data(), Formatted.data() + Formatted.size());
+}
+
+template<typename ToDuration>
+inline ToDuration
+TimeFraction(std::chrono::system_clock::time_point Tp)
+{
+ using std::chrono::duration_cast;
+ using std::chrono::seconds;
+ auto Duration = Tp.time_since_epoch();
+ auto Secs = duration_cast<seconds>(Duration);
+ return duration_cast<ToDuration>(Duration) - duration_cast<ToDuration>(Secs);
+}
+
+inline std::tm
+SafeLocaltime(std::time_t Time)
+{
+ std::tm Result{};
+#if defined(_WIN32)
+ localtime_s(&Result, &Time);
+#else
+ localtime_r(&Time, &Result);
+#endif
+ return Result;
+}
+
+inline const char*
+ShortFilename(const char* Path)
+{
+ if (Path == nullptr)
+ {
+ return Path;
+ }
+
+ const char* It = Path;
+ const char* LastSep = Path;
+ while (*It)
+ {
+ if (*It == '/' || *It == '\\')
+ {
+ LastSep = It + 1;
+ }
+ ++It;
+ }
+ return LastSep;
+}
+
+inline std::string_view
+LevelToShortString(LogLevel Level)
+{
+ return ToStringView(Level);
+}
+
+} // namespace zen::logging::helpers
diff --git a/src/zencore/include/zencore/logging/logger.h b/src/zencore/include/zencore/logging/logger.h
new file mode 100644
index 000000000..c94bc58fa
--- /dev/null
+++ b/src/zencore/include/zencore/logging/logger.h
@@ -0,0 +1,74 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+#include <atomic>
+#include <memory>
+#include <string_view>
+
+namespace zen::logging {
+
+class ErrorHandler
+{
+public:
+ virtual ~ErrorHandler() = default;
+ virtual void HandleError(const std::string_view& Msg) = 0;
+};
+
+class Logger final : public LoggerBase
+{
+public:
+ Logger(std::string_view InName, SinkPtr InSink);
+ Logger(std::string_view InName, std::span<const SinkPtr> InSinks);
+ ~Logger();
+
+ Logger(const Logger&) = delete;
+ Logger& operator=(const Logger&) = delete;
+
+ void Log(const LogPoint& Point, fmt::format_args Args);
+
+ std::string_view Name() const;
+
+ void SetSinks(std::vector<SinkPtr> InSinks);
+ void AddSink(SinkPtr InSink);
+
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter);
+
+ void SetErrorHandler(ErrorHandler* Handler);
+
+ void Flush();
+
+ Ref<Logger> Clone(std::string_view NewName) const;
+
+private:
+ void SinkIt(const LogMessage& Msg);
+ void FlushIfNeeded(LogLevel InLevel);
+
+ struct Impl;
+ std::unique_ptr<Impl> m_Impl;
+};
+
+} // namespace zen::logging
+
+namespace zen {
+
+// These can't be defined next to the LoggerRef declaration since
+// they depend on knowing about logging::Logger above. But it's fine
+// since to be able to use these you have to include this header
+// anyway
+
+inline logging::Logger*
+LoggerRef::operator->() const
+{
+ return static_cast<logging::Logger*>(m_Logger);
+}
+
+inline logging::Logger&
+LoggerRef::operator*() const
+{
+ return *static_cast<logging::Logger*>(m_Logger);
+}
+
+} // namespace zen
diff --git a/src/zencore/include/zencore/logging/logmsg.h b/src/zencore/include/zencore/logging/logmsg.h
new file mode 100644
index 000000000..1d8b6b1b7
--- /dev/null
+++ b/src/zencore/include/zencore/logging/logmsg.h
@@ -0,0 +1,66 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logbase.h>
+
+#include <chrono>
+#include <string_view>
+
+namespace zen::logging {
+
+using LogClock = std::chrono::system_clock;
+
+struct LogMessage
+{
+ LogMessage() = default;
+
+ LogMessage(const LogPoint& InPoint, std::string_view InLoggerName, std::string_view InPayload)
+ : m_LoggerName(InLoggerName)
+ , m_Level(InPoint.Level)
+ , m_Time(LogClock::now())
+ , m_Source(InPoint.Location)
+ , m_Payload(InPayload)
+ , m_Point(&InPoint)
+ {
+ }
+
+ std::string_view GetPayload() const { return m_Payload; }
+ int GetThreadId() const { return m_ThreadId; }
+ LogClock::time_point GetTime() const { return m_Time; }
+ LogLevel GetLevel() const { return m_Level; }
+ std::string_view GetLoggerName() const { return m_LoggerName; }
+ const SourceLocation& GetSource() const { return m_Source; }
+ const LogPoint& GetLogPoint() const { return *m_Point; }
+
+ void SetThreadId(int InThreadId) { m_ThreadId = InThreadId; }
+ void SetPayload(std::string_view InPayload) { m_Payload = InPayload; }
+ void SetLoggerName(std::string_view InName) { m_LoggerName = InName; }
+ void SetLevel(LogLevel InLevel) { m_Level = InLevel; }
+ void SetTime(LogClock::time_point InTime) { m_Time = InTime; }
+ void SetSource(const SourceLocation& InSource) { m_Source = InSource; }
+
+ mutable size_t ColorRangeStart = 0;
+ mutable size_t ColorRangeEnd = 0;
+
+private:
+ static constexpr LogPoint s_DefaultPoints[LogLevelCount] = {
+ {{}, Trace, {}},
+ {{}, Debug, {}},
+ {{}, Info, {}},
+ {{}, Warn, {}},
+ {{}, Err, {}},
+ {{}, Critical, {}},
+ {{}, Off, {}},
+ };
+
+ std::string_view m_LoggerName;
+ LogLevel m_Level = Off;
+ std::chrono::system_clock::time_point m_Time;
+ SourceLocation m_Source;
+ std::string_view m_Payload;
+ const LogPoint* m_Point = &s_DefaultPoints[Off];
+ int m_ThreadId = 0;
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/memorybuffer.h b/src/zencore/include/zencore/logging/memorybuffer.h
new file mode 100644
index 000000000..cd0ff324f
--- /dev/null
+++ b/src/zencore/include/zencore/logging/memorybuffer.h
@@ -0,0 +1,11 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <fmt/format.h>
+
+namespace zen::logging {
+
+using MemoryBuffer = fmt::basic_memory_buffer<char, 250>;
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/messageonlyformatter.h b/src/zencore/include/zencore/logging/messageonlyformatter.h
new file mode 100644
index 000000000..ce25fe9a6
--- /dev/null
+++ b/src/zencore/include/zencore/logging/messageonlyformatter.h
@@ -0,0 +1,22 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/formatter.h>
+#include <zencore/logging/helpers.h>
+
+namespace zen::logging {
+
+class MessageOnlyFormatter : public Formatter
+{
+public:
+ void Format(const LogMessage& Msg, MemoryBuffer& Dest) override
+ {
+ helpers::AppendStringView(Msg.GetPayload(), Dest);
+ Dest.push_back('\n');
+ }
+
+ std::unique_ptr<Formatter> Clone() const override { return std::make_unique<MessageOnlyFormatter>(); }
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/msvcsink.h b/src/zencore/include/zencore/logging/msvcsink.h
new file mode 100644
index 000000000..48ea1b915
--- /dev/null
+++ b/src/zencore/include/zencore/logging/msvcsink.h
@@ -0,0 +1,30 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+#if ZEN_PLATFORM_WINDOWS
+
+# include <mutex>
+
+namespace zen::logging {
+
+class MsvcSink : public Sink
+{
+public:
+ MsvcSink();
+ ~MsvcSink() override = default;
+
+ void Log(const LogMessage& Msg) override;
+ void Flush() override;
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter) override;
+
+private:
+ std::mutex m_Mutex;
+ std::unique_ptr<Formatter> m_Formatter;
+};
+
+} // namespace zen::logging
+
+#endif // ZEN_PLATFORM_WINDOWS
diff --git a/src/zencore/include/zencore/logging/nullsink.h b/src/zencore/include/zencore/logging/nullsink.h
new file mode 100644
index 000000000..7ac5677c6
--- /dev/null
+++ b/src/zencore/include/zencore/logging/nullsink.h
@@ -0,0 +1,17 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+namespace zen::logging {
+
+class NullSink : public Sink
+{
+public:
+ void Log(const LogMessage& /*Msg*/) override {}
+ void Flush() override {}
+ void SetFormatter(std::unique_ptr<Formatter> /*InFormatter*/) override {}
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/registry.h b/src/zencore/include/zencore/logging/registry.h
new file mode 100644
index 000000000..71e84d75f
--- /dev/null
+++ b/src/zencore/include/zencore/logging/registry.h
@@ -0,0 +1,74 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/logger.h>
+
+#include <chrono>
+#include <memory>
+#include <span>
+#include <string>
+#include <type_traits>
+#include <utility>
+
+namespace zen::logging {
+
+// Match a logger name against a pattern that may contain '*' wildcards.
+// Supports '*' anywhere in the pattern (e.g. "proxy*", "*store", "zen*store").
+bool MatchLoggerPattern(std::string_view Pattern, std::string_view Name);
+
+class Registry
+{
+public:
+ using LogLevels = std::span<const std::pair<std::string, LogLevel>>;
+
+ static Registry& Instance();
+ void Shutdown();
+
+ void Register(Ref<Logger> InLogger);
+ void Drop(const std::string& Name);
+ Ref<Logger> Get(const std::string& Name);
+
+ void SetDefaultLogger(Ref<Logger> InLogger);
+ Logger* DefaultLoggerRaw();
+ Ref<Logger> DefaultLogger();
+
+ void SetGlobalLevel(LogLevel Level);
+ LogLevel GetGlobalLevel() const;
+ void SetLevels(LogLevels Levels, LogLevel* DefaultLevel);
+
+ void FlushAll();
+ void FlushOn(LogLevel Level);
+ void FlushEvery(std::chrono::seconds Interval);
+
+ // Change formatter on all registered loggers
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter);
+
+ // Apply function to all registered loggers. Note that the function will
+ // be called while the registry mutex is held, so it should be fast and
+ // not attempt to call back into the registry.
+ template<typename Func>
+ void ApplyAll(Func&& F)
+ {
+ ApplyAllImpl([](void* Ctx, Ref<Logger> L) { (*static_cast<std::decay_t<Func>*>(Ctx))(std::move(L)); }, &F);
+ }
+
+ // Set error handler for all loggers in the registry. The handler is called
+ // if any logger encounters an error during logging or flushing.
+ // The caller must ensure the handler outlives the registry.
+ void SetErrorHandler(ErrorHandler* Handler);
+
+private:
+ void ApplyAllImpl(void (*Func)(void*, Ref<Logger>), void* Context);
+
+ Registry();
+ ~Registry();
+
+ Registry(const Registry&) = delete;
+ Registry& operator=(const Registry&) = delete;
+
+ struct Impl;
+ std::unique_ptr<Impl> m_Impl;
+};
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/sink.h b/src/zencore/include/zencore/logging/sink.h
new file mode 100644
index 000000000..172176a4e
--- /dev/null
+++ b/src/zencore/include/zencore/logging/sink.h
@@ -0,0 +1,34 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zenbase/refcount.h>
+#include <zencore/logging/formatter.h>
+#include <zencore/logging/logmsg.h>
+
+#include <atomic>
+#include <memory>
+
+namespace zen::logging {
+
+class Sink : public RefCounted
+{
+public:
+ virtual ~Sink() = default;
+
+ virtual void Log(const LogMessage& Msg) = 0;
+ virtual void Flush() = 0;
+
+ virtual void SetFormatter(std::unique_ptr<Formatter> InFormatter) = 0;
+
+ bool ShouldLog(LogLevel InLevel) const { return InLevel >= m_Level.load(std::memory_order_relaxed); }
+ void SetLevel(LogLevel InLevel) { m_Level.store(InLevel, std::memory_order_relaxed); }
+ LogLevel GetLevel() const { return m_Level.load(std::memory_order_relaxed); }
+
+protected:
+ std::atomic<LogLevel> m_Level{Trace};
+};
+
+using SinkPtr = Ref<Sink>;
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/logging/tracesink.h b/src/zencore/include/zencore/logging/tracesink.h
new file mode 100644
index 000000000..785c51e10
--- /dev/null
+++ b/src/zencore/include/zencore/logging/tracesink.h
@@ -0,0 +1,27 @@
+// Copyright Epic Games, Inc. All Rights Reserved.
+
+#pragma once
+
+#include <zencore/logging/sink.h>
+
+namespace zen::logging {
+
+#if ZEN_WITH_TRACE
+
+/**
+ * A logging sink that forwards log messages to the trace system.
+ *
+ * Work-in-progress, not fully implemented.
+ */
+
+class TraceSink : public Sink
+{
+public:
+ void Log(const LogMessage& Msg) override;
+ void Flush() override;
+ void SetFormatter(std::unique_ptr<Formatter> InFormatter) override;
+};
+
+#endif
+
+} // namespace zen::logging
diff --git a/src/zencore/include/zencore/md5.h b/src/zencore/include/zencore/md5.h
index d934dd86b..3b0b7cae6 100644
--- a/src/zencore/include/zencore/md5.h
+++ b/src/zencore/include/zencore/md5.h
@@ -43,6 +43,8 @@ public:
MD5 GetHash();
private:
+ // Opaque storage for MD5_CTX (104 bytes, aligned to uint32_t)
+ alignas(4) uint8_t m_Context[104];
};
void md5_forcelink(); // internal
diff --git a/src/zencore/include/zencore/meta.h b/src/zencore/include/zencore/meta.h
index 82eb5cc30..20ec4ac6f 100644
--- a/src/zencore/include/zencore/meta.h
+++ b/src/zencore/include/zencore/meta.h
@@ -1,4 +1,5 @@
// Copyright Epic Games, Inc. All Rights Reserved.
+#pragma once
/* This file contains utility functions for meta programming
*
diff --git a/src/zencore/include/zencore/mpscqueue.h b/src/zencore/include/zencore/mpscqueue.h
index 19e410d85..d97c433fd 100644
--- a/src/zencore/include/zencore/mpscqueue.h
+++ b/src/zencore/include/zencore/mpscqueue.h
@@ -22,10 +22,10 @@ namespace zen {
template<typename ElementType>
struct TypeCompatibleStorage
{
- ElementType* Data() { return (ElementType*)this; }
- const ElementType* Data() const { return (const ElementType*)this; }
+ ElementType* Data() { return reinterpret_cast<ElementType*>(&Storage); }
+ const ElementType* Data() const { return reinterpret_cast<const ElementType*>(&Storage); }
- alignas(ElementType) char DataMember;
+ alignas(ElementType) char Storage[sizeof(ElementType)];
};
/** Fast multi-producer/single-consumer unbounded concurrent queue.
@@ -58,7 +58,7 @@ public:
Tail = Next;
Next = Tail->Next.load(std::memory_order_relaxed);
- std::destroy_at((ElementType*)&Tail->Value);
+ std::destroy_at(Tail->Value.Data());
delete Tail;
}
}
@@ -67,7 +67,7 @@ public:
void Enqueue(ArgTypes&&... Args)
{
Node* New = new Node;
- new (&New->Value) ElementType(std::forward<ArgTypes>(Args)...);
+ new (New->Value.Data()) ElementType(std::forward<ArgTypes>(Args)...);
Node* Prev = Head.exchange(New, std::memory_order_acq_rel);
Prev->Next.store(New, std::memory_order_release);
@@ -82,7 +82,7 @@ public:
return {};
}
- ElementType* ValuePtr = (ElementType*)&Next->Value;
+ ElementType* ValuePtr = Next->Value.Data();
std::optional<ElementType> Res{std::move(*ValuePtr)};
std::destroy_at(ValuePtr);
@@ -100,9 +100,11 @@ private:
};
private:
- std::atomic<Node*> Head; // accessed only by producers
- alignas(hardware_constructive_interference_size)
- Node* Tail; // accessed only by consumer, hence should be on a different cache line than `Head`
+ // Use a fixed constant to avoid GCC's -Winterference-size warning with std::hardware_destructive_interference_size
+ static constexpr std::size_t CacheLineSize = 64;
+
+ alignas(CacheLineSize) std::atomic<Node*> Head; // accessed only by producers
+ alignas(CacheLineSize) Node* Tail; // accessed only by consumer, separate cache line from Head
};
void mpscqueue_forcelink();
diff --git a/src/zencore/include/zencore/process.h b/src/zencore/include/zencore/process.h
index e3b7a70d7..3177f64c1 100644
--- a/src/zencore/include/zencore/process.h
+++ b/src/zencore/include/zencore/process.h
@@ -6,9 +6,16 @@
#include <zencore/zencore.h>
#include <filesystem>
+#include <string>
+#include <utility>
+#include <vector>
namespace zen {
+#if ZEN_PLATFORM_WINDOWS
+class JobObject;
+#endif
+
/** Basic process abstraction
*/
class ProcessHandle
@@ -46,6 +53,7 @@ private:
/** Basic process creation
*/
+
struct CreateProcOptions
{
enum
@@ -63,6 +71,15 @@ struct CreateProcOptions
const std::filesystem::path* WorkingDirectory = nullptr;
uint32_t Flags = 0;
std::filesystem::path StdoutFile;
+
+ /// Additional environment variables for the child process. These are merged
+ /// with the parent's environment — existing variables are inherited, and
+ /// entries here override or add to them.
+ std::vector<std::pair<std::string, std::string>> Environment;
+
+#if ZEN_PLATFORM_WINDOWS
+ JobObject* AssignToJob = nullptr; // When set, the process is created suspended, assigned to the job, then resumed
+#endif
};
#if ZEN_PLATFORM_WINDOWS
@@ -99,12 +116,38 @@ private:
std::vector<HandleType> m_ProcessHandles;
};
+#if ZEN_PLATFORM_WINDOWS
+/** Windows Job Object wrapper
+ *
+ * When configured with JOB_OBJECT_LIMIT_KILL_ON_JOB_CLOSE, the OS will
+ * terminate all assigned child processes when the job handle is closed
+ * (including abnormal termination of the owning process). This provides
+ * an OS-level guarantee against orphaned child processes.
+ */
+class JobObject
+{
+public:
+ JobObject();
+ ~JobObject();
+ JobObject(const JobObject&) = delete;
+ JobObject& operator=(const JobObject&) = delete;
+
+ void Initialize();
+ bool AssignProcess(void* ProcessHandle);
+ [[nodiscard]] bool IsValid() const;
+
+private:
+ void* m_JobHandle = nullptr;
+};
+#endif // ZEN_PLATFORM_WINDOWS
+
bool IsProcessRunning(int pid);
bool IsProcessRunning(int pid, std::error_code& OutEc);
int GetCurrentProcessId();
int GetProcessId(CreateProcResult ProcId);
std::filesystem::path GetProcessExecutablePath(int Pid, std::error_code& OutEc);
+std::string GetProcessCommandLine(int Pid, std::error_code& OutEc);
std::error_code FindProcess(const std::filesystem::path& ExecutableImage, ProcessHandle& OutHandle, bool IncludeSelf = true);
/** Wait for all threads in the current process to exit (except the calling thread)
diff --git a/src/zencore/include/zencore/sentryintegration.h b/src/zencore/include/zencore/sentryintegration.h
index faf1238b7..27e5a8a82 100644
--- a/src/zencore/include/zencore/sentryintegration.h
+++ b/src/zencore/include/zencore/sentryintegration.h
@@ -11,11 +11,9 @@
#if ZEN_USE_SENTRY
-# include <memory>
+# include <zencore/logging/logger.h>
-ZEN_THIRD_PARTY_INCLUDES_START
-# include <spdlog/logger.h>
-ZEN_THIRD_PARTY_INCLUDES_END
+# include <memory>
namespace sentry {
@@ -42,6 +40,7 @@ public:
};
void Initialize(const Config& Conf, const std::string& CommandLine);
+ void Close();
void LogStartupInformation();
static void ClearCaches();
@@ -53,7 +52,7 @@ private:
std::string m_SentryUserName;
std::string m_SentryHostName;
std::string m_SentryId;
- std::shared_ptr<spdlog::logger> m_SentryLogger;
+ Ref<logging::Logger> m_SentryLogger;
};
} // namespace zen
diff --git a/src/zencore/include/zencore/sharedbuffer.h b/src/zencore/include/zencore/sharedbuffer.h
index c57e9f568..3d4c19282 100644
--- a/src/zencore/include/zencore/sharedbuffer.h
+++ b/src/zencore/include/zencore/sharedbuffer.h
@@ -116,14 +116,15 @@ public:
inline void Reset() { m_Buffer = nullptr; }
inline bool GetFileReference(IoBufferFileReference& OutRef) const
{
- if (const IoBufferExtendedCore* Core = m_Buffer->ExtendedCore())
+ if (!IsNull())
{
- return Core->GetFileReference(OutRef);
- }
- else
- {
- return false;
+ if (const IoBufferExtendedCore* Core = m_Buffer->ExtendedCore())
+ {
+ return Core->GetFileReference(OutRef);
+ }
}
+
+ return false;
}
[[nodiscard]] MemoryView GetView() const
diff --git a/src/zencore/include/zencore/string.h b/src/zencore/include/zencore/string.h
index 5a12ba5d2..60293a313 100644
--- a/src/zencore/include/zencore/string.h
+++ b/src/zencore/include/zencore/string.h
@@ -8,7 +8,6 @@
#include <stdint.h>
#include <string.h>
#include <charconv>
-#include <codecvt>
#include <compare>
#include <concepts>
#include <optional>
@@ -51,7 +50,7 @@ StringLength(const wchar_t* str)
return wcslen(str);
}
-inline bool
+inline int
StringCompare(const char16_t* s1, const char16_t* s2)
{
char16_t c1, c2;
@@ -66,7 +65,7 @@ StringCompare(const char16_t* s1, const char16_t* s2)
++s1;
++s2;
}
- return uint16_t(c1) - uint16_t(c2);
+ return int(uint16_t(c1)) - int(uint16_t(c2));
}
inline bool
@@ -122,10 +121,10 @@ public:
StringBuilderImpl() = default;
~StringBuilderImpl();
- StringBuilderImpl(const StringBuilderImpl&) = delete;
- StringBuilderImpl(const StringBuilderImpl&&) = delete;
+ StringBuilderImpl(const StringBuilderImpl&) = delete;
+ StringBuilderImpl(StringBuilderImpl&&) = delete;
const StringBuilderImpl& operator=(const StringBuilderImpl&) = delete;
- const StringBuilderImpl& operator=(const StringBuilderImpl&&) = delete;
+ StringBuilderImpl& operator=(StringBuilderImpl&&) = delete;
inline size_t AddUninitialized(size_t Count)
{
@@ -332,9 +331,10 @@ public:
return AppendAscii(Str);
}
-#if defined(__clang__) && !defined(__APPLE__)
+#if defined(__clang__) && !defined(__APPLE__) && !defined(_MSC_VER)
/* UE Toolchain Clang has different types for int64_t and long long so an override is
- needed here. Without it, Clang can't disambiguate integer overloads */
+ needed here. Without it, Clang can't disambiguate integer overloads.
+ On MSVC ABI (including clang-cl), int64_t is long long so no separate overload is needed. */
inline StringBuilderImpl& operator<<(long long n)
{
IntNum Str(n);
@@ -374,9 +374,9 @@ protected:
[[noreturn]] void Fail(const char* FailReason); // note: throws exception
- C* m_Base;
- C* m_CurPos;
- C* m_End;
+ C* m_Base = nullptr;
+ C* m_CurPos = nullptr;
+ C* m_End = nullptr;
bool m_IsDynamic = false;
bool m_IsExtendable = false;
};
@@ -773,8 +773,9 @@ std::optional<T>
ParseInt(const std::string_view& Input)
{
T Out = 0;
- const std::from_chars_result Result = std::from_chars(Input.data(), Input.data() + Input.size(), Out);
- if (Result.ec == std::errc::invalid_argument || Result.ec == std::errc::result_out_of_range)
+ const char* End = Input.data() + Input.size();
+ const std::from_chars_result Result = std::from_chars(Input.data(), End, Out);
+ if (Result.ec == std::errc::invalid_argument || Result.ec == std::errc::result_out_of_range || Result.ptr != End)
{
return std::nullopt;
}
@@ -953,6 +954,24 @@ StrCaseCompare(const char* Lhs, const char* Rhs, int64_t Length = -1)
#endif
}
+inline int32_t
+StrCaseCompare(std::string_view Lhs, std::string_view Rhs)
+{
+ int32_t Result = StrCaseCompare(Lhs.data(), Rhs.data(), std::min(Lhs.size(), Rhs.size()));
+ if (Result == 0)
+ {
+ if (Lhs.size() < Rhs.size())
+ {
+ return -1;
+ }
+ else if (Lhs.size() > Rhs.size())
+ {
+ return 1;
+ }
+ }
+ return Result;
+}
+
/**
* @brief
* Helper function to implement case sensitive spaceship operator for strings.
@@ -1265,6 +1284,8 @@ private:
uint64_t LoMask, HiMask;
};
+std::string HideSensitiveString(std::string_view String);
+
//////////////////////////////////////////////////////////////////////////
void string_forcelink(); // internal
diff --git a/src/zencore/include/zencore/system.h b/src/zencore/include/zencore/system.h
index bf3c15d3d..2e39cc660 100644
--- a/src/zencore/include/zencore/system.h
+++ b/src/zencore/include/zencore/system.h
@@ -4,6 +4,8 @@
#include <zencore/zencore.h>
+#include <chrono>
+#include <memory>
#include <string>
namespace zen {
@@ -12,7 +14,10 @@ class CbWriter;
std::string GetMachineName();
std::string_view GetOperatingSystemName();
+std::string GetOperatingSystemVersion();
+std::string_view GetRuntimePlatformName(); // "windows", "wine", "linux", or "macos"
std::string_view GetCpuName();
+std::string_view GetCompilerName();
struct SystemMetrics
{
@@ -25,7 +30,14 @@ struct SystemMetrics
uint64_t AvailVirtualMemoryMiB = 0;
uint64_t PageFileMiB = 0;
uint64_t AvailPageFileMiB = 0;
- float CpuUsagePercent = 0.0f;
+ uint64_t UptimeSeconds = 0;
+};
+
+/// Extended metrics that include CPU usage percentage, which requires
+/// stateful delta tracking via SystemMetricsTracker.
+struct ExtendedSystemMetrics : SystemMetrics
+{
+ float CpuUsagePercent = 0.0f;
};
SystemMetrics GetSystemMetrics();
@@ -33,6 +45,31 @@ SystemMetrics GetSystemMetrics();
void SetCpuCountForReporting(int FakeCpuCount);
SystemMetrics GetSystemMetricsForReporting();
+ExtendedSystemMetrics ApplyReportingOverrides(ExtendedSystemMetrics Metrics);
+
void Describe(const SystemMetrics& Metrics, CbWriter& Writer);
+void Describe(const ExtendedSystemMetrics& Metrics, CbWriter& Writer);
+
+/// Stateful tracker that computes CPU usage as a delta between consecutive
+/// Query() calls. The first call returns CpuUsagePercent = 0 (no previous
+/// sample). Thread-safe: concurrent calls are serialised internally.
+/// CPU sampling is rate-limited to MinInterval (default 1 s); calls that
+/// arrive sooner return the previously cached value.
+class SystemMetricsTracker
+{
+public:
+ explicit SystemMetricsTracker(std::chrono::milliseconds MinInterval = std::chrono::seconds(1));
+ ~SystemMetricsTracker();
+
+ SystemMetricsTracker(const SystemMetricsTracker&) = delete;
+ SystemMetricsTracker& operator=(const SystemMetricsTracker&) = delete;
+
+ /// Collect current metrics. CPU usage is computed as delta since last Query().
+ ExtendedSystemMetrics Query();
+
+private:
+ struct Impl;
+ std::unique_ptr<Impl> m_Impl;
+};
} // namespace zen
diff --git a/src/zencore/include/zencore/testing.h b/src/zencore/include/zencore/testing.h
index a00ee3166..8410216c4 100644
--- a/src/zencore/include/zencore/testing.h
+++ b/src/zencore/include/zencore/testing.h
@@ -43,8 +43,9 @@ public:
TestRunner();
~TestRunner();
- int ApplyCommandLine(int argc, char const* const* argv);
- int Run();
+ void SetDefaultSuiteFilter(const char* Pattern);
+ int ApplyCommandLine(int Argc, char const* const* Argv);
+ int Run();
private:
struct Impl;
@@ -59,6 +60,8 @@ private:
return Runner.Run(); \
}()
+int RunTestMain(int Argc, char* Argv[], const char* ExecutableName, void (*ForceLink)());
+
} // namespace zen::testing
#endif
diff --git a/src/zencore/include/zencore/testutils.h b/src/zencore/include/zencore/testutils.h
index e2a4f8346..2a789d18f 100644
--- a/src/zencore/include/zencore/testutils.h
+++ b/src/zencore/include/zencore/testutils.h
@@ -59,6 +59,33 @@ struct TrueType
static const bool Enabled = true;
};
+namespace utf8test {
+
+ // 2-byte UTF-8 (Latin extended)
+ static constexpr const char kLatin[] = u8"café_résumé";
+ static constexpr const wchar_t kLatinW[] = L"café_résumé";
+
+ // 2-byte UTF-8 (Cyrillic)
+ static constexpr const char kCyrillic[] = u8"данные";
+ static constexpr const wchar_t kCyrillicW[] = L"данные";
+
+ // 3-byte UTF-8 (CJK)
+ static constexpr const char kCJK[] = u8"日本語";
+ static constexpr const wchar_t kCJKW[] = L"日本語";
+
+ // Mixed scripts
+ static constexpr const char kMixed[] = u8"zen_éд日";
+ static constexpr const wchar_t kMixedW[] = L"zen_éд日";
+
+ // 4-byte UTF-8 (supplementary plane) — string tests only, NOT filesystem
+ static constexpr const char kEmoji[] = u8"📦";
+ static constexpr const wchar_t kEmojiW[] = L"📦";
+
+ // BMP-only test strings suitable for filesystem use
+ static constexpr const char* kFilenameSafe[] = {kLatin, kCyrillic, kCJK, kMixed};
+
+} // namespace utf8test
+
} // namespace zen
#endif // ZEN_WITH_TESTS
diff --git a/src/zencore/include/zencore/thread.h b/src/zencore/include/zencore/thread.h
index a1c68b0b2..d0d710ee8 100644
--- a/src/zencore/include/zencore/thread.h
+++ b/src/zencore/include/zencore/thread.h
@@ -58,7 +58,7 @@ public:
}
private:
- RwLock* m_Lock;
+ RwLock* m_Lock = nullptr;
};
inline auto WithSharedLock(auto&& Fun)
@@ -69,6 +69,16 @@ public:
struct ExclusiveLockScope
{
+ ExclusiveLockScope(const ExclusiveLockScope& Rhs) = delete;
+ ExclusiveLockScope(ExclusiveLockScope&& Rhs) : m_Lock(Rhs.m_Lock) { Rhs.m_Lock = nullptr; }
+ ExclusiveLockScope& operator=(ExclusiveLockScope&& Rhs)
+ {
+ ReleaseNow();
+ m_Lock = Rhs.m_Lock;
+ Rhs.m_Lock = nullptr;
+ return *this;
+ }
+ ExclusiveLockScope& operator=(const ExclusiveLockScope& Rhs) = delete;
ExclusiveLockScope(RwLock& Lock) : m_Lock(&Lock) { Lock.AcquireExclusive(); }
~ExclusiveLockScope() { ReleaseNow(); }
@@ -82,7 +92,7 @@ public:
}
private:
- RwLock* m_Lock;
+ RwLock* m_Lock = nullptr;
};
inline auto WithExclusiveLock(auto&& Fun)
@@ -195,7 +205,7 @@ public:
// false positive completion results.
void AddCount(std::ptrdiff_t Count)
{
- std::atomic_ptrdiff_t Old = Counter.fetch_add(Count);
+ std::ptrdiff_t Old = Counter.fetch_add(Count);
ZEN_ASSERT(Old > 0);
}
diff --git a/src/zencore/include/zencore/trace.h b/src/zencore/include/zencore/trace.h
index 99a565151..d17e018ea 100644
--- a/src/zencore/include/zencore/trace.h
+++ b/src/zencore/include/zencore/trace.h
@@ -13,6 +13,7 @@ ZEN_THIRD_PARTY_INCLUDES_START
# define TRACE_IMPLEMENT 0
#endif
#include <trace.h>
+#include <lane_trace.h>
#undef TRACE_IMPLEMENT
ZEN_THIRD_PARTY_INCLUDES_END
diff --git a/src/zencore/include/zencore/varint.h b/src/zencore/include/zencore/varint.h
index 9fe905f25..43ca14d38 100644
--- a/src/zencore/include/zencore/varint.h
+++ b/src/zencore/include/zencore/varint.h
@@ -1,4 +1,5 @@
// Copyright Epic Games, Inc. All Rights Reserved.
+#pragma once
#include "intmath.h"
diff --git a/src/zencore/include/zencore/xxhash.h b/src/zencore/include/zencore/xxhash.h
index fc55b513b..f79d39b61 100644
--- a/src/zencore/include/zencore/xxhash.h
+++ b/src/zencore/include/zencore/xxhash.h
@@ -87,7 +87,7 @@ struct XXH3_128Stream
}
private:
- XXH3_state_s m_State;
+ XXH3_state_s m_State{};
};
struct XXH3_128Stream_deprecated
diff --git a/src/zencore/include/zencore/zencore.h b/src/zencore/include/zencore/zencore.h
index 177a19fff..a31950b0b 100644
--- a/src/zencore/include/zencore/zencore.h
+++ b/src/zencore/include/zencore/zencore.h
@@ -70,26 +70,36 @@ protected:
} // namespace zen
-#define ZEN_ASSERT(x, ...) \
- do \
- { \
- if (x) [[unlikely]] \
- break; \
- zen::AssertImpl::ExecAssert(__FILE__, __LINE__, __FUNCTION__, #x); \
+#define ZEN_ASSERT(x, ...) \
+ do \
+ { \
+ if (x) [[unlikely]] \
+ break; \
+ zen::AssertImpl::ExecAssert(__FILE__, __LINE__, __FUNCTION__, ZEN_ASSERT_MSG_(#x, ##__VA_ARGS__)); \
} while (false)
#ifndef NDEBUG
-# define ZEN_ASSERT_SLOW(x, ...) \
- do \
- { \
- if (x) [[unlikely]] \
- break; \
- zen::AssertImpl::ExecAssert(__FILE__, __LINE__, __FUNCTION__, #x); \
+# define ZEN_ASSERT_SLOW(x, ...) \
+ do \
+ { \
+ if (x) [[unlikely]] \
+ break; \
+ zen::AssertImpl::ExecAssert(__FILE__, __LINE__, __FUNCTION__, ZEN_ASSERT_MSG_(#x, ##__VA_ARGS__)); \
} while (false)
#else
# define ZEN_ASSERT_SLOW(x, ...)
#endif
+// Internal: select between "expr" and "expr: message" forms.
+// With no extra args: ZEN_ASSERT_MSG_("expr") -> "expr"
+// With a message arg: ZEN_ASSERT_MSG_("expr", "msg") -> "expr" ": " "msg"
+// With fmt-style args: ZEN_ASSERT_MSG_("expr", "msg", args...) -> "expr" ": " "msg"
+// The extra fmt args are silently discarded here — use ZEN_ASSERT_FORMAT for those.
+#define ZEN_ASSERT_MSG_SELECT_(_1, _2, N, ...) N
+#define ZEN_ASSERT_MSG_1_(expr) expr
+#define ZEN_ASSERT_MSG_2_(expr, msg, ...) expr ": " msg
+#define ZEN_ASSERT_MSG_(expr, ...) ZEN_ASSERT_MSG_SELECT_(unused, ##__VA_ARGS__, ZEN_ASSERT_MSG_2_, ZEN_ASSERT_MSG_1_)(expr, ##__VA_ARGS__)
+
//////////////////////////////////////////////////////////////////////////
#define ZEN_NOT_IMPLEMENTED(...) ZEN_ASSERT(false, __VA_ARGS__)