Skip to content
This repository was archived by the owner on Aug 8, 2024. It is now read-only.

Commit b730b3c

Browse files
tarekghstephentoub
tarekgh
authored and
stephentoub
committed
Expose Index and Range types (dotnet/coreclr#20899)
* Expose Index and Range types * Address Review Comments * Address more feedback * Addressing more comments Signed-off-by: dotnet-bot <[email protected]>
1 parent ab9570b commit b730b3c

File tree

5 files changed

+125
-1
lines changed

5 files changed

+125
-1
lines changed

‎src/Common/src/CoreLib/System.Private.CoreLib.Shared.projitems‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,7 @@
222222
<Compile Include="$(MSBuildThisFileDirectory)System\IEquatable.cs" />
223223
<Compile Include="$(MSBuildThisFileDirectory)System\IFormatProvider.cs" />
224224
<Compile Include="$(MSBuildThisFileDirectory)System\IFormattable.cs" />
225+
<Compile Include="$(MSBuildThisFileDirectory)System\Index.cs" />
225226
<Compile Include="$(MSBuildThisFileDirectory)System\IndexOutOfRangeException.cs" />
226227
<Compile Include="$(MSBuildThisFileDirectory)System\InsufficientExecutionStackException.cs" />
227228
<Compile Include="$(MSBuildThisFileDirectory)System\InsufficientMemoryException.cs" />
@@ -308,6 +309,7 @@
308309
<Compile Include="$(MSBuildThisFileDirectory)System\PlatformNotSupportedException.cs" />
309310
<Compile Include="$(MSBuildThisFileDirectory)System\Progress.cs" />
310311
<Compile Include="$(MSBuildThisFileDirectory)System\Random.cs" />
312+
<Compile Include="$(MSBuildThisFileDirectory)System\Range.cs" />
311313
<Compile Include="$(MSBuildThisFileDirectory)System\RankException.cs" />
312314
<Compile Include="$(MSBuildThisFileDirectory)System\ReadOnlySpan.cs" />
313315
<Compile Include="$(MSBuildThisFileDirectory)System\ReadOnlySpan.Fast.cs" />
@@ -631,7 +633,7 @@
631633
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SemaphoreFullException.cs" />
632634
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SemaphoreSlim.cs" />
633635
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SendOrPostCallback.cs" />
634-
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SpinLock.cs" />
636+
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SpinLock.cs" />
635637
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SpinWait.cs" />
636638
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\SynchronizationLockException.cs" />
637639
<Compile Include="$(MSBuildThisFileDirectory)System\Threading\ThreadLocal.cs" />

‎src/Common/src/CoreLib/System/Index.cs‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
// Licensed to the .NET Foundation under one or more agreements.
2+
// The .NET Foundation licenses this file to you under the MIT license.
3+
// See the LICENSE file in the project root for more information.
4+
5+
namespace System
6+
{
7+
public readonly struct Index : IEquatable<Index>
8+
{
9+
private readonly int _value;
10+
11+
public Index(int value, bool fromEnd)
12+
{
13+
if (value < 0)
14+
{
15+
ThrowHelper.ThrowValueArgumentOutOfRange_NeedNonNegNumException();
16+
}
17+
18+
_value = fromEnd ? ~value : value;
19+
}
20+
21+
public int Value => _value < 0 ? ~_value : _value;
22+
public bool FromEnd => _value < 0;
23+
public override bool Equals(object value) => value is Index && _value == ((Index)value)._value;
24+
public bool Equals (Index other) => _value == other._value;
25+
26+
public override int GetHashCode()
27+
{
28+
return _value;
29+
}
30+
31+
public override string ToString()
32+
{
33+
string str = Value.ToString();
34+
return FromEnd ? "^" + str : str;
35+
}
36+
37+
public static implicit operator Index(int value)
38+
=> new Index(value, fromEnd: false);
39+
}
40+
}

‎src/Common/src/CoreLib/System/Range.cs‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
// Licensed to the .NET Foundation under one or more agreements.
2+
// The .NET Foundation licenses this file to you under the MIT license.
3+
// See the LICENSE file in the project root for more information.
4+
5+
namespace System
6+
{
7+
public readonly struct Range : IEquatable<Range>
8+
{
9+
public Index Start { get; }
10+
public Index End { get; }
11+
12+
private Range(Index start, Index end)
13+
{
14+
Start = start;
15+
End = end;
16+
}
17+
18+
public override bool Equals(object value)
19+
{
20+
if (value is Range)
21+
{
22+
Range r = (Range)value;
23+
return r.Start.Equals(Start) && r.End.Equals(End);
24+
}
25+
26+
return false;
27+
}
28+
29+
public bool Equals (Range other) => other.Start.Equals(Start) && other.End.Equals(End);
30+
31+
public override int GetHashCode()
32+
{
33+
return HashCode.Combine(Start.GetHashCode(), End.GetHashCode());
34+
}
35+
36+
public override string ToString()
37+
{
38+
return Start + ".." + End;
39+
}
40+
41+
public static Range Create(Index start, Index end) => new Range(start, end);
42+
public static Range FromStart(Index start) => new Range(start, new Index(0, fromEnd: true));
43+
public static Range ToEnd(Index end) => new Range(new Index(0, fromEnd: false), end);
44+
public static Range All() => new Range(new Index(0, fromEnd: false), new Index(0, fromEnd: true));
45+
}
46+
}

‎src/Common/src/CoreLib/System/ReadOnlySpan.Fast.cs‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,24 @@ public ref readonly T this[int index]
154154
#endif
155155
}
156156

157+
public ref readonly T this[Index index]
158+
{
159+
get
160+
{
161+
return ref this [index.FromEnd ? _length - index.Value : index.Value];
162+
}
163+
}
164+
165+
public ReadOnlySpan<T> this[Range range]
166+
{
167+
get
168+
{
169+
int start = range.Start.FromEnd ? _length - range.Start.Value : range.Start.Value;
170+
int end = range.End.FromEnd ? _length - range.End.Value : range.End.Value;
171+
return Slice(start, end - start);
172+
}
173+
}
174+
157175
/// <summary>
158176
/// Returns a reference to the 0th element of the Span. If the Span is empty, returns null reference.
159177
/// It can be used for pinning and is required to support the use of span within a fixed statement.

‎src/Common/src/CoreLib/System/Span.Fast.cs‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,24 @@ public ref T this[int index]
159159
#endif
160160
}
161161

162+
public ref T this[Index index]
163+
{
164+
get
165+
{
166+
return ref this [index.FromEnd ? _length - index.Value : index.Value];
167+
}
168+
}
169+
170+
public Span<T> this[Range range]
171+
{
172+
get
173+
{
174+
int start = range.Start.FromEnd ? _length - range.Start.Value : range.Start.Value;
175+
int end = range.End.FromEnd ? _length - range.End.Value : range.End.Value;
176+
return Slice(start, end - start);
177+
}
178+
}
179+
162180
/// <summary>
163181
/// Returns a reference to the 0th element of the Span. If the Span is empty, returns null reference.
164182
/// It can be used for pinning and is required to support the use of span within a fixed statement.

0 commit comments

Comments
 (0)