8000 Use `EqualityComparer<T>.Create` from .NET 8 by atifaziz · Pull Request #1051 · morelinq/MoreLINQ · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Use EqualityComparer<T>.Create from .NET 8 #1051

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 20, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions MoreLinq.Test/EndsWithTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ public void EndsWithUsesSpecifiedEqualityComparerOrDefault()

Assert.That(first.EndsWith(second), Is.False);
Assert.That(first.EndsWith(second, null), Is.False);
Assert.That(first.EndsWith(second, EqualityComparer.Create<int>(delegate { return false; })), Is.False);
Assert.That(first.EndsWith(second, EqualityComparer.Create<int>(delegate { return true; })), Is.True);
Assert.That(first.EndsWith(second, EqualityComparer<int>.Create(delegate { return false; })), Is.False);
Assert.That(first.EndsWith(second, EqualityComparer<int>.Create(delegate { return true; })), Is.True);
}

[TestCase(SourceKind.BreakingCollection)]
Expand Down
15 changes: 11 additions & 4 deletions MoreLinq.Test/EqualityComparer.cs
Original file line number Diff line number Diff line change
Expand Up @@ -15,22 +15,27 @@
// limitations under the License.
#endregion

#if !NET8_0_OR_GREATER

namespace MoreLinq.Test
{
using System;
using System.Collections.Generic;

static class EqualityComparer
static class EqualityComparer<T>
{
public static System.Collections.Generic.EqualityComparer<T>
Default => System.Collections.Generic.EqualityComparer<T>.Default;

/// <summary>
/// Creates an <see cref="IEqualityComparer{T}"/> given a
/// <see cref="Func{T,T,Boolean}"/>.
/// </summary>

public static IEqualityComparer<T> Create<T>(Func<T?, T?, bool> comparer) =>
new DelegatingComparer<T>(comparer);
public static IEqualityComparer<T> Create(Func<T?, T?, bool> comparer) =>
new DelegatingComparer(comparer);

sealed class DelegatingComparer<T> : IEqualityComparer<T>
sealed class DelegatingComparer : IEqualityComparer<T>
{
readonly Func<T?, T?, bool> comparer;
readonly Func<T, int> hasher;
Expand All @@ -49,3 +54,5 @@ public DelegatingComparer(Func<T?, T?, bool> comparer)
}
}
}

#endif
2 changes: 1 addition & 1 deletion MoreLinq.Test/PermutationsTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ public void TestPermutationsAreIndependent()
static class SequenceEqualityComparer<T>
{
public static readonly IEqualityComparer<IEnumerable<T>> Instance =
EqualityComparer.Create<IEnumerable<T>>((x, y) => x is { } sx && y is { } sy && sx.SequenceEqual(sy));
EqualityComparer<IEnumerable<T>>.Create((x, y) => x is { } sx && y is { } sy && sx.SequenceEqual(sy));
}
}
}
4 changes: 2 additions & 2 deletions MoreLinq.Test/StartsWithTest.cs
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ public void StartsWithUsesSpecifiedEqualityComparerOrDefault()

Assert.That(first.StartsWith(second), Is.False);
Assert.That(first.StartsWith(second, null), Is.False);
Assert.That(first.StartsWith(second, EqualityComparer.Create<int>(delegate { return false; })), Is.False);
Assert.That(first.StartsWith(second, EqualityComparer.Create<int>(delegate { return true; })), Is.True);
Assert.That(first.StartsWith(second, EqualityComparer<int>.Create(delegate { return false; })), Is.False);
Assert.That(first.StartsWith(second, EqualityComparer<int>.Create(delegate { return true; })), Is.True);
}

[TestCase(SourceKind.BreakingCollection)]
Expand Down
0