lgladsden/jellyfin-server
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Address comments

Browse Source
This commit is contained in:
MrTimscampi 2021-08-02 03:48:45 +02:00
parent 8594ee7a22
commit d82c2e4237
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -28,7 +28,9 @@ namespace MediaBrowser.MediaEncoding.Probing
private readonly char[] _nameDelimiters = { '/', '|', ';', '\\' }; private readonly char[] _nameDelimiters = { '/', '|', ';', '\\' };
private readonly CultureInfo _usCulture = new CultureInfo("en-US"); private readonly Regex _performerPattern = new (@"(?<name>.*) \((?<instrument>.*)\)");
private readonly CultureInfo _usCulture = new ("en-US");
private readonly ILogger _logger; private readonly ILogger _logger;
private readonly ILocalizationManager _localization; private readonly ILocalizationManager _localization;
@ -1116,8 +1118,7 @@ namespace MediaBrowser.MediaEncoding.Probing
{ {
foreach (var person in Split(performer, false)) foreach (var person in Split(performer, false))
{ {
Regex pattern = new Regex(@"(?<name>.*) \((?<instrument>.*)\)"); Match match = _performerPattern.Match(person);
Match match = pattern.Match(person);
// If the performer doesn't have any instrument/role associated, it won't match. In that case, chances are it's simply a band name, so we skip it. // If the performer doesn't have any instrument/role associated, it won't match. In that case, chances are it's simply a band name, so we skip it.
if (match.Success) if (match.Success)