Joel Holdsworth <[email protected]> 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. -- Alexandre Julliard [email protected]
