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

Add mappings for Thread32First and Thread32Next to Kernel32 #1209

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

dbwiddis
Copy link
Contributor

No description provided.

@matthiasblaesing
Copy link
Member

Looks good

@matthiasblaesing matthiasblaesing merged commit 43c4e67 into java-native-access:master Jun 16, 2020
@dbwiddis dbwiddis deleted the thread32 branch June 16, 2020 19:35
@matthiasblaesing
Copy link
Member

I just noticed: this PR is broken. The structure definition of THREADENTRY32 is broken, as the member as declared package private.

https://ci.appveyor.com/project/dblock/jna-gsxuq/branch/master#L1640

@dbwiddis could you please recheck?

@dbwiddis
Copy link
Contributor Author

dbwiddis commented Jul 6, 2020

This falls into the "how did it ever work?" category of bugs. Will fix.

dbwiddis added a commit to dbwiddis/jna that referenced this pull request Jul 6, 2020
dbwiddis added a commit to dbwiddis/jna that referenced this pull request Jul 15, 2020
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