[PR #15811] Optimizations using sets #14250

New issue

Open

opened 2025-12-22 10:24:17 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 10:24:17 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/15811
Author: @MarkCiliaVincenti
Created: 12/16/2025
Status: 🔄 Open

Base: master ← Head: setoptimizations

📝 Commits (10+)

c4fb778 Optimizations using sets
eb9c75f remove empty line
60edf8e Some more sets
54ee024 readd BOM
ce0e42f Revert some smaller sets as arrays would be more performant.
c9b63a6 Revert TvParserHelpers
6ad33a5 fix
a2606ef revert
dbf757d Reverts due to small collections
e977c78 revert empty line

📊 Changes

20 files changed (+45 additions, -60 deletions)

View changed files

📝 Emby.Naming/Audio/AudioFileParser.cs (+1 -4)
📝 Emby.Naming/AudioBook/AudioBookResolver.cs (+1 -1)
📝 Emby.Naming/Common/NamingOptions.cs (+10 -9)
📝 Emby.Naming/ExternalFiles/ExternalPathParser.cs (+2 -2)
📝 Emby.Naming/TV/EpisodeResolver.cs (+1 -1)
📝 Emby.Naming/Video/FileStack.cs (+3 -3)
📝 Emby.Naming/Video/StackResolver.cs (+3 -3)
📝 Emby.Naming/Video/VideoResolver.cs (+3 -4)
📝 Emby.Server.Implementations/Chapters/ChapterManager.cs (+1 -1)
📝 Emby.Server.Implementations/Collections/CollectionManager.cs (+1 -1)
📝 Emby.Server.Implementations/IO/FileRefresher.cs (+2 -7)
📝 Emby.Server.Implementations/Library/Resolvers/Books/BookResolver.cs (+2 -3)
📝 Emby.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs (+3 -3)
📝 Emby.Server.Implementations/Library/Resolvers/PhotoResolver.cs (+2 -3)
📝 Emby.Server.Implementations/Library/SearchEngine.cs (+4 -7)
📝 MediaBrowser.Providers/Playlists/PlaylistItemsProvider.cs (+1 -1)
📝 src/Jellyfin.Drawing/ImageProcessor.cs (+3 -4)
📝 src/Jellyfin.Extensions/EnumerableExtensions.cs (+0 -1)
📝 src/Jellyfin.MediaEncoding.Hls/Extractors/FfProbeKeyframeExtractor.cs (+1 -1)
📝 tests/Jellyfin.Naming.Tests/Video/VideoListResolverTests.cs (+1 -1)

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/15811 **Author:** [@MarkCiliaVincenti](https://github.com/MarkCiliaVincenti) **Created:** 12/16/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `setoptimizations` --- ### 📝 Commits (10+) - [`c4fb778`](https://github.com/jellyfin/jellyfin/commit/c4fb77848e8c53277b6602e68c509e4b76236b80) Optimizations using sets - [`eb9c75f`](https://github.com/jellyfin/jellyfin/commit/eb9c75faceaad8df35fec487412179951a51cb5d) remove empty line - [`60edf8e`](https://github.com/jellyfin/jellyfin/commit/60edf8e599958af6cf2b88af989c1bc2cb591521) Some more sets - [`54ee024`](https://github.com/jellyfin/jellyfin/commit/54ee024985db05da117c128536ea594f955a3d6b) readd BOM - [`ce0e42f`](https://github.com/jellyfin/jellyfin/commit/ce0e42fe6a48f5feaf20bc76f15f09b0d2e64a44) Revert some smaller sets as arrays would be more performant. - [`c9b63a6`](https://github.com/jellyfin/jellyfin/commit/c9b63a60b0d91b2955202936e7c0460723f2011d) Revert TvParserHelpers - [`6ad33a5`](https://github.com/jellyfin/jellyfin/commit/6ad33a50f595e49308140e3cd93159833fe64c72) fix - [`a2606ef`](https://github.com/jellyfin/jellyfin/commit/a2606ef5c8a157013dfc93637b03433eea3cbc52) revert - [`dbf757d`](https://github.com/jellyfin/jellyfin/commit/dbf757d874fe60037ad7f42365d5fc6b1d1bbf20) Reverts due to small collections - [`e977c78`](https://github.com/jellyfin/jellyfin/commit/e977c783f4e38cf1d0e967595a617e9021135243) revert empty line ### 📊 Changes **20 files changed** (+45 additions, -60 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Naming/Audio/AudioFileParser.cs` (+1 -4) 📝 `Emby.Naming/AudioBook/AudioBookResolver.cs` (+1 -1) 📝 `Emby.Naming/Common/NamingOptions.cs` (+10 -9) 📝 `Emby.Naming/ExternalFiles/ExternalPathParser.cs` (+2 -2) 📝 `Emby.Naming/TV/EpisodeResolver.cs` (+1 -1) 📝 `Emby.Naming/Video/FileStack.cs` (+3 -3) 📝 `Emby.Naming/Video/StackResolver.cs` (+3 -3) 📝 `Emby.Naming/Video/VideoResolver.cs` (+3 -4) 📝 `Emby.Server.Implementations/Chapters/ChapterManager.cs` (+1 -1) 📝 `Emby.Server.Implementations/Collections/CollectionManager.cs` (+1 -1) 📝 `Emby.Server.Implementations/IO/FileRefresher.cs` (+2 -7) 📝 `Emby.Server.Implementations/Library/Resolvers/Books/BookResolver.cs` (+2 -3) 📝 `Emby.Server.Implementations/Library/Resolvers/Movies/MovieResolver.cs` (+3 -3) 📝 `Emby.Server.Implementations/Library/Resolvers/PhotoResolver.cs` (+2 -3) 📝 `Emby.Server.Implementations/Library/SearchEngine.cs` (+4 -7) 📝 `MediaBrowser.Providers/Playlists/PlaylistItemsProvider.cs` (+1 -1) 📝 `src/Jellyfin.Drawing/ImageProcessor.cs` (+3 -4) 📝 `src/Jellyfin.Extensions/EnumerableExtensions.cs` (+0 -1) 📝 `src/Jellyfin.MediaEncoding.Hls/Extractors/FfProbeKeyframeExtractor.cs` (+1 -1) 📝 `tests/Jellyfin.Naming.Tests/Video/VideoListResolverTests.cs` (+1 -1) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo added the

pull-request

label 2025-12-22 10:24:17 +01:00

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#14250

No description provided.

Rows
Columns