Re: [AD] SF.net SVN: alleg:[13312] allegro/branches/4.9

[Peter Wang <novalazy@xxxxxxxxxx>]

On 25 April 2010 05:49, Thomas Fjellstrom <tfjellstrom@xxxxxxxxxx> wrote:
> On April 24, 2010, tjaden@xxxxxxxxxx wrote:
>> Revision: 13312
>>           http://alleg.svn.sourceforge.net/alleg/?rev=13312&view=rev
>> Author:   tjaden
>> Date:     2010-04-24 14:28:41 +0000 (Sat, 24 Apr 2010)
>>
>> Log Message:
>> -----------
>> Init xinerama/randr/xf86vm interfaces lazily.
>
> I'm not entirely sure this is a good idea. Especially once xrandr events are
> supported, xrandr will need to be inited on startup.

Ok, we'll see.

>> Getting the information from randr can cause flickers and background
>> sound to stutter, so delay doing so until actually necessary.
>
> So why not do it at the beginning?

Because every time I start an Allegro program those things happen, whether or
not the program cares about multi-monitor information or changing the screen
resolution.

>> +static void init_mmon_interface(ALLEGRO_SYSTEM_XGLX *s)
>> +{
>> +   if (s->mmon_interface_inited)
>> +      return;
>> +
>> +   /* Shouldn't we avoid initing any more of these than we need? */
>> +
>
> You could probably avoid initing xf86vm if you really want to. But Xinerama
> is tightly intertwined with both the xf86vm and XRandR code.

Ok.

Peter