Consider using ValueStopWatch

[GeertvanHorrik]

Source code is available here:

https://github.com/dotnet/aspnetcore/blob/main/src/Shared/ValueStopwatch/ValueStopwatch.cs

This fody addon could check for the availability of the ValueStopwatch. If it exists, use it instead of Stopwatch.

Happy to PR if this is considered a good idea.

[ltrzesniewski]

I suppose you'd like to get rid of the allocation? I'd just use Stopwatch.GetTimestamp() and compare the ticks at the end - basically what ValueStopwatch does. Also, ValueStopwatch is internal.

[GeertvanHorrik]

Interesting comment, great observations. I can refactor so it replaces the StartNew by getting the timestamp and calculate the difference manually. Only thing to take into account is to convert the ticks to timespan, but that can be a 1-time static thing.

[GeertvanHorrik]

I just cloned the repository, and noticed that @SimonCropp already started a branch for this earlier this year. Not sure why it was finished, but happy to finalize the branch if needed.

[GeertvanHorrik]

The async methods are done. Now working on the async method processor.

[GeertvanHorrik]

Added some benchmarks to make sure that we are actually improving something:

The code is faster and allocation free, so I will continue the quest finalize this implementation.

[GeertvanHorrik]

All unit tests are now passing!