[PR #547] [CLOSED] Switch to external library for getting image sizes #8196

New issue

Closed

opened 2025-12-22 06:44:07 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 06:44:07 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/547
Author: @Bond-009
Created: 1/10/2019
Status: ❌ Closed

Base: dev ← Head: imagesize

📝 Commits (2)

e341701 Switch to external libraryfor getting image sizes
b2be514 Cleanup ImageProcessor

📊 Changes

7 files changed (+123 additions, -403 deletions)

View changed files

➖ Emby.Drawing/Common/ImageHeader.cs (+0 -243)
📝 Emby.Drawing/Emby.Drawing.csproj (+4 -0)
📝 Emby.Drawing/ImageProcessor.cs (+114 -155)
📝 Emby.Drawing/NullImageEncoder.cs (+1 -1)
📝 Emby.Server.Implementations/ApplicationHost.cs (+1 -1)
📝 Emby.Server.Implementations/Dto/DtoService.cs (+2 -2)
📝 MediaBrowser.Controller/Drawing/IImageProcessor.cs (+1 -1)

📄 Description

I wrote ImageSize.Net from scratch to replace ImageHeader.cs which
supports only the most common versions of the image formats.
ImageSize.Net support more edge cases and has been heavily testing
(in contrast to ImageHeader.cs which was copied from SO)
Project link: https://github.com/Bond-009/ImageSize.Net

_{🔄 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/547 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 1/10/2019 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `imagesize` --- ### 📝 Commits (2) - [`e341701`](https://github.com/jellyfin/jellyfin/commit/e3417019f34fff350b87f2e891dc3df8130239e9) Switch to external libraryfor getting image sizes - [`b2be514`](https://github.com/jellyfin/jellyfin/commit/b2be514286489cedc76c4b1f7c58a4c1258bc4f5) Cleanup ImageProcessor ### 📊 Changes **7 files changed** (+123 additions, -403 deletions) <details> <summary>View changed files</summary> ➖ `Emby.Drawing/Common/ImageHeader.cs` (+0 -243) 📝 `Emby.Drawing/Emby.Drawing.csproj` (+4 -0) 📝 `Emby.Drawing/ImageProcessor.cs` (+114 -155) 📝 `Emby.Drawing/NullImageEncoder.cs` (+1 -1) 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+1 -1) 📝 `Emby.Server.Implementations/Dto/DtoService.cs` (+2 -2) 📝 `MediaBrowser.Controller/Drawing/IImageProcessor.cs` (+1 -1) </details> ### 📄 Description I wrote ImageSize.Net from scratch to replace ImageHeader.cs which supports only the most common versions of the image formats. ImageSize.Net support more edge cases and has been heavily testing (in contrast to ImageHeader.cs which was copied from SO) Project link: https://github.com/Bond-009/ImageSize.Net --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 06:44:07 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 08:46:23 +01:00

[PR #8196] [MERGED] Bump sharpcompress from 0.32.1 to 0.32.2 #11468

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#8196

No description provided.

Rows
Columns