Removes the redundant external condition (#102883)

Since we're anyway verifying only for isCancelled, there's no reason why we're checking for `isCompleted` externally, especially given that it's an OR.
dotnet · Jun 23, 2024 · d38e559 · d38e559
1 parent 0fc2809
commit d38e559
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/...tem.Text.Json/src/System/Text/Json/Serialization/Metadata/JsonTypeInfoOfT.WriteHelpers.cs b/...tem.Text.Json/src/System/Text/Json/Serialization/Metadata/JsonTypeInfoOfT.WriteHelpers.cs
@@ -118,12 +118,9 @@ private async Task SerializeAsync(
                     }
 
                     FlushResult result = await pipeWriter.FlushAsync(cancellationToken).ConfigureAwait(false);
-                    if (result.IsCanceled || result.IsCompleted)
+                    if (result.IsCanceled)
                     {
-                        if (result.IsCanceled)
-                        {
-                            ThrowHelper.ThrowOperationCanceledException_PipeWriteCanceled();
-                        }
+                        ThrowHelper.ThrowOperationCanceledException_PipeWriteCanceled();
                     }
                 }
                 finally