6 Apr
2010
6 Apr
'10
1:56 p.m.
Wolfram Sang wolfram@the-dreams.de wrote:
offbits = bmfh->bfOffBits - sizeof(BITMAPFILEHEADER);
- bits = (char *)info + (offbits ?: size);
The purpose of offbits is not clear. Besides "?:" is absolutely not readable, putting an explicit 0 there would help a bit. Adding a test case wouldn't hurt either.
--
Dmitry.