6 Apr
2010
6 Apr
'10
1:56 p.m.
Wolfram Sang <wolfram(a)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.