[PR #6013] [MERGED] Minor improvements #10780

Closed
opened 2026-02-07 06:26:32 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6013
Author: @Bond-009
Created: 5/8/2021
Status: Merged
Merged: 5/21/2021
Merged by: @Bond-009

Base: masterHead: minor13


📝 Commits (1)

📊 Changes

4 files changed (+84 additions, -78 deletions)

View changed files

📝 Jellyfin.Api/Controllers/DynamicHlsController.cs (+26 -28)
📝 Jellyfin.Api/Helpers/TranscodingJobHelper.cs (+0 -1)
📝 MediaBrowser.Controller/MediaEncoding/EncodingJobOptions.cs (+0 -49)
tests/Jellyfin.Api.Tests/Controllers/DynamicHlsControllerTests.cs (+58 -0)

📄 Description

  • properly dispose CancellationTokenSource
  • rewrite DynamicHlsController.GetSegmentLengths
  • remove dead code

🔄 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/6013 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 5/8/2021 **Status:** ✅ Merged **Merged:** 5/21/2021 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `minor13` --- ### 📝 Commits (1) - [`56ac64e`](https://github.com/jellyfin/jellyfin/commit/56ac64e70a5cd4b4b4de78a0fc22c8bfe96a3677) Minor improvements ### 📊 Changes **4 files changed** (+84 additions, -78 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/DynamicHlsController.cs` (+26 -28) 📝 `Jellyfin.Api/Helpers/TranscodingJobHelper.cs` (+0 -1) 📝 `MediaBrowser.Controller/MediaEncoding/EncodingJobOptions.cs` (+0 -49) ➕ `tests/Jellyfin.Api.Tests/Controllers/DynamicHlsControllerTests.cs` (+58 -0) </details> ### 📄 Description * properly dispose CancellationTokenSource * rewrite DynamicHlsController.GetSegmentLengths * remove dead code --- <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:26:32 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10780