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.

Sign up for GitHub

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

Jump to bottom

Fix various text encoding mixups #96

Closed
wants to merge 3 commits into from
Closed

Fix various text encoding mixups #96

wants to merge 3 commits into from

Conversation

maxmitti
Copy link
Contributor

@maxmitti maxmitti commented Nov 12, 2022 •
edited
Loading

  • Determine window titles with GetWindowTextW instead of GetWindowTextA and convert them to UTF-8
  • Convert injected characters and compose sequence characters to UTF-8 for logging

Fixes #93.

@maxmitti
Copy link
Contributor Author

Hier ist auch ein Build zum Testen: https://github.com/maxmitti/ReNeo/releases/tag/v0.1.0

@maxmitti maxmitti mentioned this pull request Nov 16, 2022
Closed
@maxmitti maxmitti changed the title Read window titles via the wstring API in order to support unicode Fix various encoding mixups Nov 16, 2022
@maxmitti maxmitti changed the title Fix various encoding mixups Fix various text encoding mixups Nov 16, 2022
@Rojetto
Copy link
Owner

Rojetto commented Jul 11, 2023

Thanks a lot for putting in the time to fix this! Sorry it took this long to merge, the changes will be in the next release.

@Rojetto Rojetto closed this Jul 11, 2023
@maxmitti maxmitti deleted the pr/widestring_window_title branch July 12, 2023 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Textkodierungschaos in Konsolenausgabe und Logdatei
2 participants
@maxmitti @Rojetto