[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workar
From: |
Eli Zaretskii |
Subject: |
bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround |
Date: |
Fri, 30 Dec 2011 11:00:20 +0200 |
> Date: Thu, 29 Dec 2011 19:18:13 -0800
> From: Daniel Colascione <dancol@dancol.org>
> Cc: 10397@debbugs.gnu.org
>
> > NUMCOLORS
> > Number of entries in the device's color table, if the device has a
> > color depth of no more than 8 bits per pixel. For devices with greater
> > color depths, 1 is returned.
> >
> > (It says "1", but it's a typo for "-1".)
>
> Good catch. What about this (untested) code?
>
> hdc = GetDC (dpyinfo->root_window);
> if (dpyinfo->has_palette)
> cap = GetDeviceCaps (hdc, SIZEPALETTE);
> else if (dpyinfo->n_cbits <= 8)
> /* According to the MSDN, GetDeviceCaps (NUMCOLORS) is valid only
> for devices with at most eight bits per pixel. It's supposed
> to return -1 for other displays, but because it actually
> returns other, incorrect values under some conditions (e.g.,
> remote desktop), only use it when we know it's valid. */
> cap = GetDeviceCaps (hdc, NUMCOLORS);
> else
> cap = -1;
Looks good to me. I think this could go into 24.1, unless Jason
(or someone else) objects.
As I wrote elsewhere, past 24.1, we could explore the possibility of
not calling GetDeviceCaps here at all (assuming that using the number
of planes and bits per plane does the job even when the latter is 8 or
less).
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, (continued)
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Daniel Colascione, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Daniel Colascione, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Daniel Colascione, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Daniel Colascione, 2011/12/29
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Eli Zaretskii, 2011/12/30
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround,
Eli Zaretskii <=
- bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Juanma Barranquero, 2011/12/30
bug#10397: [PATCH] Under Remote Desktop, NUMCOLORS is unreliable; workaround, Jason Rumney, 2011/12/29