search

StackExchange / wmi

WMI for Go
http://godoc.org/github.com/StackExchange/wmi
MIT License
433 stars 173 forks source link

should be `runtime.UnlockOSThread()` #43

Closed flowchartsman closed 6 years ago

flowchartsman commented 6 years ago

This appears to be a typo: https://github.com/StackExchange/wmi/blob/cdffdb33acae0e14efff2628f9bae377b597840e/swbemservices.go#L76-L80

captncraig commented 6 years ago

That's... embarrassing.

If that alone accounts for the unexplained memory leaks we've seen....

I don't want to know enough to look at history on it. Thanks for the spot.

captncraig commented 6 years ago

Fixed in b12b22c5341f0c26d88c4d66176330500e84db68