Use Path.Combine instead of building path as a string

davschne-unity · davschne-unity · commit 858011a973f2 · 2023-09-21T13:30:01.000-07:00
diff --git a/UnityDataTool/Program.cs b/UnityDataTool/Program.cs
@@ -177,7 +177,7 @@ static int HandleExtractArchive(FileInfo filename, DirectoryInfo outputFolder)
             foreach (var node in archive.Nodes)
             {
                 Console.WriteLine($"Extracting {node.Path}...");
-                CopyFile("/" + node.Path, $"{outputFolder}/{node.Path}");
+                CopyFile("/" + node.Path, Path.Combine(outputFolder.FullName, node.Path));
             }
         }
         catch (NotSupportedException)

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,7 @@ static int HandleExtractArchive(FileInfo filename, DirectoryInfo outputFolder)`
`177`	`177`	`foreach (var node in archive.Nodes)`
`178`	`178`	`{`
`179`	`179`	`Console.WriteLine($"Extracting {node.Path}...");`
`180`		`- CopyFile("/" + node.Path, $"{outputFolder}/{node.Path}");`
	`180`	`+ CopyFile("/" + node.Path, Path.Combine(outputFolder.FullName, node.Path));`
`181`	`181`	`}`
`182`	`182`	`}`
`183`	`183`	`catch (NotSupportedException)`