Re: [AD] I've made a modification to the set_gfx_mode function

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


Eric Botcazou <ebotcazou@xxxxxxxxxx> wrote:

> I think Evert's point of view is rather sensible: each GFX_AUTODETECT*
mode
> is meant to have its own gfx_card entry. Since the default modes are
> (generally) fullscreen in Allegro, GFX_AUTODETECT and
> GFX_AUTODETECT_FULLSCREEN can share the same entry, but we need one for
> GFX_AUTODETECT_WINDOWED.

I think so too. I'll put this one in the repository.

Peter Wang <tjaden@xxxxxxxxxx> wrote:

> Does anyone have thoughts on this, either way?  The only problem I
> have with it is that it makes `set_gfx_mode' even more convoluted
> (that monster becomes 334 lines).

Well, it IS mentioned in the todo list that it needs a rewrite. Be my guest,
put it on a diet. :)

-henrik




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/