aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOdd Stråbø <oddstr13@openshell.no>2021-03-29 00:46:19 +0200
committerOdd Stråbø <oddstr13@openshell.no>2021-04-11 08:21:23 +0200
commite0edbc5754ad4d6afad1f01874ca2085d4f0791d (patch)
tree6b06ee68ae3d8dad56f877e2aa4b462f129b51d7
parentafff226514a6f645a6738f1e407208826b84f1d6 (diff)
Apply suggestions from code review
Co-authored-by: Claus Vium <cvium@users.noreply.github.com>
-rw-r--r--Emby.Drawing/ImageProcessor.cs2
-rw-r--r--MediaBrowser.Controller/Drawing/ImageProcessingOptions.cs7
2 files changed, 2 insertions, 7 deletions
diff --git a/Emby.Drawing/ImageProcessor.cs b/Emby.Drawing/ImageProcessor.cs
index 646620c43..78b53f36a 100644
--- a/Emby.Drawing/ImageProcessor.cs
+++ b/Emby.Drawing/ImageProcessor.cs
@@ -274,7 +274,7 @@ namespace Emby.Drawing
string backgroundColor,
string foregroundLayer)
{
- System.Text.StringBuilder filename = new System.Text.StringBuilder(128);
+ var filename = new StringBuilder(128);
filename.Append(originalPath);
filename.Append(",quality=");
diff --git a/MediaBrowser.Controller/Drawing/ImageProcessingOptions.cs b/MediaBrowser.Controller/Drawing/ImageProcessingOptions.cs
index e1a2811ac..230a0af60 100644
--- a/MediaBrowser.Controller/Drawing/ImageProcessingOptions.cs
+++ b/MediaBrowser.Controller/Drawing/ImageProcessingOptions.cs
@@ -97,12 +97,7 @@ namespace MediaBrowser.Controller.Drawing
return false;
}
- if (FillWidth.HasValue && sizeValue.Width > FillWidth.Value)
- {
- return false;
- }
-
- if (FillHeight.HasValue && sizeValue.Height > FillHeight.Value)
+ if (sizeValue.Width > FillWidth || sizeValue.Height > FillHeight)
{
return false;
}