Skip to content

Conversation

@elinor-fung
Copy link
Member

  • Always treat IDispatch::GetTypeInfoCount returning a count of 0 as not supporting type info (even when the returned HRESULT is failure)
  • Remove block for getting type lib if an assembly is not marked ImportedFromTypeLib

Fixes #47329

cc @AaronRobinsonMSFT @jkoritzinsky

@elinor-fung elinor-fung added this to the 6.0.0 milestone Feb 9, 2021
Copy link
Member

@AaronRobinsonMSFT AaronRobinsonMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@elinor-fung we spoke offline about this and I agree porting this back to .NET 5 doesn't seem to much effort and probably worth it. We would likely need a user impact request though.

@elinor-fung
Copy link
Member Author

Ergh. I guess the NotSupportedException for marshalling structs in variants was based on the check in GetITypeLibForAssembly? -_-

@elinor-fung elinor-fung merged commit c8e3d8c into dotnet:master Feb 17, 2021
@elinor-fung elinor-fung deleted the dynamicNetServer branch February 17, 2021 00:32
elinor-fung added a commit to elinor-fung/runtime that referenced this pull request Feb 18, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invoking method of .NET 5 COM object (inproc server) from .NET 5 client app using the 'dynamic' keyword throws 'NotSupportedException'

2 participants