It seems there is either a documentation vs. reality mismatch or a real bug
The exact blit in question is a 32bpp RGBA->RGB, where RGBA has SDL_COLORKEY
and no SDL_SRCALPHA flags. The doc in SDL_video.h states:
SDL_SRCALPHA not set:
- copy RGB.
- if SDL_SRCCOLORKEY set, only copy the pixels matching the
- RGB values of the source colour key, ignoring alpha in the
BlitNtoNKey(), however, forgets to “ignore alpha in the comparison”. The
documentation makes perfect sense, so I think it is the code that is faulty.
The attached patch corrects the code.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed…