From d2b8eeae813acbbb5051d016f5a8e6512553cc0d Mon Sep 17 00:00:00 2001
From: Bart De Smet <bartdesmet@hotmail.com>
Date: Mon, 25 Feb 2019 13:33:55 -0800
Subject: [PATCH] Add comment to SelectMany.

---
 .../System.Linq.Async/System/Linq/Operators/SelectMany.cs       | 2 ++
 1 file changed, 2 insertions(+)

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
index 86763b4c17..9c60a7c046 100644
--- 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)