ValueTask usage cleanups

Author: csrakowski

CSRakowski.AsyncStreamsPreparations.Tests/BasicApiTests.cs

@@ -21,15 +21,15 @@ public async Task SimpleForeachLogic()
             long summedTotal = 0;
             try
             {
-                while (await enumerator.MoveNextAsync())
+                while (await enumerator.MoveNextAsync().ConfigureAwait(false))
                 {
                     int current = enumerator.Current;
                     summedTotal += current;
                 }
             }
             finally
             {
-                await enumerator.DisposeAsync();
+                await enumerator.DisposeAsync().ConfigureAwait(false);
             }
 
             Assert.That(expectedSum == summedTotal, "Summed total should be {0}, but was found to be {1}", expectedSum, summedTotal);

CSRakowski.AsyncStreamsPreparations.Tests/CSRakowski.AsyncStreamsPreparations.Tests.csproj

@@ -8,7 +8,7 @@
     <Authors>Christiaan Rakowski</Authors>
     <Copyright>Christiaan Rakowski - 2018-2021</Copyright>
     <LangVersion>latest</LangVersion>
-    <Version>1.3.0</Version>
+    <Version>1.4.1</Version>
     <RootNamespace>CSRakowski.AsyncStreamsPreparations.Tests</RootNamespace>
     <AssemblyName>CSRakowski.AsyncStreamsPreparations.Tests</AssemblyName>
 

CSRakowski.AsyncStreamsPreparations/AsyncEnumerable.cs

@@ -53,7 +53,7 @@ internal AsyncEnumerator(IEnumerator<T> enumerator)
         /// of the collection.
         /// </returns>
         public ValueTask<bool> MoveNextAsync() =>
-            new ValueTask<bool>(Task.FromResult(_enumerator.MoveNext()));
+            new ValueTask<bool>(_enumerator.MoveNext());
 
         /// <summary>
         /// Gets the element in the collection at the current position of the enumerator.
@@ -70,6 +70,8 @@ public ValueTask DisposeAsync()
 
 #if NETSTANDARD1_1
             return new ValueTask(Task.FromResult(true));
+#elif NET5_0_OR_GREATER
+            return ValueTask.CompletedTask;
 #else
             return new ValueTask(Task.CompletedTask);
 #endif

CSRakowski.AsyncStreamsPreparations/CSRakowski.AsyncStreamsPreparations.csproj

@@ -15,8 +15,8 @@
     <LangVersion>latest</LangVersion>
     <SignAssembly>true</SignAssembly>
     <AssemblyOriginatorKeyFile>CSRakowski.AsyncStreamsPreparations.snk</AssemblyOriginatorKeyFile>
-    <PackageReleaseNotes>* Dropped netcoreapp3.0.</PackageReleaseNotes>
-    <Version>1.4.0</Version>
+    <PackageReleaseNotes>* ValueTask usage cleanups.</PackageReleaseNotes>
+    <Version>1.4.1</Version>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(TargetFramework)' == 'net50' or '$(TargetFramework)' == 'netcoreapp3.1' or '$(TargetFramework)' == 'netstandard2.1'">