yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use span in StripFileExtension to speed up and use less memory (#13101)

Browse Source
This commit is contained in:
patrickdemooij9
2022-11-28 22:12:30 +01:00
committed by GitHub
parent 33668a5d95
commit 475cea40a1
2 changed files with 85 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/Umbraco.Core/Extensions/StringExtensions.cs
View File

@@ -83,18 +83,19 @@ public static class StringExtensions
return fileName;
}
var lastIndex = fileName.LastIndexOf('.');
var spanFileName = fileName.AsSpan();
var lastIndex = spanFileName.LastIndexOf('.');
if (lastIndex > 0)
{
var ext = fileName.Substring(lastIndex);
var ext = spanFileName[lastIndex..];
// file extensions cannot contain whitespace
if (ext.Contains(" "))
if (ext.Contains(' '))
{
return fileName;
}
return string.Format("{0}", fileName.Substring(0, fileName.IndexOf(ext, StringComparison.Ordinal)));
return new string(spanFileName[..lastIndex]);
}
return fileName;