Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[release/8.0-rc2] IDispatch should accept HRESULT as valuetype #92494

Merged
merged 4 commits into from
Sep 23, 2023
Merged

[release/8.0-rc2] IDispatch should accept HRESULT as valuetype #92494

merged 4 commits into from
Sep 23, 2023

Commits on Sep 22, 2023

  1. IDispatch should accept HRESULT as valuetype 

    This is a regression from .NET Framework.
The current behavior has existed since
IDispatch was introduced into .NET Core.
Added tests for the current behavior.
    @AaronRobinsonMSFT
    AaronRobinsonMSFT committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    def03f0 View commit details
    Browse the repository at this point in the history

  2. Ensure vtables line up for early bound.

    @AaronRobinsonMSFT
    AaronRobinsonMSFT committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    de3d2bf View commit details
    Browse the repository at this point in the history

  3. Add larger comment to workaround.

    @AaronRobinsonMSFT
    AaronRobinsonMSFT committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    922b53a View commit details
    Browse the repository at this point in the history

  4. Comment

    @AaronRobinsonMSFT
    AaronRobinsonMSFT committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    1507ca8 View commit details
    Browse the repository at this point in the history