Joel Holdsworth joel@airwebreathe.org.uk writes:
Taking this into account would yield:
ok ((result & 0x00F8F8F8) == (modern_expected & 0x00F8F8F8) || /* Windows 2000 and up */ broken((result & 0x00F8F8F8) == (legacy_expected & 0x00F8F8F8)) || /* Windows NT 4.0, 9X and below */ broken(GetDeviceCaps(hdc, BITSPIXEL) <= 8), ... /* Windows NT 4.0, 9X and below at 8bpp or less*/
What do you think?
You can just skip the whole test on 8-bit displays. And please add some sort of color_match() function for the comparison, don't hardcode 0x00F8F8F8 all over the place.