Add comment to SelectMany.

dotnet · Feb 25, 2019 · d2b8eea · d2b8eea
1 parent ad78405
commit d2b8eea
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/Ix.NET/Source/System.Linq.Async/System/Linq/Operators/SelectMany.cs b/Ix.NET/Source/System.Linq.Async/System/Linq/Operators/SelectMany.cs
@@ -21,6 +21,8 @@ public static IAsyncEnumerable<TResult> SelectMany<TSource, TResult>(this IAsync
             return new SelectManyAsyncIterator<TSource, TResult>(source, selector);
         }
 
+        // REVIEW: Should we keep these overloads that return ValueTask<IAsyncEnumerable<TResult>>? One could argue the selector is async twice.
+
         public static IAsyncEnumerable<TResult> SelectMany<TSource, TResult>(this IAsyncEnumerable<TSource> source, Func<TSource, ValueTask<IAsyncEnumerable<TResult>>> selector)
         {
             if (source == null)