don't shorten file names unless necessary

losttech · Aug 25, 2023 · c7e645e · c7e645e
1 parent e6e3236
commit c7e645e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/NDetours.csproj b/src/NDetours.csproj
@@ -8,7 +8,7 @@
     <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
 
     <!-- Package stuff -->
-    <Version>0.0.10</Version>
+    <Version>0.0.11</Version>
     <PackageLicenseFile>LICENSE</PackageLicenseFile>
     <PackageReadmeFile>README.md</PackageReadmeFile>
     <Description>Managed package that simplifies injection of Detours-based DLLs</Description>

diff --git a/src/ProcessDetour.cs b/src/ProcessDetour.cs
@@ -51,8 +51,12 @@ public static unsafe Process Start(StartInfo startInfo) {
         try {
             for (int i = 0; i < startInfo.InjectDlls.Count; i++) {
                 string unicodePath = startInfo.InjectDlls[i];
-                string shortPath = UnicodePaths.GetShortPath(unicodePath);
-                dlls[i] = new(shortPath, Encoding.ASCII);
+                string compatiblePath =
+                    unicodePath.Any(c => c > 127)
+                        ? UnicodePaths.GetShortPath(unicodePath)
+                        : unicodePath;
+                Debug.WriteLine(compatiblePath);
+                dlls[i] = new(compatiblePath, Encoding.ASCII);
                 dllsMarked[i] = (byte*)dlls[i].RawPointer;
             }