[PR #5919] [CLOSED] Code optimizations #10730

New Issue

OVERLORD · 2026-02-07T06:25:44+03:00

OVERLORD commented

2026-02-07 06:25:44 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5919
Author: @BaronGreenback
Created: 4/26/2021
Status: ❌ Closed

Base: master ← Head: AppHost_Changes

📝 Commits (6)

1acd3d8 Change PublishedServerUrl type
04559bc removed duplicate property in interface
07e9034 Merge branch 'master' into AppHost_Changes
4afa3e0 removed code
40058fd Update ApplicationHost.cs
6bfbe30 change type to enable code removal

📊 Changes

6 files changed (+16 additions, -67 deletions)

View changed files

📝 Emby.Server.Implementations/ApplicationHost.cs (+11 -36)
📝 Emby.Server.Implementations/LiveTv/LiveTvMediaSourceProvider.cs (+1 -1)
📝 Jellyfin.Networking/Manager/NetworkManager.cs (+0 -6)
📝 Jellyfin.Server/StartupOptions.cs (+2 -2)
📝 MediaBrowser.Common/Net/INetworkManager.cs (+0 -10)
📝 MediaBrowser.Controller/IServerApplicationHost.cs (+2 -12)

📄 Description

LiveTV

changed function call to the https://github.com/jellyfin/jellyfin/pull/5905 to ensure continuity.

Optimizations

trimmed end / off PublishedServerUrl at startup, instead of each iteration.
removed unused parameters.
removed unused method.

_{🔄 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/5919 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 4/26/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `AppHost_Changes` --- ### 📝 Commits (6) - [`1acd3d8`](https://github.com/jellyfin/jellyfin/commit/1acd3d80a40e6c7822f39bf4e7344bb0f37818c9) Change PublishedServerUrl type - [`04559bc`](https://github.com/jellyfin/jellyfin/commit/04559bcc4610bc3906d03db068f55aab70a30f34) removed duplicate property in interface - [`07e9034`](https://github.com/jellyfin/jellyfin/commit/07e9034f68153fd576599695b7d4afcd7a5cfb35) Merge branch 'master' into AppHost_Changes - [`4afa3e0`](https://github.com/jellyfin/jellyfin/commit/4afa3e088898eb201998b33699bd9fdb7b99d3c7) removed code - [`40058fd`](https://github.com/jellyfin/jellyfin/commit/40058fd553dbde67e496bd124c3a46aa9fce6f1d) Update ApplicationHost.cs - [`6bfbe30`](https://github.com/jellyfin/jellyfin/commit/6bfbe30ba4a901681730e7ef86cc6c794b98eedc) change type to enable code removal ### 📊 Changes **6 files changed** (+16 additions, -67 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+11 -36) 📝 `Emby.Server.Implementations/LiveTv/LiveTvMediaSourceProvider.cs` (+1 -1) 📝 `Jellyfin.Networking/Manager/NetworkManager.cs` (+0 -6) 📝 `Jellyfin.Server/StartupOptions.cs` (+2 -2) 📝 `MediaBrowser.Common/Net/INetworkManager.cs` (+0 -10) 📝 `MediaBrowser.Controller/IServerApplicationHost.cs` (+2 -12) </details> ### 📄 Description LiveTV - changed function call to the https://github.com/jellyfin/jellyfin/pull/5905 to ensure continuity. Optimizations - trimmed end / off PublishedServerUrl at startup, instead of each iteration. - removed unused parameters. - removed unused method. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2026-02-07 06:25:44 +03:00

OVERLORD closed this issue

2026-02-07 06:25:46 +03:00

OVERLORD referenced this issue

2026-02-07 07:07:45 +03:00

[PR #12293] [CLOSED] Add env var to always use legacy audio prober #13136

OVERLORD referenced this issue

2026-02-07 07:08:26 +03:00

[PR #12385] [MERGED] Add non-standard multi-value audio tag support #13175

Sign in to join this conversation.