mirror of
https://github.com/Lime3DS/Lime3DS
synced 2024-12-27 01:22:37 -06:00
common/logging: don't use regex for path trimming
This commit is contained in:
parent
020cd56ad8
commit
b3beff0f4c
2 changed files with 24 additions and 17 deletions
|
@ -115,15 +115,12 @@ private:
|
||||||
using std::chrono::duration_cast;
|
using std::chrono::duration_cast;
|
||||||
using std::chrono::steady_clock;
|
using std::chrono::steady_clock;
|
||||||
|
|
||||||
// matches from the beginning up to the last '../' or 'src/'
|
|
||||||
static const std::regex trim_source_path(R"(.*([\/\\]|^)((\.\.)|(src))[\/\\])");
|
|
||||||
|
|
||||||
Entry entry;
|
Entry entry;
|
||||||
entry.timestamp =
|
entry.timestamp =
|
||||||
duration_cast<std::chrono::microseconds>(steady_clock::now() - time_origin);
|
duration_cast<std::chrono::microseconds>(steady_clock::now() - time_origin);
|
||||||
entry.log_class = log_class;
|
entry.log_class = log_class;
|
||||||
entry.log_level = log_level;
|
entry.log_level = log_level;
|
||||||
entry.filename = std::regex_replace(filename, trim_source_path, "");
|
entry.filename = filename;
|
||||||
entry.line_num = line_nr;
|
entry.line_num = line_nr;
|
||||||
entry.function = function;
|
entry.function = function;
|
||||||
entry.message = std::move(message);
|
entry.message = std::move(message);
|
||||||
|
|
|
@ -9,6 +9,15 @@
|
||||||
|
|
||||||
namespace Log {
|
namespace Log {
|
||||||
|
|
||||||
|
// trims up to and including the last of ../, ..\, src/, src\ in a string
|
||||||
|
constexpr const char* TrimSourcePath(std::string_view source) {
|
||||||
|
const auto rfind = [source](const std::string_view match) {
|
||||||
|
return source.rfind(match) == source.npos ? 0 : (source.rfind(match) + match.size());
|
||||||
|
};
|
||||||
|
auto idx = std::max({rfind("src/"), rfind("src\\"), rfind("../"), rfind("..\\")});
|
||||||
|
return source.data() + idx;
|
||||||
|
}
|
||||||
|
|
||||||
/// Specifies the severity or level of detail of the log message.
|
/// Specifies the severity or level of detail of the log message.
|
||||||
enum class Level : u8 {
|
enum class Level : u8 {
|
||||||
Trace, ///< Extremely detailed and repetitive debugging information that is likely to
|
Trace, ///< Extremely detailed and repetitive debugging information that is likely to
|
||||||
|
@ -120,28 +129,29 @@ void FmtLogMessage(Class log_class, Level log_level, const char* filename, unsig
|
||||||
|
|
||||||
// Define the fmt lib macros
|
// Define the fmt lib macros
|
||||||
#define LOG_GENERIC(log_class, log_level, ...) \
|
#define LOG_GENERIC(log_class, log_level, ...) \
|
||||||
::Log::FmtLogMessage(log_class, log_level, __FILE__, __LINE__, __func__, __VA_ARGS__)
|
::Log::FmtLogMessage(log_class, log_level, ::Log::TrimSourcePath(__FILE__), __LINE__, \
|
||||||
|
__func__, __VA_ARGS__)
|
||||||
|
|
||||||
#ifdef _DEBUG
|
#ifdef _DEBUG
|
||||||
#define LOG_TRACE(log_class, ...) \
|
#define LOG_TRACE(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Trace, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Trace, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
#else
|
#else
|
||||||
#define LOG_TRACE(log_class, fmt, ...) (void(0))
|
#define LOG_TRACE(log_class, fmt, ...) (void(0))
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#define LOG_DEBUG(log_class, ...) \
|
#define LOG_DEBUG(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Debug, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Debug, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
#define LOG_INFO(log_class, ...) \
|
#define LOG_INFO(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Info, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Info, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
#define LOG_WARNING(log_class, ...) \
|
#define LOG_WARNING(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Warning, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Warning, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
#define LOG_ERROR(log_class, ...) \
|
#define LOG_ERROR(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Error, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Error, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
#define LOG_CRITICAL(log_class, ...) \
|
#define LOG_CRITICAL(log_class, ...) \
|
||||||
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Critical, __FILE__, __LINE__, \
|
::Log::FmtLogMessage(::Log::Class::log_class, ::Log::Level::Critical, \
|
||||||
__func__, __VA_ARGS__)
|
::Log::TrimSourcePath(__FILE__), __LINE__, __func__, __VA_ARGS__)
|
||||||
|
|
Loading…
Reference in a new issue