Code quality

[RomekAtomek]

Is there a reason to call GetDIBits twice in the vncdesktop.cpp file?

line 1118

result = ::GetDIBits(m_hmemdc, m_membitmap, 0, 1, NULL, &m_bminfo.bmi, DIB_RGB_COLORS);
if (result == 0) {
vnclog.Print(LL_INTERR, VNCLOG("unable to get display format\n"));
return ERROR_DESKTOP_NO_DISPLAYFORMAT;
}
result = ::GetDIBits(m_hmemdc, m_membitmap, 0, 1, NULL, &m_bminfo.bmi, DIB_RGB_COLORS);
if (result == 0) {
vnclog.Print(LL_INTERR, VNCLOG("unable to get display colour info\n"));
return ERROR_DESKTOP_NO_DISPLAYFORMAT;
}

[Rudi De Vos]

is needed