In dlls/winedos/int10.c we currently have the following code
VGA_SetBright((BL_reg(context) & 0x10) && 1);
which I am not sure I understand. Is the purpose of this to only pass one of 0 or 1 to VGA_SetBright? If so, I guess I'd find "... ? 1 : 0" more intuitive, and it's only three characters more, or "!= 0" at least.
Thoughts?
Gerald