Skip to content
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

Visual Studio update breaks is_utf8_enabled() #4119

Closed
emveepee opened this issue Aug 13, 2024 · 4 comments
Closed

Visual Studio update breaks is_utf8_enabled() #4119

emveepee opened this issue Aug 13, 2024 · 4 comments
Labels

Comments

@emveepee
Copy link

I just updated to the latest VS 2022 and had tons of fmt warning which were removed when I updated to 11.0.2 and it reduced it to one that is repeating which might relate to #2297 The compiler shows the assert message "Unicode support requires compiling with /utf-8" but it doesn't matter if /utf-8 is added or not.

For now I am just returning true from is_utf8_enabled in base.h and haven't noticed anything. Is this a known issue or did the compiler fix get released change PR 2297?

@emveepee emveepee changed the title Visual Studio update Visual Studio update break is_utf8_enabled() Aug 13, 2024
@emveepee emveepee changed the title Visual Studio update break is_utf8_enabled() Visual Studio update breaks is_utf8_enabled() Aug 13, 2024
@vitaut
Copy link
Contributor

vitaut commented Aug 14, 2024

Please provide a repro, ideally on godbolt.

@vitaut
Copy link
Contributor

vitaut commented Aug 14, 2024

I was not able to repro this issue on MSVC 2022 (v19.30): https://www.godbolt.org/z/fo4aKMao7. Most likely the problem is in your build config, make sure that /utf-8 is correctly passed to the compiler.

@vitaut vitaut closed this as completed Aug 14, 2024
@vitaut vitaut added the invalid label Aug 14, 2024
@emveepee
Copy link
Author

I was not able to repro this issue on MSVC 2022 (v19.30): https://www.godbolt.org/z/fo4aKMao7. Most likely the problem is in your build config, make sure that /utf-8 is correctly passed to the compiler.

Yes sorry that was it I didn't pass /utf-8 in both debug and release. Sorry to bother you.

@vitaut
Copy link
Contributor

vitaut commented Aug 14, 2024

No worries, glad the issue has been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants