From 225490163e0bda089deaf46a77c19eb42cbc134a Mon Sep 17 00:00:00 2001 From: Rangi <35663410+Rangi42@users.noreply.github.com> Date: Tue, 29 Jul 2025 14:52:20 -0400 Subject: [PATCH] Fix RGBGFX "decanting" on "components" (color sets sharing colors) (#1768) --- src/gfx/pal_packing.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gfx/pal_packing.cpp b/src/gfx/pal_packing.cpp index 3c9db3a0..46f086b7 100644 --- a/src/gfx/pal_packing.cpp +++ b/src/gfx/pal_packing.cpp @@ -300,15 +300,15 @@ static void decant(std::vector &assignments, std::vector Options::VERB_DEBUG, "%zu palettes after decanting on palettes\n", assignments.size() ); - // Decant on "components" (= color sets sharing colors) + // Decant on "components" (color sets sharing colors) decantOn([&colorSets](AssignedSets &to, AssignedSets &from) { // We need to iterate on all the "components", which are groups of color sets sharing at - // least one color with another color sets in the group. + // least one color with another color set in the group. // We do this by adding the first available color set, and then looking for palettes with // common colors. (As an optimization, we know we can skip palettes already scanned.) std::vector processed(from.nbColorSets(), false); for (std::vector::iterator iter; - (iter = std::find(RANGE(processed), true)) != processed.end();) { + (iter = std::find(RANGE(processed), false)) != processed.end();) { auto attrs = from.begin(); std::advance(attrs, iter - processed.begin());