Skip to content

fix: add WINBOOL type#49

Closed
gynt wants to merge 1 commit intomainfrom
fix/resolve-type-conflicts-14-missing-type
Closed

fix: add WINBOOL type#49
gynt wants to merge 1 commit intomainfrom
fix/resolve-type-conflicts-14-missing-type

Conversation

@gynt
Copy link
Copy Markdown
Contributor

@gynt gynt commented Apr 12, 2026

Adds missing WINBOOL type in a predictable location

@gynt gynt requested a review from TheRedDaemon April 12, 2026 21:25
@@ -1,5 +1,7 @@
#pragma once

typedef int WINBOOL;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I am curious:
This one is not part of the Windows.h header?
From where is it and where do we need it?

Copy link
Copy Markdown
Contributor Author

@gynt gynt Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exported Direct x 8 stuff needs it (function signatures). It should be in Windows psapi.h but I couldn't find it. Therefore this stub.
I remember now, WINBOOL is in psapi.h in windows 10 SDK but not in the msvc1400 SDK. WINBOOL doesn't exist in MSVC1400 for some reason while it should. I also can't trace it using the dplay.h headers I found online.
Funnily enough, our dplay.h from 2007 just uses BOOL... I guess this is a non-issue and should be fixed in Ghidra instead.

@gynt gynt closed this Apr 13, 2026
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