Skip to content

Conversation

@Scottj1s
Copy link
Member

@Scottj1s Scottj1s commented Jun 8, 2021

fixes #842

@Scottj1s Scottj1s requested a review from manodasanW June 8, 2021 16:33
@Scottj1s Scottj1s added the AssemblyVersion change Requires assembly version change for WinRT.Runtime label Jun 8, 2021
@Scottj1s Scottj1s requested a review from ujjwalchadha June 9, 2021 17:27
@Scottj1s Scottj1s merged commit fa038af into master Jun 11, 2021
@Scottj1s Scottj1s deleted the scottj1s/event_leak branch June 11, 2021 02:15
Scottj1s added a commit that referenced this pull request Jun 15, 2021
…ibes across garbage collections (#861)"

This reverts commit fa038af.
manodasanW pushed a commit that referenced this pull request Jun 17, 2021
…ibes across garbage collections (#861)"

This reverts commit fa038af.
Scottj1s added a commit that referenced this pull request Jun 17, 2021
* Revert "Added caching of event registration state to support unsubscribes across garbage collections (#861)"

This reverts commit fa038af.

* fix merge bug and make weak ref interfaces public

* Add back null check to avoid future issues if older runtime is used.

Co-authored-by: Manodasan Wignarajah <mawign@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AssemblyVersion change Requires assembly version change for WinRT.Runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Garbage collection of EventSource<T> causes subsequent failure to unsubscribe handlers

2 participants