Wolfram Sang <wolfram at 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.