Subject: Re: xsrc/31163
To: None <macallan@netbsd.org, gnats-admin@netbsd.org,>
From: Michael Lorenz <macallan@netbsd.org>
List: netbsd-bugs
Date: 09/06/2005 10:40:02
The following reply was made to PR xsrc/31163; it has been noted by GNATS.

From: Michael Lorenz <macallan@netbsd.org>
To: Martin Husemann <martin@duskware.de>
Cc: macallan@NetBSD.org, NetBSD GNATS <gnats-bugs@netbsd.org>
Subject: Re: xsrc/31163
Date: Tue, 6 Sep 2005 06:39:19 -0400

 --Signature_Tue__6_Sep_2005_06_39_19_-0400_7Rr45s3MJNInLO_t
 Content-Type: text/plain; charset=US-ASCII
 Content-Transfer-Encoding: quoted-printable
 
 Hello,
 
 > Nope, it's not that simple. True, the "depth" argument returned from
 > setup_visual() is not passed to open_window() - but after changing
 > that (and using the new arg in open_window) it still fails on first
 > color allocation. There must be more to it...
 
 Yes, X also wants a colour map. You can't inherit it because the parent
 as a 24bit window doesn't have one and DefaultColormap() doesn't
 something suitable either as long as the default visual is 24bit.=20
 
 > So there are two things:
 >=20
 >  (1) the app is easily fixed by making the check for
 >      DefaultVisual()->class accept TrueColor as well as PseudoColor
 
 It does, but it checks for PseudoColor availability first and since
 that's available on ffb it tries to use it.
 
 >  (2) if the server returns a visual array including a PseudoColor 8
 >      bit depth visual, it should be usable - it's a server error,
 >      otherwise (and the failure is internal, isn't it?)
 
 No. The server wants a colour map on window creation if it can't inherit
 one somewhere ( that's ancient code in Xserver/dix/window.c ). In this
 case things should Just Work(tm) when xuvmstat would just pick the
 TrueColor visual or inherit both depth and visual.
 
 have fun
 Michael
 
 --Signature_Tue__6_Sep_2005_06_39_19_-0400_7Rr45s3MJNInLO_t
 Content-Type: application/pgp-signature
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.0 (NetBSD)
 
 iQEVAwUBQx1x3cpnzkX8Yg2nAQJCPggAkwtsm0jf6MebZZjpnh0uckCdZ3NjIKFx
 CZyNvgPxZ4GdtgFVxQhRGDfUdIA9RFn+Eg8EErCOKddm+KR5q8Fop8w8qzPRTjdh
 sV4DAnMa44mhknzyseuRVhi+yf/N3EznkSgMxBkj4stK3EfxZs8ruMuV/GOUHYXm
 COk+aJkntZK54KsO6Yjmh4G6y//H6m+ejKF2TsIh6TDm51w6nzrRnUsMz1tY/cNA
 oxzgf2AIEKIkQ/6rGC87FC0yTV2q2YQiIDB8dF7amMPQ+C/F7DRWKNXyors6lV1j
 qw/iMw66XUF1lqK0RisPTZiLWqehoWZr23tn6nMi0PY62Cs+Anb+MA==
 =9Ql5
 -----END PGP SIGNATURE-----
 
 --Signature_Tue__6_Sep_2005_06_39_19_-0400_7Rr45s3MJNInLO_t--