Use AsTask() (#349)

graphql-dotnet · Oct 12, 2023 · 4911e52 · 4911e52
1 parent a22e1e0
commit 4911e52
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/src/GraphQLParser/Visitors/SDLPrinterExtensions.cs b/src/GraphQLParser/Visitors/SDLPrinterExtensions.cs
@@ -22,9 +22,7 @@ public static string Print(this SDLPrinter printer, ASTNode node)
     /// Prints the specified AST into the specified <see cref="StringBuilder"/> as a SDL document.
     /// </summary>
     public static void Print(this SDLPrinter printer, ASTNode node, StringBuilder stringBuilder)
-#pragma warning disable CA2012 // Use ValueTasks correctly
-        => printer.PrintAsync(node, new StringWriter(stringBuilder), default).GetAwaiter().GetResult();
-#pragma warning restore CA2012 // Use ValueTasks correctly
+        => printer.PrintAsync(node, new StringWriter(stringBuilder), default).AsTask().GetAwaiter().GetResult();
 
     /// <summary>
     /// Prints the specified AST into the specified <see cref="MemoryStream"/> as a SDL document.
@@ -33,9 +31,7 @@ public static void Print(this SDLPrinter printer, ASTNode node, StringBuilder st
     public static void Print(this SDLPrinter printer, ASTNode node, MemoryStream memoryStream, Encoding? encoding = null)
     {
         using var streamWriter = new StreamWriter(memoryStream, encoding, -1 /* default */, true);
-#pragma warning disable CA2012 // Use ValueTasks correctly
-        printer.PrintAsync(node, streamWriter, default).GetAwaiter().GetResult();
-#pragma warning restore CA2012 // Use ValueTasks correctly
+        printer.PrintAsync(node, streamWriter, default).AsTask().GetAwaiter().GetResult();
         // flush encoder state to stream
         streamWriter.Flush();
     }

diff --git a/src/GraphQLParser/Visitors/SDLSorter.cs b/src/GraphQLParser/Visitors/SDLSorter.cs
@@ -18,9 +18,7 @@ private SDLSorter()
     /// Nodes that have the same sort order will be retain their relative position.
     /// </summary>
     public static void Sort(ASTNode node, SDLSorterOptions? options = null)
-#pragma warning disable CA2012 // Use ValueTasks correctly
-        => _sorter.VisitAsync(node, options ?? SDLSorterOptions.Default).GetAwaiter().GetResult();
-#pragma warning restore CA2012 // Use ValueTasks correctly
+        => _sorter.VisitAsync(node, options ?? SDLSorterOptions.Default).AsTask().GetAwaiter().GetResult();
 
     /// <inheritdoc/>
     protected override ValueTask VisitAsync<T>(List<T>? nodes, SDLSorterOptions context)