From f4ee04dda52b90c37f3b19583c4a8ba37d67abb8 Mon Sep 17 00:00:00 2001 From: Brian Popow Date: Sun, 29 Nov 2020 17:10:01 +0100 Subject: [PATCH] Avoid LINQ in ReadTransformation --- .../Formats/WebP/Lossless/WebPLosslessDecoder.cs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/ImageSharp/Formats/WebP/Lossless/WebPLosslessDecoder.cs b/src/ImageSharp/Formats/WebP/Lossless/WebPLosslessDecoder.cs index f115fd6b70..2c7248fe3e 100644 --- a/src/ImageSharp/Formats/WebP/Lossless/WebPLosslessDecoder.cs +++ b/src/ImageSharp/Formats/WebP/Lossless/WebPLosslessDecoder.cs @@ -629,10 +629,12 @@ namespace SixLabors.ImageSharp.Formats.Experimental.WebP.Lossless var transform = new Vp8LTransform(transformType, xSize, ySize); // Each transform is allowed to be used only once. - // TODO: No Linq, avoid 'transform' closure allocation. - if (decoder.Transforms.Any(t => t.TransformType == transform.TransformType)) + foreach (Vp8LTransform decoderTransform in decoder.Transforms) { - WebPThrowHelper.ThrowImageFormatException("Each transform can only be present once"); + if (decoderTransform.TransformType == transform.TransformType) + { + WebPThrowHelper.ThrowImageFormatException("Each transform can only be present once"); + } } switch (transformType)