Wolfram Sang <wolf...@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.