task run not needed

neo4j · May 12, 2022 · e7508e2 · e7508e2
1 parent deae0c5
commit e7508e2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Neo4j.Driver/Neo4j.Driver/Internal/AsyncQueryRunner.cs b/Neo4j.Driver/Neo4j.Driver/Internal/AsyncQueryRunner.cs
@@ -69,16 +69,16 @@ protected virtual void Dispose(bool disposing)
 
 		public async ValueTask DisposeAsync()
 		{
-			await DisposeAsyncCore();
+			await DisposeAsyncCore().ConfigureAwait(false);
 
 			Dispose(disposing: false);
 			GC.SuppressFinalize(this);
 		}
 
-		protected virtual async ValueTask DisposeAsyncCore()
+		protected virtual ValueTask DisposeAsyncCore()
 		{
 			//Nothing to dispose of in this class. Methods required for derived classes and correct dispose pattern
-			await Task.CompletedTask;
+			return new ValueTask(Task.CompletedTask);
 		}
 	}
 

diff --git a/Neo4j.Driver/Neo4j.Driver/Internal/AsyncSession.cs b/Neo4j.Driver/Neo4j.Driver/Internal/AsyncSession.cs
@@ -264,7 +264,7 @@ protected override void Dispose(bool disposing)
                 //Dispose managed resources
 
                 //call it synchronously
-                Task.Run(CloseAsync).GetAwaiter().GetResult();
+                CloseAsync().GetAwaiter().GetResult();
             }
 
             _disposed = true;