Support for AsyncVoidMethodBuilder, AsyncValueTaskMethodBuilder and AsyncIteratorMethodBuilder

[igitur]

Hi,

I noticed that in

MethodBoundaryAspect.Fody/src/MethodBoundaryAspect.Fody/AsyncMethodWeaver.cs

Line 223 in 1c94bab

if (i.Operand is FieldReference field && field.FieldType.FullName.StartsWith(typeof(AsyncTaskMethodBuilder).FullName))

only AsyncTaskMethodBuilder is supported. This causes MethodBoundaryAspect.Fody to choke on some other async methods.

e.g. if I have a class:

[MyAspect]
public class AsyncClass
{
    public async void MyVoidMethod()
    {
        await Task.Delay(10);
    }

    public async ValueTask<int> MyValueTask()
    {
        await Task.Delay(10);
        return 10;
    }    
}

TODO: I still have to add an example for Async iterators, but I think you'll get the idea.

I can try my hand at a PR, but just want to check whether support for these are deliberately omitted or any pitfulls that I should be aware of.

[Ralf1108]

Hi @igitur, we would welcome PR if they improve the MBA.
The support for async methods where done by @keith-anders and unfortunatly he is not active anymore. So I am not aware of any reasons why it was not included. Maybe there wasn't a need for it.
Please feel free to improve the support for async methods.