On Tue Feb 4 16:44:01 2025 +0000, Rémi Bernon wrote:
It's as misleading as this is, data->use_alpha is also incorrect and it's basically exactly the same as here: it is set to TRUE if visual != default_visual, which doesn't actually say anything about alpha.
Well, the difference is that escape.use_alpha is set in the place where such things are managed, while xrender doesn't need to have logic tangled with the entirety of winex11.drv. data->use_alpha is set whenever we consider alpha-enabled visual, for ULW windows and conditionally for systray. If we set use_alpha because we wanted alpha but our visual doesn't have it then the proper xrender format will be selected based on visual depth check? In your variant what is going to make sure if we are going to actually select argb visual when we want it? depth can be 32 both for, e. g., WXR_FORMAT_A8R8G8B8 / WXR_FORMAT_X8R8G8B8. Any will be compatible as in avoiding X error probably, but we do want one or another depending on whether we do use alpha or not (and that is given by data->use_alpha)?