-
Notifications
You must be signed in to change notification settings - Fork 116
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
COM interfaces should not redefine members from base interfaces #119
Comments
I think it's just this particular type because this is not a valid COM interface: class IVssWriterComponentsExt :
public IVssWriterComponents,
public IUnknown
{
}; COM requires single inheritance and this is, well, something else. |
I would just skip over this type when parsing - Steve should probably just exclude it. |
There are several such types. So lets fix or exclude all of them. |
It looks like all of these related "interfaces" are C++ classes and not interfaces. The docs call them that as well. I guess I will remove them unless anyone has a better idea. |
Other problematic interfaces:
|
I believe we previously agreed that COM interfaces in the metadata should derive from interfaces and should not redefine members from those base interfaces, instead leaving that to the projection.
However several interfaces (e.g.
IVssWriterComponentsExt
) are redefining members from the base interfaces, which is causing problems with our C# projection.The text was updated successfully, but these errors were encountered: