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

Windows additions #3040

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

Conversation

NicknEma
Copy link
Contributor

I started to add missing bindings for user32, and the required types; to make the process easier, I sorted the list alphabetically, that's why it looks like a lot of lines were deleted. I also fixed some incorrect existing declarations (for example, they used int instead of c_int/i32) and added some comments.

Added some missing declarations, started reordering
Reordered functions from "Windows and Messages" section
Added "A" procedures
Added "B" functions
Added HDWP
Removed semicolons and a comment
Some functions returned "int" where they should've returned a "c_int".
Added "S", "T", "U" and "W" procedures from "Windows and Messages" section
Added SENDASYNCPROC
Some more fixes to slightly incorrect function signatures (es. int vs c_int), minor stylistic changes and some todo comments.
@NicknEma
Copy link
Contributor Author

No more *A versions of procedures, at least in user32.odin. I also finished adding all those stubs I left commented out before.

@github-actions github-actions bot added the stale label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants