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

Let #defined functions on 32-bit have same signatures as on 64-bit. #735

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

Conversation

IslandUsurper
Copy link

Because Rust treats isize and i32 as different types even when they are the same size, aliasing SetWindowLongPtrW to SetWindowLongW doesn't work without a cast. So, let's do the cast in winapi on x86.

@retep998
Copy link
Owner

This is a breaking change that will have to be postponed until the next major version.

@IslandUsurper
Copy link
Author

@retep998, should I redo this PR against master, then?

@retep998
Copy link
Owner

retep998 commented Jan 28, 2019

No, because that is not what master is for (and also I can change what branch a PR is against anyway, no need to recreate PRs).

@micwoj92
Copy link

This branch has conflicts that must be resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for 0.4 This issue will be fixed in 0.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants