Skip to content

Fix crash on x64 windows #4

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Conversation

MrTraan
Copy link

@MrTraan MrTraan commented Aug 23, 2019

Switch DWORD to DWORD_PTR type for parameter dwInstance in waveOutProcWrap, as using a 32bit value as a pointer on a 64 bit system would result in a crash

Switch DWORD to DWORD_PTR type for parameter dwInstance in waveOutProcWrap, as using a 32bit value as a pointer on a 64 bit system would result in a crash
@joshbodily
Copy link

Per https://learn.microsoft.com/en-us/previous-versions/dd743869(v=vs.85), all three params (dwPointer, dwParam1, dwParam2) should be DWORD_PTR Also, consider updating dwParam1 & dwParam2 here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants