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

Rename reserved variables in shift.hpp #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcelwa
Copy link

The file shift.hpp defined macro variables _x0 to _x688. Generally, names starting with _ are reserved by the C++ standard and might invoke undefined behavior when used. In this specific case, _x86 is used by the Windows SDK, more precisely winnt.h. It thus causes compilation issues when used on modern Windows platforms. This PR fixes that issue.

@HappyCat2
Copy link

HappyCat2 commented Nov 25, 2024 via email

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.

2 participants