From d298b093b9d25797c8f4b7d4d2a4d3763e399189 Mon Sep 17 00:00:00 2001 From: James Newton-King Date: Wed, 26 Feb 2020 16:58:45 +1300 Subject: [PATCH] Fix closure allocation in scope DisposeAsync (dotnet/Extensions#3024) Commit migrated from https://github.com/dotnet/Extensions/commit/b190507b207f6d13f6dc7b00b33f8ca44ea3404b --- .../src/ServiceLookup/ServiceProviderEngineScope.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/libraries/Microsoft.Extensions.DependencyInjection/src/ServiceLookup/ServiceProviderEngineScope.cs b/src/libraries/Microsoft.Extensions.DependencyInjection/src/ServiceLookup/ServiceProviderEngineScope.cs index 07ee884..45d5ec6 100644 --- a/src/libraries/Microsoft.Extensions.DependencyInjection/src/ServiceLookup/ServiceProviderEngineScope.cs +++ b/src/libraries/Microsoft.Extensions.DependencyInjection/src/ServiceLookup/ServiceProviderEngineScope.cs @@ -98,7 +98,7 @@ namespace Microsoft.Extensions.DependencyInjection.ServiceLookup var vt = asyncDisposable.DisposeAsync(); if (!vt.IsCompletedSuccessfully) { - return Await(i, vt); + return Await(i, vt, toDispose); } // If its a IValueTaskSource backed ValueTask, @@ -119,7 +119,7 @@ namespace Microsoft.Extensions.DependencyInjection.ServiceLookup return default; - async ValueTask Await(int i, ValueTask vt) + static async ValueTask Await(int i, ValueTask vt, List toDispose) { await vt; -- 2.7.4