Re: [AD] al_toggle_display_flag ALLEGRO_NOFRAME

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


On Sat Mar 3, 2012, Thomas Fjellstrom wrote:
> On Sat Mar 3, 2012, Elias Pschernig wrote:
> > On Sun, 4 Mar 2012 12:01:45 +1100
> > 
> > Peter Wang <novalazy@xxxxxxxxxx> wrote:
> > > Hi,
> > > 
> > > Why is the verb "toggle" instead of just "set"?
> > > 5.1 adds al_toggle_menu_item_flags -- that should be reconsidered.
> > 
> > Can we rename it to al_set_display_flag?
> 
> Don't we also now need a unset_display flag? My assumption was that toggle
> acted on a given existing display. So there would need to be a set and
> unset pair to take the place of toggle.

Nevermind.

> > > The NOFRAME flag is just confusing.  Both the Windows and X11
> > > implementations got al_toggle_display_flag(ALLEGRO_NOFRAME) wrong.
> > > 
> > > Unfortunately we have a few negative flags already, but we should
> > > avoid adding more.
> > 
> > I don't think they are so bad, else we get confusion with some of the
> > flags being on by default and others off. Right now (I think) they are
> > all off by default. In this particular case something like
> > ALLEGRO_BORDERLESS might be better. But it's also negative so don't
> > know.
> > 
> > -------------------------------------------------------------------------
> > -- --- Virtualization & Cloud Management Using Capacity Planning
> > Cloud computing makes use of virtualization - but cloud computing
> > also focuses on allowing computing to be delivered as a service.
> > http://www.accelacomm.com/jaw/sfnl/114/51521223/


-- 
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx




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