-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add ConcurrentDictionary.Comparer property #31350
Labels
api-approved
API was approved in API review, it can be implemented
area-System.Collections
help wanted
[up-for-grabs] Good issue for external contributors
Milestone
Comments
Makes sense from a consistency standpoint. |
Proposed API: namespace System.Collections.Concurrent
{
public partial class ConcurrentDictionary<TKey, TValue>
{
+ public IEqualityComparer<TKey> Comparer { get; }
}
} |
namespace System.Collections.Concurrent
{
public partial class ConcurrentDictionary<TKey, TValue>
{
public IEqualityComparer<TKey> Comparer { get; }
}
} |
I-SER-I
added a commit
to I-SER-I/runtime
that referenced
this issue
Feb 2, 2021
eiriktsarpalis
pushed a commit
that referenced
this issue
Feb 3, 2021
* fix #31350 * fix ref file * fix getter * added tests
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
api-approved
API was approved in API review, it can be implemented
area-System.Collections
help wanted
[up-for-grabs] Good issue for external contributors
It would be useful to expose this as a getter property. I believe all other BCL collections that accept comparers expose them as properties (i.e.
Dictionary<,>.Comparer
,HashSet<>.Comparer
).The text was updated successfully, but these errors were encountered: