-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Re-generate System.Collections.Immutable ref assembly and fix a param name #38264
Re-generate System.Collections.Immutable ref assembly and fix a param name #38264
Conversation
src/System.Collections.Immutable/src/System/Collections/Immutable/ImmutableSortedDictionary.cs
Outdated
Show resolved
Hide resolved
13ec5f2
to
b794c56
Compare
This is ready to be merged. PTAL |
@@ -96,7 +96,7 @@ public static partial class ImmutableArray | |||
} | |||
public partial struct ImmutableArray<T> : System.Collections.Generic.ICollection<T>, System.Collections.Generic.IEnumerable<T>, System.Collections.Generic.IList<T>, System.Collections.Generic.IReadOnlyCollection<T>, System.Collections.Generic.IReadOnlyList<T>, System.Collections.ICollection, System.Collections.IEnumerable, System.Collections.IList, System.Collections.Immutable.IImmutableList<T>, System.Collections.IStructuralComparable, System.Collections.IStructuralEquatable, System.IEquatable<System.Collections.Immutable.ImmutableArray<T>> | |||
{ | |||
internal T[] array; | |||
private T[] array; |
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.
Question: How do we know this is the right change (internal -> private)?
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.
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 is fine, as long as it is what GenAPI generates.
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.
Since the ref doesn't have an [InternalsVisibleTo]
there shouldn't be any observable difference in those access levels.
… name (dotnet/corefx#38264) * Re-generate System.Collections.Immutable ref assembly and fix a param name * Use builder as the param name instead Commit migrated from dotnet/corefx@2e6661d
Fixes: https://github.com/dotnet/corefx/issues/38110
Picked to preserve
source
as the param name since that is what we use in other APIs, so I wanted to be consistent.cc: @stephentoub @ahsonkhan @danmosemsft