Skip to content

Add missing xchar.h header for fmt > 11.0.2#3274

Closed
ahoarau wants to merge 1 commit intogabime:v1.xfrom
ahoarau:patch-1
Closed

Add missing xchar.h header for fmt > 11.0.2#3274
ahoarau wants to merge 1 commit intogabime:v1.xfrom
ahoarau:patch-1

Conversation

@ahoarau
Copy link
Copy Markdown

@ahoarau ahoarau commented Nov 30, 2024

Using fmt main branch, after the 11.0.2 release, fmt::basic_format_string is defined in xchar.h.

Otherwise compile error:

spdlog/common.h:369:49: error: no template named 'basic_format_string' in namespace 'fmt'; did you mean 'std::basic_format_string'?
[build]   369 | inline fmt::basic_string_view<T> to_string_view(fmt::basic_format_string<T, Args...> fmt) {
[build]       |                                                 ^~~~~
[build] /usr/lib/gcc/x86_64-linux-gnu/14/../../../../include/c++/14/format:128:12: note: 'std::basic_format_string' declared here
[build]   128 |     struct basic_format_string
[build]       |            ^
[build] 1 error generated.

using fmt main branch, after the 11.0.2 release, `fmt::basic_format_string` is defined in `xchar.h`.
Otherwise compile error:
```
spdlog/common.h:369:49: error: no template named 'basic_format_string' in namespace 'fmt'; did you mean 'std::basic_format_string'?
[build]   369 | inline fmt::basic_string_view<T> to_string_view(fmt::basic_format_string<T, Args...> fmt) {
[build]       |                                                 ^~~~~
[build] /usr/lib/gcc/x86_64-linux-gnu/14/../../../../include/c++/14/format:128:12: note: 'std::basic_format_string' declared here
[build]   128 |     struct basic_format_string
[build]       |            ^
[build] 1 error generated.
```
@gabime
Copy link
Copy Markdown
Owner

gabime commented Nov 30, 2024

Something is wrong. It is already defined here

#include <fmt/xchar.h>

@ahoarau
Copy link
Copy Markdown
Author

ahoarau commented Nov 30, 2024

@gabime I just saw that.. Not the right patch for the issue. Investiguating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants