From 70cde89480e43bb1369c1063c7b19f757784f269 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Sun, 16 Oct 2022 14:41:45 -0400
Subject: tweak naming in new code

---
 .../AssetPathUtilities/AssetNamePartEnumerator.cs  | 66 +++++++++++++++++++++
 .../AssetPathUtilities/AssetPartYielder.cs         | 67 ----------------------
 2 files changed, 66 insertions(+), 67 deletions(-)
 create mode 100644 src/SMAPI/Utilities/AssetPathUtilities/AssetNamePartEnumerator.cs
 delete mode 100644 src/SMAPI/Utilities/AssetPathUtilities/AssetPartYielder.cs

(limited to 'src/SMAPI/Utilities')

diff --git a/src/SMAPI/Utilities/AssetPathUtilities/AssetNamePartEnumerator.cs b/src/SMAPI/Utilities/AssetPathUtilities/AssetNamePartEnumerator.cs
new file mode 100644
index 00000000..0840617a
--- /dev/null
+++ b/src/SMAPI/Utilities/AssetPathUtilities/AssetNamePartEnumerator.cs
@@ -0,0 +1,66 @@
+using System;
+using ToolkitPathUtilities = StardewModdingAPI.Toolkit.Utilities.PathUtilities;
+
+namespace StardewModdingAPI.Utilities.AssetPathUtilities;
+
+/// <summary>
+/// A helper class that yields out each bit of an asset path
+/// </summary>
+internal ref struct AssetNamePartEnumerator
+{
+    private ReadOnlySpan<char> RemainderImpl;
+
+    /// <summary>
+    /// Construct an instance.
+    /// </summary>
+    /// <param name="assetName">The asset name.</param>
+    internal AssetNamePartEnumerator(ReadOnlySpan<char> assetName)
+    {
+        this.RemainderImpl = AssetNamePartEnumerator.TrimLeadingPathSeparators(assetName);
+    }
+
+    /// <summary>
+    /// The remainder of the assetName (that hasn't been yielded out yet.)
+    /// </summary>
+    internal ReadOnlySpan<char> Remainder => this.RemainderImpl;
+
+    /// <summary>
+    /// The current segment.
+    /// </summary>
+    public ReadOnlySpan<char> Current { get; private set; } = default;
+
+    // this is just so it can be used in a foreach loop.
+    public AssetNamePartEnumerator GetEnumerator() => this;
+
+    /// <summary>
+    /// Moves the enumerator to the next element.
+    /// </summary>
+    /// <returns>True if there is a new</returns>
+    public bool MoveNext()
+    {
+        if (this.RemainderImpl.Length == 0)
+        {
+            return false;
+        }
+
+        int index = this.RemainderImpl.IndexOfAny(ToolkitPathUtilities.PossiblePathSeparators);
+
+        // no more separator characters found, I'm done.
+        if (index < 0)
+        {
+            this.Current = this.RemainderImpl;
+            this.RemainderImpl = ReadOnlySpan<char>.Empty;
+            return true;
+        }
+
+        // Yield the next separate character bit
+        this.Current = this.RemainderImpl[..index];
+        this.RemainderImpl = AssetNamePartEnumerator.TrimLeadingPathSeparators(this.RemainderImpl[(index + 1)..]);
+        return true;
+    }
+
+    private static ReadOnlySpan<char> TrimLeadingPathSeparators(ReadOnlySpan<char> span)
+    {
+        return span.TrimStart(new ReadOnlySpan<char>(ToolkitPathUtilities.PossiblePathSeparators));
+    }
+}
diff --git a/src/SMAPI/Utilities/AssetPathUtilities/AssetPartYielder.cs b/src/SMAPI/Utilities/AssetPathUtilities/AssetPartYielder.cs
deleted file mode 100644
index a55a0ab4..00000000
--- a/src/SMAPI/Utilities/AssetPathUtilities/AssetPartYielder.cs
+++ /dev/null
@@ -1,67 +0,0 @@
-using System;
-
-using ToolkitPathUtilities = StardewModdingAPI.Toolkit.Utilities.PathUtilities;
-
-namespace StardewModdingAPI.Utilities.AssetPathUtilities;
-
-/// <summary>
-/// A helper class that yields out each bit of an asset path
-/// </summary>
-internal ref struct AssetPartYielder
-{
-    private ReadOnlySpan<char> remainder;
-
-    /// <summary>
-    /// Construct an instance.
-    /// </summary>
-    /// <param name="assetName">The asset name.</param>
-    internal AssetPartYielder(ReadOnlySpan<char> assetName)
-    {
-        this.remainder = AssetPartYielder.TrimLeadingPathSeperators(assetName);
-    }
-
-    /// <summary>
-    /// The remainder of the assetName (that hasn't been yielded out yet.)
-    /// </summary>
-    internal ReadOnlySpan<char> Remainder => this.remainder;
-
-    /// <summary>
-    /// The current segment.
-    /// </summary>
-    public ReadOnlySpan<char> Current { get; private set; } = default;
-
-    // this is just so it can be used in a foreach loop.
-    public AssetPartYielder GetEnumerator() => this;
-
-    /// <summary>
-    /// Moves the enumerator to the next element.
-    /// </summary>
-    /// <returns>True if there is a new</returns>
-    public bool MoveNext()
-    {
-        if (this.remainder.Length == 0)
-        {
-            return false;
-        }
-
-        int index = this.remainder.IndexOfAny(ToolkitPathUtilities.PossiblePathSeparators);
-
-        // no more seperator characters found, I'm done.
-        if (index < 0)
-        {
-            this.Current = this.remainder;
-            this.remainder = ReadOnlySpan<char>.Empty;
-            return true;
-        }
-
-        // Yield the next seperate character bit
-        this.Current = this.remainder[..index];
-        this.remainder = AssetPartYielder.TrimLeadingPathSeperators(this.remainder[(index + 1)..]);
-        return true;
-    }
-
-    private static ReadOnlySpan<char> TrimLeadingPathSeperators(ReadOnlySpan<char> span)
-    {
-        return span.TrimStart(new ReadOnlySpan<char>(ToolkitPathUtilities.PossiblePathSeparators));
-    }
-}
-- 
cgit