Skip to content

Conversation

@firewave
Copy link
Collaborator

No description provided.

@firewave

This comment was marked as outdated.

@firewave
Copy link
Collaborator Author

This also needs some kind of hook to implicitly replace the TCHAR and _t*() variables to use the proper functions.

@firewave firewave changed the title replaced hard-coded platform type checks with additional Platform attributes replaced hard-coded Windows platform type checks with additional Platform attributes Oct 23, 2024
@firewave firewave force-pushed the platfowm-w branch 3 times, most recently from e0d41c0 to 88a72a9 Compare October 28, 2024 09:49
@firewave firewave force-pushed the platfowm-w branch 3 times, most recently from 4b7d4d7 to e1c108a Compare November 12, 2025 15:17
@sonarqubecloud
Copy link

@firewave firewave changed the title replaced hard-coded Windows platform type checks with additional Platform attributes added Platform::windows to specify if a platform is a Windows one Jan 12, 2026
@firewave firewave changed the title added Platform::windows to specify if a platform is a Windows one fixed #14384 - added Platform::windows to specify if a platform is a Windows one Jan 12, 2026
@firewave
Copy link
Collaborator Author

This also needs some kind of hook to implicitly replace the TCHAR and _t*() variables to use the proper functions.

I filed about it https://trac.cppcheck.net/ticket/14385. This ties into the proper handling of the UNICODE define.

@firewave firewave marked this pull request as ready for review January 12, 2026 02:41
@sonarqubecloud
Copy link

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.

1 participant