Closed lucasg closed 4 years ago
Hi !
Thanks for the PR, I definitvely want to integrate your work to PythonForWindows.
I will take some time on my side to try to understand the underlying APIs and their capabilities.
Il will also see how to propely integrate your work to the ctypes_generation
and winproxy
.
I made some adaptations for the PythonForWindows codebase and created some samples/documentation. I hope the resulting code/APIs/samples still suits you. For the PR itself, your commits have been rebased on master after some works on my side. (For documentation purpose the first/last commits related to this PR on master are e74089cd / 176066d9)
As far as I can tell, it means:
But, as you can see, this PR is not marked as merged. It may because of my erratic git workflow :D I am not aware of a way to mark in as resolved in the UI.. I may be forced to simply close it.
Anyway thank you a lot for your contribution !
Salut,
Je te pose une grosse PR bien baveuse comme asse, avec du code de qualitay. Hésite pas a me tacler sur ce qu'il te semble mal écrit ou pas clair.
Il y a pas longtemps j'ai eu besoin de lister les resources prises par les devices qui tournent (IO ports, MMIO, etc.) sans devoir cliquer partout dans le Device Manager.
Hé bien dis toi que c'est bien la galère pour faire ça, il y a même pas de requete WMI adaptée. Non, à la place faut aller taper dans la bonne vieille API Win32, et plus spécifiquement dans
setupapi.dll
(qui est designée avec les fesses) etcfmgr32.dll
dont le créateur est probablement pas tout seul dans sa tête.Bref, la douleur habituelle mais au final on s'en sort :