github-mirrors/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2025-04-24 05:57:20 -04:00
Code Issues Projects Releases Packages Wiki Activity

remove unnecessary ToList in DlnaManager

Browse source
This commit is contained in:
ignacio laborde 2022-01-03 15:13:55 -03:00
parent 53447976f0
commit 8952819494
1 changed files with 4 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

16
Emby.Dlna/DlnaManager.cs
View file

@ -83,8 +83,7 @@ namespace Emby.Dlna
{
lock (_profiles)
{
var list = _profiles.Values.ToList();
return list
return _profiles.Values
.OrderBy(i => i.Item1.Info.Type == DeviceProfileType.User ? 0 : 1)
.ThenBy(i => i.Item1.Info.Name)
.Select(i => i.Item2)
@ -226,11 +225,8 @@ namespace Emby.Dlna
{
try
{
var xmlFies = _fileSystem.GetFilePaths(path)
return _fileSystem.GetFilePaths(path)
.Where(i => string.Equals(Path.GetExtension(i), ".xml", StringComparison.OrdinalIgnoreCase))
.ToList();
return xmlFies
.Select(i => ParseProfileFile(i, type))
.Where(i => i != null)
.ToList()!; // We just filtered out all the nulls
@ -252,11 +248,8 @@ namespace Emby.Dlna
try
{
DeviceProfile profile;
var tempProfile = (DeviceProfile)_xmlSerializer.DeserializeFromFile(typeof(DeviceProfile), path);
profile = ReserializeProfile(tempProfile);
var profile = ReserializeProfile(tempProfile);
profile.Id = path.ToLowerInvariant().GetMD5().ToString("N", CultureInfo.InvariantCulture);
@ -295,8 +288,7 @@ namespace Emby.Dlna
{
lock (_profiles)
{
var list = _profiles.Values.ToList();
return list
return _profiles.Values
.Select(i => i.Item1)
.OrderBy(i => i.Info.Type == DeviceProfileType.User ? 0 : 1)
.ThenBy(i => i.Info.Name);