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


Reply via email to