[PR #4929] [CLOSED] Enable nullable for MediaBrowser.Providers project #10332

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4929
Author: @martinek-stepan
Created: 1/2/2021
Status: Closed

Base: masterHead: nullable-providers


📝 Commits (3)

  • 13eda0c Enable nullable for MediaBrowser.Providers project
  • 2c3d976 Fixed errors in Release build
  • f9648a8 More errors after rebase to new head

📊 Changes

55 files changed (+540 additions, -440 deletions)

View changed files

📝 Jellyfin.Api/Controllers/RemoteImageController.cs (+1 -1)
📝 MediaBrowser.Controller/Providers/IProviderManager.cs (+3 -2)
📝 MediaBrowser.MediaEncoding/Attachments/AttachmentExtractor.cs (+1 -1)
📝 MediaBrowser.MediaEncoding/BdInfo/BdInfoDirectoryInfo.cs (+3 -3)
📝 MediaBrowser.MediaEncoding/BdInfo/BdInfoFileInfo.cs (+1 -1)
📝 MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs (+5 -5)
📝 MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs (+30 -24)
📝 MediaBrowser.MediaEncoding/MediaBrowser.MediaEncoding.csproj (+2 -1)
📝 MediaBrowser.MediaEncoding/Probing/FFProbeHelpers.cs (+1 -1)
📝 MediaBrowser.MediaEncoding/Probing/InternalMediaInfoResult.cs (+3 -3)
📝 MediaBrowser.MediaEncoding/Probing/MediaChapter.cs (+4 -4)
📝 MediaBrowser.MediaEncoding/Probing/MediaFormatInfo.cs (+8 -8)
📝 MediaBrowser.MediaEncoding/Probing/MediaStreamInfo.cs (+33 -33)
📝 MediaBrowser.MediaEncoding/Probing/ProbeResultNormalizer.cs (+19 -19)
📝 MediaBrowser.MediaEncoding/Subtitles/AssParser.cs (+12 -2)
📝 MediaBrowser.MediaEncoding/Subtitles/SrtParser.cs (+1 -1)
📝 MediaBrowser.MediaEncoding/Subtitles/SsaParser.cs (+1 -1)
📝 MediaBrowser.MediaEncoding/Subtitles/SubtitleEncoder.cs (+14 -10)
📝 MediaBrowser.Model/Entities/ProviderIdsExtensions.cs (+2 -2)
📝 MediaBrowser.Providers/Manager/ImageSaver.cs (+39 -18)

...and 35 more files

📄 Description

Changes

Enable nullable on the project level and reflect changes in code.

Issues

Part of issue #2768


🔄 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/4929 **Author:** [@martinek-stepan](https://github.com/martinek-stepan) **Created:** 1/2/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `nullable-providers` --- ### 📝 Commits (3) - [`13eda0c`](https://github.com/jellyfin/jellyfin/commit/13eda0cd4d0c503e37dae8ffa059f49c398713ff) Enable nullable for MediaBrowser.Providers project - [`2c3d976`](https://github.com/jellyfin/jellyfin/commit/2c3d97643706e6080fdc911813578b41a05fbbc6) Fixed errors in Release build - [`f9648a8`](https://github.com/jellyfin/jellyfin/commit/f9648a860d64fa11bece1a48fe575f340e73882c) More errors after rebase to new head ### 📊 Changes **55 files changed** (+540 additions, -440 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/RemoteImageController.cs` (+1 -1) 📝 `MediaBrowser.Controller/Providers/IProviderManager.cs` (+3 -2) 📝 `MediaBrowser.MediaEncoding/Attachments/AttachmentExtractor.cs` (+1 -1) 📝 `MediaBrowser.MediaEncoding/BdInfo/BdInfoDirectoryInfo.cs` (+3 -3) 📝 `MediaBrowser.MediaEncoding/BdInfo/BdInfoFileInfo.cs` (+1 -1) 📝 `MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs` (+5 -5) 📝 `MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs` (+30 -24) 📝 `MediaBrowser.MediaEncoding/MediaBrowser.MediaEncoding.csproj` (+2 -1) 📝 `MediaBrowser.MediaEncoding/Probing/FFProbeHelpers.cs` (+1 -1) 📝 `MediaBrowser.MediaEncoding/Probing/InternalMediaInfoResult.cs` (+3 -3) 📝 `MediaBrowser.MediaEncoding/Probing/MediaChapter.cs` (+4 -4) 📝 `MediaBrowser.MediaEncoding/Probing/MediaFormatInfo.cs` (+8 -8) 📝 `MediaBrowser.MediaEncoding/Probing/MediaStreamInfo.cs` (+33 -33) 📝 `MediaBrowser.MediaEncoding/Probing/ProbeResultNormalizer.cs` (+19 -19) 📝 `MediaBrowser.MediaEncoding/Subtitles/AssParser.cs` (+12 -2) 📝 `MediaBrowser.MediaEncoding/Subtitles/SrtParser.cs` (+1 -1) 📝 `MediaBrowser.MediaEncoding/Subtitles/SsaParser.cs` (+1 -1) 📝 `MediaBrowser.MediaEncoding/Subtitles/SubtitleEncoder.cs` (+14 -10) 📝 `MediaBrowser.Model/Entities/ProviderIdsExtensions.cs` (+2 -2) 📝 `MediaBrowser.Providers/Manager/ImageSaver.cs` (+39 -18) _...and 35 more files_ </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> Enable nullable on the project level and reflect changes in code. **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Part of issue #2768 --- <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:18:23 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10332