Fallback to display thread id of no name was assigned, OpenVulkano::Utils::GetThreadId()

This commit is contained in:
Vladyslav Baranovskyi
2024-05-30 17:52:02 +03:00
parent 3127af5bc7
commit 75b5f635ef
3 changed files with 23 additions and 4 deletions

View File

@@ -11,6 +11,7 @@
#include <iostream>
#include <filesystem>
#include <csignal>
#include <inttypes.h>
#include <spdlog/sinks/rotating_file_sink.h>
#include <spdlog/sinks/null_sink.h>
#ifndef NO_CONSOLE_LOG
@@ -182,9 +183,18 @@ namespace OpenVulkano
void CustomThreadNameFormatterFlag::format(const spdlog::details::log_msg &msg, const std::tm &tmTime, spdlog::memory_buf_t &dest)
{
std::string threadName = OpenVulkano::Utils::GetThreadName();
// NOTE(vb): What should be displayed if the thread name wasn't specified?
if(threadName.length())
{
dest.append(threadName.c_str(), threadName.c_str() + threadName.length());
}
else
{
uint64_t threadId = OpenVulkano::Utils::GetThreadId();
char buffer[64];
snprintf(buffer, sizeof(buffer), "%" PRIu64, threadId);
dest.append(buffer, buffer + strlen(buffer));
}
}
std::unique_ptr<spdlog::custom_flag_formatter> CustomThreadNameFormatterFlag::clone() const
{

View File

@@ -43,14 +43,22 @@ namespace OpenVulkano
return str;
#else
// NOTE(vb): Accoring to linux man page size of this array must be _at least_ 16 bytes...
// But I don't know whether is it applicable to have the buffer of bigger size
char threadName[16];
char threadName[64];
pthread_getname_np(pthread_self(), threadName, sizeof(threadName));
std::string str = threadName;
return str;
#endif
}
uint64_t Utils::GetThreadId()
{
#ifdef _MSC_VER
return (uint64_t)::GetThreadId(::GetCurrentThread());
#else
return (uint64_t)pthread_self();
#endif
}
Array<char> Utils::ReadFile(const std::string& filePath, bool emptyOnMissing)
{
std::ifstream file(filePath, std::ios::ate | std::ios::binary);

View File

@@ -20,6 +20,7 @@ namespace OpenVulkano
public:
static void SetThreadName(const std::string& name);
static std::string GetThreadName();
static uint64_t GetThreadId();
static inline std::vector<const char*> toCString(const std::vector<std::string>& values)
{