Re: [AD] retrace_proc is wrong

[Chris <kc...@st...>]

Peter Wang wrote:
> retrace_proc works just like a "timer" callback, so could be run from a 
> background thread.  This means things like altering the palette should 
> not be allowed

Why? As long as you don't set the palette in two threads at once I don't 
see a real problem. Though yes, it's usefulness is still questionable.

> I suggest we deprecate retrace_proc in the unstable branch.  Given its 
> uselessness, I would like to see it removed.  It could be retained as a 
> DOS-only feature, but deprecated.

Can you deprecate variables? I think doing that would also cause Allegro 
to spit out a bunch of deprecation warnings in where it itself uses it 
(since it would still need to maintain compatibility by using it). There 
is a flag to make GCC not warn about using derprecated stuff, but then I 
fear we wouldn't catch everything in the future when we deprecate more 
things.

- Kitty Cat