starred/jellyfin-jellyfin-1
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2025-12-18 15:03:06 +03:00
Code Issues Packages Projects Releases Wiki Activity

Add tests for ProbeResultNormalizer.GetFrameRate

Browse Source
This commit is contained in:
Bond_009
2021-12-12 02:22:30 +01:00
parent 923720c988
commit f8fcbc88fc
2 changed files with 31 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
MediaBrowser.MediaEncoding/Probing/ProbeResultNormalizer.cs
View File

@@ -1027,27 +1027,32 @@ namespace MediaBrowser.MediaEncoding.Probing
/// </summary>
/// <param name="value">The value.</param>
/// <returns>System.Nullable{System.Single}.</returns>
private float? GetFrameRate(string value)
internal static float? GetFrameRate(ReadOnlySpan<char> value)
{
if (string.IsNullOrEmpty(value))
if (value.IsEmpty)
{
return null;
}
var parts = value.Split('/');
float result;
if (parts.Length == 2)
int index = value.IndexOf('/');
if (index == -1)
{
result = float.Parse(parts[0], CultureInfo.InvariantCulture) / float.Parse(parts[1], CultureInfo.InvariantCulture);
}
else
{
result = float.Parse(parts[0], CultureInfo.InvariantCulture);
// REVIEW: is this branch actually required? (i.e. does ffprobe ever output something other than a fraction?)
if (float.TryParse(value, NumberStyles.AllowThousands | NumberStyles.Float, CultureInfo.InvariantCulture, out var result))
{
return result;
}
return null;
}
return float.IsNaN(result) ? null : result;
if (!float.TryParse(value[..index], NumberStyles.Integer, CultureInfo.InvariantCulture, out var dividend)
|| !float.TryParse(value[(index + 1)..], NumberStyles.Integer, CultureInfo.InvariantCulture, out var divisor))
{
return null;
}
return divisor == 0f ? 0f : dividend / divisor;
}
private void SetAudioRuntimeTicks(InternalMediaInfoResult result, MediaInfo data)