Re: [AD] al_update_display_region

[Elias Pschernig <elias.pschernig@xxxxxxxxxx>]

On Wed, 2009-02-25 at 20:41 +0100, Milan Mimica wrote:
> Elias Pschernig wrote:
> > 
> > Something like A4's old X11 driver also would implement it. I think it
> > should be more a hint to the display driver, something like "do an
> > al_flip_display, but really only this rectangle would require updating".
> > 
> > If you want to have two code pathes, one with a DRS, one without, you
> > likely want to know this at the start of the program - not later when
> > you actually try to call the function.
> 
> Okay, that makes sense.
> 
> I don't think it should be a flag, but a option.
> al_get_display_option(ALLEGRO_UPDATE_DISPLAY_REGION) will indicate whether
> a) al_update_display_region() does nothing
> b) acts as a hint
> c) does what the name says
> 

Well, a) is not needed if we require drivers to always implement it -
and I think we should. There really is no reason to allow NULL entries
in vtables. Changed it accordingly in SVN (Windows and OSX versions are
untested, I hope I didn't break anything there).

-- 
Elias Pschernig <elias@xxxxxxxxxx>