RM
Там и так асинк запись
Size: a a a
2020 September 06
RM
В спдлог
RM
Вроде
CC
auto dailySink = std::make_shared<spdlog::sinks::daily_file_sink_mt>(dir, 23, 59);
Магические константы идут нахуй сразу
CC
Особенно те, что повторяются
CC
(надо выносить выше)
RM
Это не повтор
IL
auto dailySink = std::make_shared<spdlog::sinks::daily_file_sink_mt>(dir, 23, 59);
Магические константы идут нахуй сразу
Я сначала подумал что забыли синки в разговоре про дайверсити, а потом понял что это не тот чат... Пора идти спать
RM
Там либо многопоток либо однопоток
RM
У однопотока там _st префикс
CC
я про 23, 59
RM
я про 23, 59
Ага, мне что 23 и 59 в констекспр выносить?)
CC
Разумеется
RM
Это надо тоже в файл выносить
CC
Это магические константы
CC
Далее, когнитивная сложность процедуры PluginLog::ConfigLogger слишком высока, этот код сложно читать, разбей его на несколько процедур(в идеале функций)
RM
Нее, пускай так
CC
Чем аргументировано твое решение выделить это в одну процедуру?
RM
Я и так вынес его в отдельную функцию, до этого было в лямбде
RM
Прям в call once