[PR #4587] [CLOSED] Reduction in code nesting in networkManager. #10175

New Issue

OVERLORD · 2026-02-07T06:15:24+03:00

OVERLORD commented

2026-02-07 06:15:24 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4587
Author: @BaronGreenback
Created: 11/26/2020
Status: ❌ Closed

Base: master ← Head: NetworkManagerOptimization

📝 Commits (5)

dc1cec9 Merge pull request #98 from jellyfin/master
2dacaa5 Changes as recommended by ReSharper.
fcd0c9f Merge branch 'master' into NetworkManagerOptimization
f2290aa simplification
3e2972c Merge branch 'master' into NetworkManagerOptimization

📊 Changes

7 files changed (+347 additions, -362 deletions)

View changed files

📝 Jellyfin.Networking/Configuration/NetworkConfigurationExtensions.cs (+0 -1)
📝 Jellyfin.Networking/Manager/NetworkManager.cs (+195 -173)
📝 MediaBrowser.Common/Net/INetworkManager.cs (+0 -1)
📝 MediaBrowser.Common/Net/IPHost.cs (+58 -62)
📝 MediaBrowser.Common/Net/IPNetAddress.cs (+23 -23)
📝 MediaBrowser.Common/Net/IPObject.cs (+66 -64)
📝 MediaBrowser.Common/Net/NetworkExtensions.cs (+5 -38)

📄 Description

Implemented changes as recommended by ReSharper.

99.9% of the changes are code nesting reductions.
The rest are suggested optimisations.

_{🔄 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/4587 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 11/26/2020 **Status:** ❌ Closed **Base:** `master` ← **Head:** `NetworkManagerOptimization` --- ### 📝 Commits (5) - [`dc1cec9`](https://github.com/jellyfin/jellyfin/commit/dc1cec9fd82af8742cf2b20f2da1fdf1de6bd41d) Merge pull request #98 from jellyfin/master - [`2dacaa5`](https://github.com/jellyfin/jellyfin/commit/2dacaa529656463df2f5671b326559c753ac118f) Changes as recommended by ReSharper. - [`fcd0c9f`](https://github.com/jellyfin/jellyfin/commit/fcd0c9f1bef6cee37a58ad8660c153082470300a) Merge branch 'master' into NetworkManagerOptimization - [`f2290aa`](https://github.com/jellyfin/jellyfin/commit/f2290aaaf0533ac92b3ba6e816478a802943ed76) simplification - [`3e2972c`](https://github.com/jellyfin/jellyfin/commit/3e2972c893e25ab30fe2256e717e2eb85af793e8) Merge branch 'master' into NetworkManagerOptimization ### 📊 Changes **7 files changed** (+347 additions, -362 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Networking/Configuration/NetworkConfigurationExtensions.cs` (+0 -1) 📝 `Jellyfin.Networking/Manager/NetworkManager.cs` (+195 -173) 📝 `MediaBrowser.Common/Net/INetworkManager.cs` (+0 -1) 📝 `MediaBrowser.Common/Net/IPHost.cs` (+58 -62) 📝 `MediaBrowser.Common/Net/IPNetAddress.cs` (+23 -23) 📝 `MediaBrowser.Common/Net/IPObject.cs` (+66 -64) 📝 `MediaBrowser.Common/Net/NetworkExtensions.cs` (+5 -38) </details> ### 📄 Description Implemented changes as recommended by ReSharper. 99.9% of the changes are code nesting reductions. The rest are suggested optimisations. --- <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:15:24 +03:00

OVERLORD closed this issue

2026-02-07 06:15:26 +03:00

OVERLORD referenced this issue

2026-02-07 06:56:56 +03:00

[PR #10448] Forward user_agent config to ffprobe #12496

OVERLORD referenced this issue

2026-02-07 07:05:46 +03:00

[PR #11857] Fix ffprobe -user_agent parameter #13019

Sign in to join this conversation.