search

BADF00D / DisposableFixer

This is a Visual Studio Extension and NuGet package that should identify and fix problems as memleaks while using IDisposables.
Other
35 stars 7 forks source link

Add support for IAsyncDisposable #150

Open TwentyFourMinutes opened 4 years ago

TwentyFourMinutes commented 4 years ago

Prerequisites

Version: 3.3.0

Description

If I dispose a MemoryStream with DisposeAsync the Analyzer still yells at me DF0010. I really searched the whole repo, but couldn't find anything regarding to async disposal.

Source Code

var memoryStream = new MemoryStream();

await memoryStream.DisposeAsync();
BADF00D commented 4 years ago

Hello, First of all, thank you for reporting. You are right, there is no support for IAsyncDisposable yet. Neither when recognizing nor with CodeFix. Unfortunately, I am currently working on a project that uses VS2017 and .Net 4.5.2. So I don't stumble upon these shortcomings myself and am dependent on other people who do this.

Basically, it shouldn't be a big deal to do this, but thanks to the current pandemia, I don't have the time. So it can take some time for me to fix the problem.

TwentyFourMinutes commented 4 years ago

Sure thing, just wanted to note it and get some reports on it. Thanks for letting me know though.