Merge 37984a6 into dd8e8a8

morelinq · Oct 23, 2023 · 40023eb · 40023eb
2 parents dd8e8a8 + 37984a6
commit 40023eb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/MoreLinq.Test/NullArgumentTest.cs b/MoreLinq.Test/NullArgumentTest.cs
@@ -124,7 +124,6 @@ static bool CanBeNull(ParameterInfo parameter)
                 nameof(MoreEnumerable.From) + ".function1",
                 nameof(MoreEnumerable.From) + ".function2",
                 nameof(MoreEnumerable.From) + ".function3",
-                nameof(MoreEnumerable.ToDataTable) + ".expressions",
                 nameof(MoreEnumerable.Trace) + ".format"
             };
 

diff --git a/MoreLinq/ToDataTable.cs b/MoreLinq/ToDataTable.cs
@@ -97,10 +97,7 @@ public static TTable ToDataTable<T, TTable>(this IEnumerable<T> source, TTable t
         {
             if (source == null) throw new ArgumentNullException(nameof(source));
             if (table == null) throw new ArgumentNullException(nameof(table));
-
-            // TODO disallow null for "expressions" in next major update
-
-            expressions ??= EmptyArray<Expression<Func<T, object?>>>.Value;
+            if (expressions == null) throw new ArgumentNullException(nameof(expressions));
 
             var members = PrepareMemberInfos(expressions).ToArray();
             var boundMembers = BuildOrBindSchema(table, members);