-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Added basic Pdh API implementation #469
Conversation
@@ -47,6 +47,7 @@ Features | |||
* [#432](https://github.com/twall/jna/pull/432): Added SetLocalTime definition to 'com.sun.jna.platform.win32.Kernel32' - [@lgoldstein](https://github.com/lgoldstein). | |||
* [#434](https://github.com/twall/jna/pull/434): Added GetEnvironmentStrings to 'com.sun.jna.platform.win32.Kernel32' - [@lgoldstein](https://github.com/lgoldstein). | |||
* Loosen OSGI OS name matching to accommodate Windows 8 family - Niels Bertram. | |||
* [#436] (https://github.com/twall/jna/pull/469): Added basic Pdh API implementation to `com.sun.jna.platform.win32' - [@lgoldstein](https://github.com/lgoldstein). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The second quote should also be a back quote, this doesn't render right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done - will post an update pull request
This is nice and clean, great work! Fix the typo in CHANGELOG please via |
public int HeaderLength; | ||
public int NumObjectTypes; | ||
public int DefaultObject; | ||
public SYSTEMTIME SystemTime=new SYSTEMTIME(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might want to auto-format this file too, there should be a space around the =
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done - will post an updated pull request
FWIW performance counters are one of the best features of Windows and I have used it extensively. I encourage you to write up a blog post somewhere with an example of how to use this. |
Merging this, thanks. |
Added basic Pdh API implementation
Added some (not all) of the API(s) in the pdh.dll - enough to enable using raw performance counters - as can be clearly demonstrated from the test.