Re: [AD] al_update_display_region

[Milan M. <mil...@gm...>]

Elias Pschernig wrote:
> 
> Well, I get the below. ex_expose has this for some reason:
> 
> al_set_new_display_format(ALLEGRO_PIXEL_FORMAT_ANY_32_NO_ALPHA);
> 
> If that is wrong, what should it be instead?
> ALLEGRO_PIXEL_FORMAT_ANY_NO_ALPHA? ALLEGRO_PIXEL_FORMAT_ANY_32?
> ALLEGRO_PIXEL_FORMAT_ANY_24? Or should I just remove the call
> completely?
> 
> I guess it would be better to have a requested format only fail if
> absolutely necessary. In this case, all the 24 bit formats with alpha 0
> are what the user intended - they are RGBX after all, just GLX doesn't
> report that "X" separately as bits.

24 bit and 0 alpha is not RGBX, that's RGB. ALLEGRO_PIXEL_FORMAT_ANY_NO_ALPHA 
covers both. I don't know why ex_expose had that call.



-- 
Milan Mimica
http://sparklet.sf.net