-
Notifications
You must be signed in to change notification settings - Fork 4.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Getting compile-error in logger_impl.h#L255 #571
Comments
Because the clang tests were removed a while ago :( |
Fixed in 88fe218 Please verify it solves your problem |
I would guess you never call #define SPDLOG_WCHAR_TO_UTF8_SUPPORT in your tests? |
Yes, that fixes it for me. |
Btw, since I have you here :) when do you plan to release spdlog next time? |
Very soon. Hopefully in the next few days. Why? |
I'm waiting for two of the wstring-fixes, but also a fmt-fix related to wstring. (ref: I have just copied the sources for now, but ideally we would get spdlog + fmt via conan. |
Please note that spdlog bundles only official fmt versions, not necessarily with the latest fixes from the fmt repo.. |
Yeah, I guessed as much, but I also agree. I had a tiny dream that they could be co-released :) |
Do you know when fmt plan to release? that can be arranged |
Seems like they accepted to do a release the next few days: fmtlib/fmt#621 (comment) |
Oh thats good. I will wait.. |
spdlog/include/spdlog/details/logger_impl.h
Line 255 in b1be7b9
There is an extra right curly braces. (at least for clang 5)
How come your build-server flagged this as green?
The text was updated successfully, but these errors were encountered: