Skip to content

Replace expensive WMI query with Win32 call - #2534

Closed
Steve Lee (SteveL-MSFT) wants to merge 8 commits into
PowerShell:masterfrom
SteveL-MSFT:master
Closed

Replace expensive WMI query with Win32 call#2534
Steve Lee (SteveL-MSFT) wants to merge 8 commits into
PowerShell:masterfrom
SteveL-MSFT:master

Conversation

@SteveL-MSFT

Copy link
Copy Markdown
Member

TFS:9333493

@msftclas

Hi Steve Lee (@SteveL-MSFT), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Steve Lee (POWERSHELL)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@mirichmo

Copy link
Copy Markdown
Member

Your commit history is cluttered. If you submit this PR from a separate branch other than master, it will not push the entire history of your master branch.

Mike Richmond (mirichmo) pushed a commit that referenced this pull request Oct 31, 2016
)

* Fixes #2534 by replacing expensive WMI query with Win32 API calls

* fix break on unix build

* added tests for #2535

* although test passed, fixing exception that shows up

* fixed Describe text

* addressing code review feedback

* addressing review feedback to comment on why sleep is needed
added check that test processes are created before we try to kill them

* fixed test to timeout and pending fix for #2561
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
… calls (PowerShell#2535)

* Fixes PowerShell#2534 by replacing expensive WMI query with Win32 API calls

* fix break on unix build

* added tests for PowerShell#2535

* although test passed, fixing exception that shows up

* fixed Describe text

* addressing code review feedback

* addressing review feedback to comment on why sleep is needed
added check that test processes are created before we try to kill them

* fixed test to timeout and pending fix for PowerShell#2561
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.

3 participants