Use LastIndexOf in ZipArchiveEntry.GetFileName_Unix (dotnet#70701)

There's not particularly good reason to open-code the loop here.
Nisden · Jun 16, 2022 · 9ab00ef · 9ab00ef
1 parent f25caab
commit 9ab00ef
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/libraries/System.IO.Compression/src/System/IO/Compression/ZipArchiveEntry.cs b/src/libraries/System.IO.Compression/src/System/IO/Compression/ZipArchiveEntry.cs
@@ -1104,11 +1104,10 @@ private static string GetFileName_Windows(string path)
         /// </summary>
         private static string GetFileName_Unix(string path)
         {
-            int length = path.Length;
-            for (int i = length; --i >= 0;)
-                if (path[i] == '/')
-                    return path.Substring(i + 1);
-            return path;
+            int i = path.LastIndexOf('/');
+            return i >= 0 ?
+                path.Substring(i + 1) :
+                path;
         }
 
         private sealed class DirectToArchiveWriterStream : Stream