-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Clean up dead code for System.Collections* #28840
Clean up dead code for System.Collections* #28840
Conversation
/// </summary> | ||
internal int Count | ||
{ | ||
get { return _count; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the _count
field used anywhere else? Should it be deleted too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I'll delete.
@@ -17,10 +17,6 @@ internal CompatibleComparer(IHashCodeProvider hashCodeProvider, IComparer compar | |||
_comparer = comparer; | |||
} | |||
|
|||
internal IHashCodeProvider HashCodeProvider => _hcp; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think splitting the file into two just because of these two trivial properties is not an improvement. I would keep it as it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was suggested to me, but I agree with you. Will fix.
/// <summary> | ||
/// Initializes a new instance of the <see cref="ImmutableDictionary{TKey, TValue}.MutationInput"/> struct. | ||
/// </summary> | ||
/// <param name="root">The root.</param> | ||
/// <param name="comparers">The comparers.</param> | ||
/// <param name="count">The current number of elements in the collection.</param> | ||
/// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Delete this empty comment line.
Marking no-merge to hold this for a week or so until master = 2.2 |
Commit migrated from dotnet/corefx@d7396b2
#17905