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

phnt: Fix some problems in ntuser.h #2427

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

xdqi
Copy link
Contributor

@xdqi xdqi commented Feb 15, 2025

I Found problems by linking to win32u.lib with the headers. :)

NtUserGetClassName

https://github.com/wine-mirror/wine/blob/4de563994426e258d1f2848b663f6ed85dd1298d/include/ntuser.h#L811
https://doxygen.reactos.org/dd/d79/include_2ntuser_8h.html#a1c63176401b004bdffb56c8e8ea42b50

NtUserOpenWindowStation, NtUserCreateWindowStation

https://github.com/wine-mirror/wine/blob/4de563994426e258d1f2848b663f6ed85dd1298d/include/ntuser.h#L899
Should be typos, no large struct are passed by value in syscall

NtUserCalculatePopupWindowPosition

typo

ChildWindowFromPoint

https://j00ru.vexillium.org/syscalls/nt/64/
There's only syscall NtUserChildWindowFromPointEx, but ChildWindowFromPoint exists in user32.dll

NtUserQueryInformationThread

https://doxygen.reactos.org/dd/d79/include_2ntuser_8h.html#a87b3e470ffd45fe18b9edc4e6f968484

@xdqi xdqi requested review from dmex and jxy-s as code owners February 15, 2025 13:35
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