Skip to content

converted fractional_part_rounding_thresholds to use an array of u32s - #3604

Closed
SystemPanic wants to merge 1 commit into
gabime:v1.xfrom
SystemPanic:fix-format-windows
Closed

SystemPanic wants to merge 1 commit into
gabime:v1.xfrom
SystemPanic:fix-format-windows

Conversation

@SystemPanic

Copy link
Copy Markdown

MSVC fails to compile format.h with "character value is out of range" errors.

PR is the same already merged on fmtlib/fmt#4719

Signed-off-by: Javier <25750030+SystemPanic@users.noreply.github.com>
@tt4g

tt4g commented May 1, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution.
However, spdlog uses a method where the entire source code is replaced when a new version of fmt is released.
Please don’t be disappointed if this PR isn’t merged.

If you can't wait for the next fmt release, you can use an external fmt instead of the fmt bundled in spdlog.

@SystemPanic

SystemPanic commented May 1, 2026

Copy link
Copy Markdown
Author

@tt4g Sure, no problem. I already have a temporary patch for FlashInfer that fix it automatically.

Feel free to close this.

@gabime gabime closed this May 6, 2026
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.

3 participants