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

Wrong output type for CallNtPowerInformation #990

Closed
daladim opened this issue Jul 7, 2022 · 4 comments
Closed

Wrong output type for CallNtPowerInformation #990

daladim opened this issue Jul 7, 2022 · 4 comments
Assignees
Labels
bug Something isn't working usability Touch-up to improve the user experience for a language projection

Comments

@daladim
Copy link

daladim commented Jul 7, 2022

CallNtPowerInformation original signature says it returns an NTSTATUS

However, its Rust binding returns an int (i64, instead of the proper NTSTATUS type)

I haven't digged into win32metadata, but I think I'm creating an issue at the right place. If not, please tell me.

Thanks

@kennykerr
Copy link
Contributor

Yes, this is the right place. Thanks for reporting!

@daladim
Copy link
Author

daladim commented Jul 7, 2022

All right :)

Also, while I'm on it: PROCESSOR_POWER_INFORMATION is missing a Rust binding as well (whereas other possible output types for CallNtPowerInformation, e.g. SYSTEM_BATTERY_STATE have theirs...)

@kennykerr
Copy link
Contributor

Thanks, I'll reference this in a new issue. In future, please create a new issue. It's easy to lose sight of unrelated issues in comments.

@mikebattista mikebattista self-assigned this Aug 1, 2022
@mikebattista mikebattista added bug Something isn't working usability Touch-up to improve the user experience for a language projection labels Aug 1, 2022
@mikebattista
Copy link
Contributor

Windows.Win32.System.Power.Apis.CallNtPowerInformation : return...Int32 => NTSTATUS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working usability Touch-up to improve the user experience for a language projection
Projects
None yet
Development

No branches or pull requests

3 participants