Update path normalization for Windows-like behavior on Linux

Replaced LinuxDirectorySeparatorNormalizeOptions with PGFileSystemDirectorySeparatorNormalizeOptions to ensure consistent Windows-like path handling on Linux. Updated class documentation to clarify behavior. Refactored IsDirectorySeparator to use defined separator constants for improved clarity.

Author: AnakinRaW

src/PetroglyphTools/PG.StarWarsGame.Engine.FileSystem/IO/PetroglyphFileSystem.Normalize.cs

@@ -12,16 +12,7 @@ internal void NormalizePath(ref ValueStringBuilder stringBuilder)
     }
 
     private static void NormalizePath(Span<char> path)
-    {
-        PathNormalizer.Normalize(path, path, LinuxDirectorySeparatorNormalizeOptions);
-        
-        //if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
-        //    return;
-        //for (var i = 0; i < path.Length; i++)
-        //{
-        //    var c = path[i];
-        //    if (IsDirectorySeparator(c))
-        //        path[i] = '/';
-        //}
+    { 
+        PathNormalizer.Normalize(path, path, PGFileSystemDirectorySeparatorNormalizeOptions);
     } 
 }

src/PetroglyphTools/PG.StarWarsGame.Engine.FileSystem/IO/PetroglyphFileSystem.PathEqual.cs

@@ -13,8 +13,8 @@ public bool PathsAreEqual(string pathA, string pathB)
         if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
             return _underlyingFileSystem.Path.AreEqual(pathA, pathB);
 
-        var normalizedA = PathNormalizer.Normalize(pathA, LinuxDirectorySeparatorNormalizeOptions);
-        var normalizedB = PathNormalizer.Normalize(pathB, LinuxDirectorySeparatorNormalizeOptions);
+        var normalizedA = PathNormalizer.Normalize(pathA, PGFileSystemDirectorySeparatorNormalizeOptions);
+        var normalizedB = PathNormalizer.Normalize(pathB, PGFileSystemDirectorySeparatorNormalizeOptions);
 
         var fullA = _underlyingFileSystem.Path.GetFullPath(normalizedA);
         var fullB = _underlyingFileSystem.Path.GetFullPath(normalizedB);

src/PetroglyphTools/PG.StarWarsGame.Engine.FileSystem/IO/PetroglyphFileSystem.cs

@@ -11,14 +11,16 @@ namespace PG.StarWarsGame.Engine.IO;
 /// A file system abstraction for the Petroglyph game engine.
 /// </summary>
 /// <remarks>
-/// The file system enforces Windows behavior for all its methods independent of the current operating system.
+/// This file system abstraction simulates Windows-like behavior for all its public methods on Linux,
+/// such as correct handling of paths containing backslashes ("\"). On Windows itself the behavior is unchanged.
 /// </remarks>
 public sealed partial class PetroglyphFileSystem
 {
     private const char DirectorySeparatorChar = '/';
     private const char AltDirectorySeparatorChar = '\\';
 
-    private static readonly PathNormalizeOptions LinuxDirectorySeparatorNormalizeOptions = new()
+    // ReSharper disable once InconsistentNaming
+    private static readonly PathNormalizeOptions PGFileSystemDirectorySeparatorNormalizeOptions = new()
     {
         TreatBackslashAsSeparator = true, // Ensure that we treat backslashes as separators on Linux
         UnifyDirectorySeparators = true,
@@ -51,7 +53,7 @@ internal FileSystemStream OpenRead(string filePath)
 
     private static bool IsPathRooted(ReadOnlySpan<char> path)
     {
-        // The original implementation, of course, also checks for drive signatures (e.g, c:, X:).
+        // The original implementation, obviously, also checks for drive signatures (e.g, c:, X:).
         // We don't expect such paths ever when running in linux mode, so we simply ignore these
         var length = path.Length;
         return length >= 1 && IsDirectorySeparator(path[0]);
@@ -89,6 +91,6 @@ private static bool IsEffectivelyEmpty(ReadOnlySpan<char> path)
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     private static bool IsDirectorySeparator(char c)
     {
-        return c is '\\' or '/';
+        return c is DirectorySeparatorChar or AltDirectorySeparatorChar;
     }
 }