On Wed May 14 12:31:20 2025 +0000, Connor McAdams wrote:
So, IIRC, I originally did have it the way you're describing here and then I later changed it once I remembered we always initialize missing channels to the maximum value (0xff) when we convert to the color key format. So I think either way would work, having a range of `0xff->0xff` or `0x00->0xff`. But, having it as `0x00->0xff` would make it clearer that we're checking against a channel that isn't present in the source format. I might be missing something though, I can't think of a case where this being `0xff->0xff` might fail.
No, I don't think you are... I was pretty confident that there was something off here but today I don't see it.
MR might be cursed :sweat_smile: