From fb8003c80d1a37e8f3a3128ac7771ac326371942 Mon Sep 17 00:00:00 2001 From: James Jackson-South Date: Thu, 15 Dec 2022 10:12:56 +1000 Subject: [PATCH] Remove double clamping --- .../Implementation/Icc/Calculators/ClutCalculator.cs | 2 -- .../Implementation/Icc/Calculators/LutEntryCalculator.cs | 3 +-- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/ClutCalculator.cs b/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/ClutCalculator.cs index 4b65f46a8..a1891d127 100644 --- a/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/ClutCalculator.cs +++ b/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/ClutCalculator.cs @@ -29,8 +29,6 @@ internal class ClutCalculator : IVector4Calculator public unsafe Vector4 Calculate(Vector4 value) { - value = Vector4.Clamp(value, Vector4.Zero, Vector4.One); - Vector4 result = default; this.Interpolate((float*)&value, this.inputCount, (float*)&result, this.outputCount); diff --git a/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/LutEntryCalculator.cs b/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/LutEntryCalculator.cs index 643090c9d..a5645266f 100644 --- a/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/LutEntryCalculator.cs +++ b/src/ImageSharp/ColorSpaces/Conversion/Implementation/Icc/Calculators/LutEntryCalculator.cs @@ -41,9 +41,8 @@ internal class LutEntryCalculator : IVector4Calculator } [MethodImpl(MethodImplOptions.AggressiveInlining)] - private static unsafe Vector4 CalculateLut(LutCalculator[] lut, Vector4 value) + private static Vector4 CalculateLut(LutCalculator[] lut, Vector4 value) { - value = Vector4.Clamp(value, Vector4.Zero, Vector4.One); ref float f = ref Unsafe.As(ref value); for (int i = 0; i < lut.Length; i++) {