Skip to content

Commit bc6f70b

Browse files
Sergio0694Sergio0694
committed
Minor code tweaks for clarity
1 parent f3e7e00 commit bc6f70b

File tree

1 file changed

+31
-31
lines changed

1 file changed

+31
-31
lines changed

‎Microsoft.Toolkit.HighPerformance/Helpers/Internals/RefEnumerableHelper.cs

Lines changed: 31 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,8 @@ public static void Clear<T>(ref T r0, nint length, nint step)
6666
/// <param name="sourceRef">The source reference to copy from.</param>
6767
/// <param name="destinationRef">The target reference to copy to.</param>
6868
/// <param name="length">The total number of items to copy.</param>
69-
/// <param name="step">The step between consecutive items in the memory area pointed to by <paramref name="sourceRef"/>.</param>
70-
public static void CopyTo<T>(ref T sourceRef, ref T destinationRef, nint length, nint step)
69+
/// <param name="sourceStep">The step between consecutive items in the memory area pointed to by <paramref name="sourceRef"/>.</param>
70+
public static void CopyTo<T>(ref T sourceRef, ref T destinationRef, nint length, nint sourceStep)
7171
{
7272
nint
7373
sourceOffset = 0,
@@ -76,28 +76,28 @@ public static void CopyTo<T>(ref T sourceRef, ref T destinationRef, nint length,
7676
while (length >= 8)
7777
{
7878
Unsafe.Add(ref destinationRef, destinationOffset + 0) = Unsafe.Add(ref sourceRef, sourceOffset);
79-
Unsafe.Add(ref destinationRef, destinationOffset + 1) = Unsafe.Add(ref sourceRef, sourceOffset += step);
80-
Unsafe.Add(ref destinationRef, destinationOffset + 2) = Unsafe.Add(ref sourceRef, sourceOffset += step);
81-
Unsafe.Add(ref destinationRef, destinationOffset + 3) = Unsafe.Add(ref sourceRef, sourceOffset += step);
82-
Unsafe.Add(ref destinationRef, destinationOffset + 4) = Unsafe.Add(ref sourceRef, sourceOffset += step);
83-
Unsafe.Add(ref destinationRef, destinationOffset + 5) = Unsafe.Add(ref sourceRef, sourceOffset += step);
84-
Unsafe.Add(ref destinationRef, destinationOffset + 6) = Unsafe.Add(ref sourceRef, sourceOffset += step);
85-
Unsafe.Add(ref destinationRef, destinationOffset + 7) = Unsafe.Add(ref sourceRef, sourceOffset += step);
79+
Unsafe.Add(ref destinationRef, destinationOffset + 1) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
80+
Unsafe.Add(ref destinationRef, destinationOffset + 2) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
81+
Unsafe.Add(ref destinationRef, destinationOffset + 3) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
82+
Unsafe.Add(ref destinationRef, destinationOffset + 4) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
83+
Unsafe.Add(ref destinationRef, destinationOffset + 5) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
84+
Unsafe.Add(ref destinationRef, destinationOffset + 6) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
85+
Unsafe.Add(ref destinationRef, destinationOffset + 7) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
8686

8787
length -= 8;
88-
sourceOffset += step;
88+
sourceOffset += sourceStep;
8989
destinationOffset += 8;
9090
}
9191

9292
if (length >= 4)
9393
{
9494
Unsafe.Add(ref destinationRef, destinationOffset + 0) = Unsafe.Add(ref sourceRef, sourceOffset);
95-
Unsafe.Add(ref destinationRef, destinationOffset + 1) = Unsafe.Add(ref sourceRef, sourceOffset += step);
96-
Unsafe.Add(ref destinationRef, destinationOffset + 2) = Unsafe.Add(ref sourceRef, sourceOffset += step);
97-
Unsafe.Add(ref destinationRef, destinationOffset + 3) = Unsafe.Add(ref sourceRef, sourceOffset += step);
95+
Unsafe.Add(ref destinationRef, destinationOffset + 1) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
96+
Unsafe.Add(ref destinationRef, destinationOffset + 2) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
97+
Unsafe.Add(ref destinationRef, destinationOffset + 3) = Unsafe.Add(ref sourceRef, sourceOffset += sourceStep);
9898

9999
length -= 4;
100-
sourceOffset += step;
100+
sourceOffset += sourceStep;
101101
destinationOffset += 4;
102102
}
103103

@@ -106,7 +106,7 @@ public static void CopyTo<T>(ref T sourceRef, ref T destinationRef, nint length,
106106
Unsafe.Add(ref destinationRef, destinationOffset) = Unsafe.Add(ref sourceRef, sourceOffset);
107107

108108
length -= 1;
109-
sourceOffset += step;
109+
sourceOffset += sourceStep;
110110
destinationOffset += 1;
111111
}
112112
}
@@ -166,14 +166,14 @@ public static void CopyTo<T>(ref T sourceRef, ref T destinationRef, nint length,
166166

167167
/// <summary>
168168
/// Copies a sequence of discontiguous items from one memory area to another. This mirrors
169-
/// <see cref="CopyTo{T}(ref T,ref T,nint,nint)"/>, but <paramref name="step"/> refers to <paramref name="destinationRef"/> instead.
169+
/// <see cref="CopyTo{T}(ref T,ref T,nint,nint)"/>, but <paramref name="sourceStep"/> refers to <paramref name="destinationRef"/> instead.
170170
/// </summary>
171171
/// <typeparam name="T">The type of items to copy.</typeparam>
172172
/// <param name="sourceRef">The source reference to copy from.</param>
173173
/// <param name="destinationRef">The target reference to copy to.</param>
174174
/// <param name="length">The total number of items to copy.</param>
175-
/// <param name="step">The step between consecutive items in the memory area pointed to by <paramref name="sourceRef"/>.</param>
176-
public static void CopyFrom<T>(ref T sourceRef, ref T destinationRef, nint length, nint step)
175+
/// <param name="sourceStep">The step between consecutive items in the memory area pointed to by <paramref name="sourceRef"/>.</param>
176+
public static void CopyFrom<T>(ref T sourceRef, ref T destinationRef, nint length, nint sourceStep)
177177
{
178178
nint
179179
sourceOffset = 0,
@@ -182,29 +182,29 @@ public static void CopyFrom<T>(ref T sourceRef, ref T destinationRef, nint lengt
182182
while (length >= 8)
183183
{
184184
Unsafe.Add(ref destinationRef, destinationOffset) = Unsafe.Add(ref sourceRef, sourceOffset);
185-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 1);
186-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 2);
187-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 3);
188-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 4);
189-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 5);
190-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 6);
191-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 7);
185+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 1);
186+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 2);
187+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 3);
188+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 4);
189+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 5);
190+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 6);
191+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 7);
192192

193193
length -= 8;
194194
sourceOffset += 8;
195-
destinationOffset += step;
195+
destinationOffset += sourceStep;
196196
}
197197

198198
if (length >= 4)
199199
{
200200
Unsafe.Add(ref destinationRef, destinationOffset) = Unsafe.Add(ref sourceRef, sourceOffset);
201-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 1);
202-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 2);
203-
Unsafe.Add(ref destinationRef, destinationOffset += step) = Unsafe.Add(ref sourceRef, sourceOffset + 3);
201+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 1);
202+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 2);
203+
Unsafe.Add(ref destinationRef, destinationOffset += sourceStep) = Unsafe.Add(ref sourceRef, sourceOffset + 3);
204204

205205
length -= 4;
206206
sourceOffset += 4;
207-
destinationOffset += step;
207+
destinationOffset += sourceStep;
208208
}
209209

210210
while (length > 0)
@@ -213,7 +213,7 @@ public static void CopyFrom<T>(ref T sourceRef, ref T destinationRef, nint lengt
213213

214214
length -= 1;
215215
sourceOffset += 1;
216-
destinationOffset += step;
216+
destinationOffset += sourceStep;
217217
}
218218
}
219219

0 commit comments

Comments
 (0)