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/6.0] Remove locks from COM events delegate management. #76035

Merged
merged 9 commits into from
Oct 5, 2022
Merged

[release/6.0] Remove locks from COM events delegate management. #76035

merged 9 commits into from
Oct 5, 2022

Commits on Sep 22, 2022

  1. Remove locks from COM events delegate management. 

    This removes locks and instead assumes the collection is immutable.
    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    ebfb3f0 View commit details
    Browse the repository at this point in the history

  2. Remove uses of var.

    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    1d6afbc View commit details
    Browse the repository at this point in the history

  3. Use array instead of List<T>

    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    95172d8 View commit details
    Browse the repository at this point in the history

  4. Revert "Use array instead of List<T>" 

    This reverts commit 5ba348f.
    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    1836620 View commit details
    Browse the repository at this point in the history

  5. Revert "Revert "Use array instead of List<T>"" 

    This reverts commit aa3dddd.
    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    b1a3caa View commit details
    Browse the repository at this point in the history

  6. Remove usage of Delegate.Combine. Upon deeper inspection there does… 

    …n't seem to be any value to using that mechanism. An added bonus is that it simplifies the logic.
    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    9ef4e91 View commit details
    Browse the repository at this point in the history

  7. Use some sugar

    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    7d21627 View commit details
    Browse the repository at this point in the history

  8. Simplify the RemoveAll scenario.

    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    2f5e3cd View commit details
    Browse the repository at this point in the history

  9. Style nits.

    @AaronRobinsonMSFT
    AaronRobinsonMSFT authored and github-actions committed Sep 22, 2022
    Configuration menu
    Copy the full SHA
    fe3b73f View commit details
    Browse the repository at this point in the history