Remove unused methods

Author: rameel

Ramstack.Globbing.Tests/Traversal/PathHelperTests.cs

@@ -154,45 +154,6 @@ public void GetPartialPattern_Generated()
         }
     }
 
-    [Test]
-    public void SearchPathSeparator()
-    {
-        for (var n = 0; n < 5000; n++)
-        {
-            var p0 = new string('a', n);
-
-            var p1 = p0 + "\\";
-            var index1 = PathHelper.SearchPathSeparator(p1, MatchFlags.Windows);
-            var index2 = PathHelper.SearchPathSeparator(p1, MatchFlags.Unix);
-
-            Assert.That(index1, Is.EqualTo(p1.IndexOf('\\')), $"length: {n}");
-            Assert.That(index2, Is.EqualTo(-1),               $"length: {n}");
-
-            var p2 = p0 + "/";
-            var index3 = PathHelper.SearchPathSeparator(p2, MatchFlags.Windows);
-            var index4 = PathHelper.SearchPathSeparator(p2, MatchFlags.Unix);
-
-            Assert.That(index3, Is.EqualTo(p2.IndexOf('/')), $"length: {n}");
-            Assert.That(index4, Is.EqualTo(index3),          $"length: {n}");
-        }
-    }
-
-    [Test]
-    public void SearchPathSeparator_Nothing()
-    {
-        var source = new string('a', 5000);
-
-        for (var n = 0; n < 5000; n++)
-        {
-            var p = source.AsSpan(0, n);
-            var index1 = PathHelper.SearchPathSeparator(p, MatchFlags.Windows);
-            var index2 = PathHelper.SearchPathSeparator(p, MatchFlags.Unix);
-
-            Assert.That(index1, Is.EqualTo(-1));
-            Assert.That(index2, Is.EqualTo(-1));
-        }
-    }
-
     private static IEnumerable<(string path, char slash, MatchFlags flags)> GeneratePaths()
     {
         var flags = new[]

Ramstack.Globbing/Traversal/PathHelper.cs

@@ -15,135 +15,6 @@ namespace Ramstack.Globbing.Traversal;
 [SuppressMessage("ReSharper", "RedundantCast")]
 internal static class PathHelper
 {
-    /// <summary>
-    /// Searches for a path separator within a span of characters.
-    /// </summary>
-    /// <param name="path">The span of characters representing the path.</param>
-    /// <param name="flags">The flags indicating the type of path separators to match.</param>
-    /// <returns>
-    /// The index of the path separator if found; otherwise, <c>-1</c>.
-    /// </returns>
-    public static int SearchPathSeparator(scoped ReadOnlySpan<char> path, MatchFlags flags)
-    {
-        var count = (nint)(uint)path.Length;
-        ref var p = ref MemoryMarshal.GetReference(path);
-        return SearchPathSeparator(ref p, count, flags);
-    }
-
-    /// <summary>
-    /// Searches for a path separator within a range of characters.
-    /// </summary>
-    /// <param name="p">A reference to the first character of the range.</param>
-    /// <param name="count">The number of characters to search through.</param>
-    /// <param name="flags">The flags indicating the type of path separators to match.</param>
-    /// <returns>
-    /// The index of the path separator if found; otherwise, <c>-1</c>.
-    /// </returns>
-    public static int SearchPathSeparator(scoped ref char p, nint count, MatchFlags flags)
-    {
-        var index = (nint)0;
-
-        if (!Sse41.IsSupported || count < Vector128<ushort>.Count)
-        {
-            for (; index < count; index++)
-                if (Unsafe.Add(ref p, index) == '/'
-                    || (Unsafe.Add(ref p, index) == '\\' && flags == MatchFlags.Windows))
-                    return (int)index;
-
-            return -1;
-        }
-
-        if (!Avx2.IsSupported || count < Vector256<ushort>.Count)
-        {
-            var slash = Vector128.Create((ushort)'/');
-            var backslash = Vector128.Create((ushort)'\\');
-            var allowEscaping = CreateAllowEscaping128Bitmask(flags);
-
-            var tail = count - Vector128<ushort>.Count;
-
-            Vector128<ushort> chunk;
-            Vector128<ushort> comparison;
-
-            do
-            {
-                chunk = LoadVector128(ref p, index);
-                comparison = Sse2.Or(
-                    Sse2.CompareEqual(chunk, slash),
-                    Sse2.AndNot(
-                        allowEscaping,
-                        Sse2.CompareEqual(chunk, backslash)));
-
-                if (!Sse41.TestZ(comparison, comparison))
-                {
-                    var position = BitOperations.TrailingZeroCount(Sse2.MoveMask(comparison.AsByte()));
-                    return (int)index + (position >>> 1);
-                }
-
-                index += Vector128<ushort>.Count;
-            }
-            while (index + Vector128<ushort>.Count <= count);
-
-            chunk = LoadVector128(ref p, tail);
-            comparison = Sse2.Or(
-                Sse2.CompareEqual(chunk, slash),
-                Sse2.AndNot(
-                    allowEscaping,
-                    Sse2.CompareEqual(chunk, backslash)));
-
-            if (!Sse41.TestZ(comparison, comparison))
-            {
-                var position = BitOperations.TrailingZeroCount(Sse2.MoveMask(comparison.AsByte()));
-                return (int)tail + (position >>> 1);
-            }
-
-            return -1;
-        }
-        else
-        {
-            var slash = Vector256.Create((ushort)'/');
-            var backslash = Vector256.Create((ushort)'\\');
-            var allowEscaping = CreateAllowEscaping256Bitmask(flags);
-            var tail = count - Vector256<ushort>.Count;
-
-            Vector256<ushort> chunk;
-            Vector256<ushort> comparison;
-
-            do
-            {
-                chunk = LoadVector256(ref p, index);
-                comparison = Avx2.Or(
-                    Avx2.CompareEqual(chunk, slash),
-                    Avx2.AndNot(
-                        allowEscaping,
-                        Avx2.CompareEqual(chunk, backslash)));
-
-                if (!Avx.TestZ(comparison, comparison))
-                {
-                    var position = BitOperations.TrailingZeroCount(Avx2.MoveMask(comparison.AsByte()));
-                    return (int)index + (position >>> 1);
-                }
-
-                index += Vector256<ushort>.Count;
-            }
-            while (index + Vector256<ushort>.Count <= count);
-
-            chunk = LoadVector256(ref p, tail);
-            comparison = Avx2.Or(
-                Avx2.CompareEqual(chunk, slash),
-                Avx2.AndNot(
-                    allowEscaping,
-                    Avx2.CompareEqual(chunk, backslash)));
-
-            if (!Avx.TestZ(comparison, comparison))
-            {
-                var position = BitOperations.TrailingZeroCount(Avx2.MoveMask(comparison.AsByte()));
-                return (int)tail + (position >>> 1);
-            }
-
-            return -1;
-        }
-    }
-
     /// <summary>
     /// Determines whether the specified path matches any of the specified patterns.
     /// </summary>