From a2955501d3d1a3da08db2c6684d9cf9f776726bf Mon Sep 17 00:00:00 2001 From: Lucas CHOLLET Date: Fri, 21 Jul 2023 14:48:11 -0400 Subject: [PATCH] LibGfx/JPEGXL: Don't decode the header twice This is something I missed when I ported the JPEG XL decoder to the new plugin interface (decoding the header at creation). First sorry because that's entirely my fault, second sorry because a test should have caught that. --- Userland/Libraries/LibGfx/ImageFormats/JPEGXLLoader.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibGfx/ImageFormats/JPEGXLLoader.cpp b/Userland/Libraries/LibGfx/ImageFormats/JPEGXLLoader.cpp index 3886350862..990f202c06 100644 --- a/Userland/Libraries/LibGfx/ImageFormats/JPEGXLLoader.cpp +++ b/Userland/Libraries/LibGfx/ImageFormats/JPEGXLLoader.cpp @@ -1639,7 +1639,7 @@ public: auto result = [this]() -> ErrorOr { // A.1 - Codestream structure - TRY(decode_image_header()); + // The header is already decoded in JPEGXLImageDecoderPlugin::create() if (m_metadata.colour_encoding.want_icc) TODO();