Skip to content

Use MutableDisposable in codeLensController#75286

Merged
jrieken merged 1 commit intomicrosoft:masterfrom
mjbvz:mutable-disposable-in-code-lens-controller
Jun 12, 2019
Merged

Use MutableDisposable in codeLensController#75286
jrieken merged 1 commit intomicrosoft:masterfrom
mjbvz:mutable-disposable-in-code-lens-controller

Conversation

@mjbvz
Copy link
Copy Markdown
Collaborator

@mjbvz mjbvz commented Jun 11, 2019

The MutableDisposable class represents a disposable value that can be changed. The class automatically disposes of the old value when a new one is set, which can prevent a common programming mistake. MutableDisposable is useful in places we are currently using private property?: IDisposable.

This change uses a MutableDisposable to manage the code lens model.

The `MutableDisposable` class represents a disposable value that can be changed. The class automatically disposeds of the old value when a new one is set, which can prevent a common programming mistake. `MutableDisposable` is useful in places we are currently using `private property?: IDisposable`.

This change uses a `MutableDisposable` to manage the code lens model.
@mjbvz mjbvz requested a review from jrieken June 11, 2019 17:28
@mjbvz mjbvz self-assigned this Jun 11, 2019
Copy link
Copy Markdown
Member

@jrieken jrieken left a comment

Choose a reason for hiding this comment

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

I like MutableDisposable - spread the word @mjbvz

@jrieken jrieken merged commit 622f083 into microsoft:master Jun 12, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants