Skip to content

Commit 21d9ee8

Browse files
Changed parameter types and fixed cmake issue
1 parent 8c9c630 commit 21d9ee8

File tree

3 files changed

+9
-6
lines changed

3 files changed

+9
-6
lines changed

CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ set(version 1.3.0)
1010

1111
message("Adding source files")
1212
set(LWLOG_SOURCE_FILES
13+
${CMAKE_SOURCE_DIR}/lwlog/src/details/topic_registry.cpp
1314
${CMAKE_SOURCE_DIR}/lwlog/src/details/pattern/attribute.cpp
1415
)
1516

lwlog/src/policy/log_policy.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ namespace lwlog
3030
static void init(backend<Config, BufferLimits, ConcurrencyModelPolicy>&) {}
3131

3232
template<typename Config, typename BufferLimits, typename ConcurrencyModelPolicy, typename... Args>
33-
static void log(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, std::string_view message,
33+
static void log(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, const char* const message,
3434
level log_level, const details::source_meta& meta, Args&&... args);
3535
};
3636

@@ -68,7 +68,7 @@ namespace lwlog
6868
static void init(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend);
6969

7070
template<typename Config, typename BufferLimits, typename ConcurrencyModelPolicy, typename... Args>
71-
static void log(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, std::string_view message,
71+
static void log(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, const char* const message,
7272
level log_level, const details::source_meta& meta, Args&&... args);
7373
};
7474
}

lwlog/src/policy/log_policy_impl.h

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
#pragma once
22

3+
#include "log_policy.h"
4+
35
namespace lwlog
46
{
57
template<typename Config, typename BufferLimits, typename ConcurrencyModelPolicy, typename... Args>
68
void synchronous_policy::log(backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend,
7-
std::string_view message, level log_level, const details::source_meta& meta, Args&&... args)
9+
const char* const message, level log_level, const details::source_meta& meta, Args&&... args)
810
{
911
backend.message_buffer.reset();
1012
backend.message_buffer.append(message);
@@ -87,12 +89,12 @@ namespace lwlog
8789
template<typename OverflowPolicy, std::size_t Capacity, std::uint64_t ThreadAffinity>
8890
template<typename Config, typename BufferLimits, typename ConcurrencyModelPolicy, typename... Args>
8991
void asynchronous_policy<OverflowPolicy, Capacity, ThreadAffinity>::log(
90-
backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, std::string_view message,
92+
backend<Config, BufferLimits, ConcurrencyModelPolicy>& backend, const char* const message,
9193
level log_level, const details::source_meta& meta, Args&&... args)
9294
{
9395
if constexpr (sizeof...(args) == 0)
9496
{
95-
backend.queue.enqueue({ false, 0, message.data(), log_level, meta });
97+
backend.queue.enqueue({ false, 0, message, log_level, meta });
9698
}
9799
else
98100
{
@@ -103,7 +105,7 @@ namespace lwlog
103105
(details::convert_to_chars(args_buffer[buffer_index++],
104106
BufferLimits::argument, std::forward<Args>(args)), ...);
105107

106-
backend.queue.enqueue({ true, buff_index, message.data(), log_level, meta });
108+
backend.queue.enqueue({ true, buff_index, message, log_level, meta });
107109
}
108110
}
109111

0 commit comments

Comments
 (0)