Skip to content

Feature: remove tchar.h, bring the project code base to Unicode#55

Open
Demonese wants to merge 2 commits intomicrosoft:mainfrom
Demonese:feature/no-tchar
Open

Feature: remove tchar.h, bring the project code base to Unicode#55
Demonese wants to merge 2 commits intomicrosoft:mainfrom
Demonese:feature/no-tchar

Conversation

@Demonese
Copy link
Contributor

@Demonese Demonese commented Dec 26, 2024

Welcome to Unicode!

There is a problem of mixing char and tchar in this project.

So I spent some time migrating it to Unicode.

@walbourn
Copy link
Member

This codebase is ancient, so that's why TCHAR still was there... I appreciate the effort to clean it up.

@Demonese Demonese marked this pull request as ready for review August 6, 2025 02:59
@Demonese
Copy link
Contributor Author

Demonese commented Aug 6, 2025

I ran all the GitHub actions in my fork and they all passed.

I think it's ready to be merge.

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

Labels

None yet

2 participants