update type annotations for `_IUnknown_Base` and `IUnknown`

enthought / comtypes

A pure Python, lightweight COM client and server framework, based on the ctypes Python FFI package.

Other

290 stars 96 forks source link

update type annotations for `_IUnknown_Base` and `IUnknown` #540

Closed junkmd closed 4 months ago

junkmd commented 4 months ago

Defining __com_AddRef, __com_Release, and __com_QueryInterface as members of _IUnknown_Base, type checkers interpret these members as accessible not only to IUnknown but also to their subclasses.

I think that this could lead to confusion, I choose to remove the definition of these members from _IUnknown_Base.

junkmd commented 4 months ago

CIs are passed.

https://ci.appveyor.com/project/pywinauto/pywinauto/builds/49792592 https://github.com/enthought/comtypes/actions/runs/9039394519