FarManager icon indicating copy to clipboard operation
FarManager copied to clipboard

Use a thirdparty logging library

Open data-man opened this issue 8 years ago • 2 comments

Just an idea. It's a header-only C++ logging library with support fmtlib formatting.

data-man avatar Jan 03 '18 15:01 data-man

Or spdlog.

data-man avatar Jan 03 '18 15:01 data-man

The idea is good, we definitely need to improve the logging, but...

loguru:

  • has no idea that wide strings (std::wstring, const wchar_t*) exist.
  • doesn't work with the current fmt version.

spdlog:

  • supports only const wchar_t* in the format string. LOG(str.c_str(), ...) every time, really?
  • supports it in a truly abominable way: through a conversion to UTF-8 and using deprecated <codecvt> interfaces.

Overall it looks like it would be simpler to create our own logger from scratch than trying to fix those libraries.

alabuzhev avatar Jul 11 '19 18:07 alabuzhev