From c64ac5f8ecaa63707f72eaabf8a9ba94032feb04 Mon Sep 17 00:00:00 2001 From: Dmitry Pentin Date: Wed, 28 Jul 2021 00:19:52 +0300 Subject: [PATCH] Comments, quantization table scaling with clamp call --- .../Formats/Jpeg/Components/Quantization.cs | 15 ++------------- src/ImageSharp/Formats/Jpeg/JpegMetadata.cs | 7 +++++++ 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/src/ImageSharp/Formats/Jpeg/Components/Quantization.cs b/src/ImageSharp/Formats/Jpeg/Components/Quantization.cs index 7e528d056..394ad71af 100644 --- a/src/ImageSharp/Formats/Jpeg/Components/Quantization.cs +++ b/src/ImageSharp/Formats/Jpeg/Components/Quantization.cs @@ -179,19 +179,8 @@ namespace SixLabors.ImageSharp.Formats.Jpeg.Components Block8x8F table = default; for (int j = 0; j < Block8x8F.Size; j++) { - int x = unscaledTable[j]; - x = ((x * scale) + 50) / 100; - if (x < 1) - { - x = 1; - } - - if (x > 255) - { - x = 255; - } - - table[j] = x; + int x = ((unscaledTable[j] * scale) + 50) / 100; + table[j] = Numerics.Clamp(x, 1, 255); } return table; diff --git a/src/ImageSharp/Formats/Jpeg/JpegMetadata.cs b/src/ImageSharp/Formats/Jpeg/JpegMetadata.cs index 1b17bdce7..0a4b970f4 100644 --- a/src/ImageSharp/Formats/Jpeg/JpegMetadata.cs +++ b/src/ImageSharp/Formats/Jpeg/JpegMetadata.cs @@ -11,7 +11,14 @@ namespace SixLabors.ImageSharp.Formats.Jpeg /// public class JpegMetadata : IDeepCloneable { + /// + /// Backing field for + /// private int? luminanceQuality; + + /// + /// Backing field for + /// private int? chrominanceQuality; ///