Skip to content

Fix C4819 warning: replace non-ASCII em dash in base_macros.h comment#1606

Merged
DefaultRyan merged 2 commits into
masterfrom
copilot/fix-warning-c4819
Jul 8, 2026
Merged

Fix C4819 warning: replace non-ASCII em dash in base_macros.h comment#1606
DefaultRyan merged 2 commits into
masterfrom
copilot/fix-warning-c4819

Conversation

Copilot AI commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

An em dash (U+2014) in a comment in strings/base_macros.h triggered MSVC warning C4819 (character not representable in current code page), breaking builds with /WX enabled.

  • strings/base_macros.h: Replace with ASCII - in comment on line 47

Copilot AI linked an issue Jul 8, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix warning C4819 caused by non-ASCII character in C++/WinRT Fix C4819 warning: replace non-ASCII em dash in base_macros.h comment Jul 8, 2026
Copilot AI requested a review from DefaultRyan July 8, 2026 18:46
@DefaultRyan DefaultRyan marked this pull request as ready for review July 8, 2026 18:49
@sylveon

sylveon commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

A whole copilot summon for a 1 character change lol

@DefaultRyan

Copy link
Copy Markdown
Member

A whole copilot summon for a 1 character change lol

I know, right?

But seriously, I'm working on something else at the moment, and didn't feel like creating a branch and validating it myself. 🤷‍♂️

@DefaultRyan DefaultRyan merged commit a535d38 into master Jul 8, 2026
89 checks passed
@DefaultRyan DefaultRyan deleted the copilot/fix-warning-c4819 branch July 8, 2026 19:38
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.

Bug: C++/WinRT will cause warning C4819

6 participants